-
Notifications
You must be signed in to change notification settings - Fork 85
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
Refs #33337 - Reflect Puppet isn't enabled by default #2690
Conversation
Since Foreman 3.1 Puppet hasn't been enabled in the Katello scenario, but it is in vanilla Foreman.
The PR preview for d96a760 is available at theforeman-foreman-documentation-preview-pr-2690.surge.sh The following output files are affected by this PR: |
20c9b16
to
9d6e993
Compare
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.
This is great! Just a tiny detail.
Puppet is an implementation detail. Having it in gives the impression that Puppet runs as a daemon, but it doesn't.
9d6e993
to
d96a760
Compare
@Lennonka please merge this if you think it's good to go. |
Cherry-picked both commits to: Cherry-picked only the first commit to:
The second commit causes conflicts on 3.7 and older. FYI @ekohl |
Since Foreman 3.1 Puppet hasn't been enabled in the Katello scenario, but it is in vanilla Foreman. It also rephrases the installer managed parts to avoid the implication that Puppet runs as a deamon.
Please cherry-pick my commits into: