-
Notifications
You must be signed in to change notification settings - Fork 271
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
Remove unused parameters from puppetmaster #824
Conversation
These config settings are never actually read by anything.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ekohl it looks like the spec test create puppet.yaml
needs to be updated not to look for user/password since these are removed from the template.
Parameters with a single value aren't really useful.
Thanks, updated! (I grepped for foreman_user and found nothing) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The test failure here also looks like: https://projects.theforeman.org/issues/29429
merged, bedankt @ekohl! |
Hello. Maybe this is causing Katello Nightly installer to fail (installation worked ~5 days back, is not working now)?
|
Will be fixed with the next build which also contains theforeman/puppet-puppet#736 |
Please do you know when the next build will be ready? |
So, when can we expect this installation will work? |
No idea about RPMs, but DEBs are fixed since yesterday. |
It sounds like you're using staging repositories because this hasn't made it into the release repositories. |
These config settings are never actually read by anything.