-
-
Notifications
You must be signed in to change notification settings - Fork 167
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(Feature) Webhook should generate a new SSL certificate rather than reusing pe-internal-peadmin-mcollective-client.pem #140
Comments
I totally agree on this one , adding this to 4.0 milestone |
In the meantime, the readme points to
|
@acidprime @elyscape I was able to use the puppetdb private/public pair (it didn't like the master's public/private). If someone can point me at how to generate the pem files properly, I'd be glad to get started on this. |
For reference, the puppetdb package in PC1 installation creates the pem files, so you can order it by adding this to your profile containing r10k:
|
I was able to make the webhook (r10k 2.5.1 with module v4.0.2) work with FOSS (puppetserver 2.7.2) and the puppet certs using the following:
I also set the webhook user to puppet and it works once I changed the ownership of cache and environment files. Note: The puppetdb certs are owned by puppetdb with 0600 permissions and are not visible to the puppet user so using the puppetdb certs will require the webhook running as root user. |
Sinatra 2.0.0 was released on the 7th of May 2017. It requires rack ~> 2.0 and *that* requires ruby 2.2. Puppet 4 AIO ships with ruby 2.1, so pin sinatra to prevent errors like. ``` Error: Execution of '/opt/puppetlabs/puppet/bin/gem install --no-rdoc --no-ri sinatra' returned 1: ERROR: Error installing sinatra: rack requires Ruby version >= 2.2.2. ``` Fixes voxpupuli#140
Sinatra 2.0.0 was released on the 7th of May 2017. It requires rack ~> 2.0 and *that* requires ruby 2.2. Puppet 4 AIO ships with ruby 2.1, so pin sinatra to prevent errors like. ``` Error: Execution of '/opt/puppetlabs/puppet/bin/gem install --no-rdoc --no-ri sinatra' returned 1: ERROR: Error installing sinatra: rack requires Ruby version >= 2.2.2. ``` Fixes voxpupuli#140
Sinatra 2.0.0 was released on the 7th of May 2017. It requires rack ~> 2.0 and *that* requires ruby 2.2. Puppet 4 AIO ships with ruby 2.1, so pin sinatra to prevent errors like. ``` Error: Execution of '/opt/puppetlabs/puppet/bin/gem install --no-rdoc --no-ri sinatra' returned 1: ERROR: Error installing sinatra: rack requires Ruby version >= 2.2.2. ``` Fixes voxpupuli#140
Sinatra 2.0.0 was released on the 7th of May 2017. It requires rack ~> 2.0 and *that* requires ruby 2.2. Puppet 4 AIO ships with ruby 2.1, so pin sinatra to prevent errors like. ``` Error: Execution of '/opt/puppetlabs/puppet/bin/gem install --no-rdoc --no-ri sinatra' returned 1: ERROR: Error installing sinatra: rack requires Ruby version >= 2.2.2. ``` Fixes voxpupuli#140
Sorry, I referenced this issue by mistake in one of my commits. |
Currently, the webhook defaults to using the pe-internal-peadmin-mcollective-client SSL certificate. It would make more sense to generate an SSL cert specifically for the webhook.
The text was updated successfully, but these errors were encountered: