[Puppet-dev] [PATCH/puppet 2/2] (#8147) Update test for default reporturl

2011-06-29 Thread nfagerlund
Edit test to accomodate the default URL change made in commit f6882d6d5779883e931a6f558c06f631098011c5. Reviewed-By: Jacob Helwig Signed-off-by: nfagerlund --- spec/integration/defaults_spec.rb |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/spec/integration/defaults_s

Re: [Puppet-dev] [PATCH/puppet 1/1] Prototype code for #7243 - Additional data in Puppet CSRs

2011-06-29 Thread James Turnbull
Eric Sorenson wrote: James -- you may already be onto this, but one thing I found last time I did a home-rolled PKI (attempting chained CA certs for puppet) was that the CA needs to be explicitly configured to accept extended attributes from the CSR and put them into the issued certificate. This

Re: [Puppet-dev] [PATCH/puppet 1/1] Prototype code for #7243 - Additional data in Puppet CSRs

2011-06-29 Thread Eric Sorenson
James -- you may already be onto this, but one thing I found last time I did a home-rolled PKI (attempting chained CA certs for puppet) was that the CA needs to be explicitly configured to accept extended attributes from the CSR and put them into the issued certificate. This is controlled by th

[Puppet-dev] [PATCH/puppet 1/1] Prototype code for #7243 - Additional data in Puppet CSRs

2011-06-29 Thread James Turnbull
This adds the ability to add arbitrary attributes to Puppet certificate requests. It is controlled by setting the allow_csr_attributes setting in the puppet.conf configuration file on the Puppet agent. allow_csr_attributes = true This option defaults to false. If set to true it looks for a

[Puppet-dev] [PATCH/puppet 2/2] Further fix for #7243 - Added certdnsnames to CSR

2011-06-29 Thread James Turnbull
This patch adds the contents of certdnsnames to the CSR being generated on the Puppet agent as Extensions. Signed-off-by: James Turnbull --- Local-branch: tickets/master/7243 lib/puppet/ssl/certificate_request.rb | 20 spec/unit/ssl/certificate_request_spec.rb |1 +

[Puppet-dev] [PATCH/puppet 2/2] Further fix for #7243 - Added certdnsnames to CSR

2011-06-29 Thread James Turnbull
This patch adds the contents of certdnsnames to the CSR being generated on the Puppet agent as Extensions. Signed-off-by: James Turnbull --- Local-branch: tickets/master/7243 lib/puppet/ssl/certificate_request.rb | 20 spec/unit/ssl/certificate_request_spec.rb |1 +

[Puppet-dev] [PATCH/puppet 1/1] (#8147) Change default reporturl to match newer Dashboard versions

2011-06-29 Thread nfagerlund
Puppet's default reporturl setting was http://localhost:3000/reports, which has been deprecated in Puppet Dashboard in favor of http://localhost:3000/reports/upload. As Dashboard is the first-class destination for the http report processor, this commit changes Puppet's default to match what current

[Puppet-dev] [PATCH/puppet-dashboard 1/1] (#8146) Change default DASHBOARD_URL in external_node to localhost

2011-06-29 Thread nfagerlund
In the most common scenario, in which Dashboard and puppet master are installed on the same system, Dashboard was unable to be used as an external node classifier without editing the script or changing the system's name resolution. This commit allows the most common case to work silently, by changi

[Puppet-dev] [PATCH/puppet 1/1] Prototype code for #7243 - Additional data in Puppet CSRs

2011-06-29 Thread James Turnbull
This adds the ability to add arbitrary attributes to Puppet certificate requests. It is controlled by setting the allow_csr_attributes setting in the puppet.conf configuration file on the Puppet agent. allow_csr_attributes = true This option defaults to false. If set to true it looks for a