-
Notifications
You must be signed in to change notification settings - Fork 130
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 #24012 - Add PuppetCA providers settings #435
Refs #24012 - Add PuppetCA providers settings #435
Conversation
Thanks! |
@@ -9,12 +9,23 @@ Part of the Foreman installer: <https://github.com/theforeman/foreman-installer> | |||
|
|||
| Module version | Proxy versions | Notes | | |||
|----------------|----------------|-------------------------------------------------| | |||
| 5.x | 1.16 and newer | | | |||
| 5.x | 1.16 and newer | See compatibility notes here for 1.16-1.18 | |
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 Version 5? What about 6,7,8 and 9?
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 should be 5.x and newer. The breaking changes have been for different reasons than compatibility. Untested, but I think 9 should still allow managing 1.16. It'd be good to be explicit about this though.
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.
Updated in current master
| 4.x | 1.12 - 1.17 | See compatibility notes in its README for 1.15+ | | ||
| 3.x | 1.11 | | | ||
| 2.x | 1.5 - 1.10 | | | ||
| 1.x | 1.4 and older | | | ||
|
||
### Compatibility notes for Smart Proxy < 1.18 |
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.
Should be < 1.19
?
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.
You're right.
This adds the settings for the
puppetca_hostname_whitelisting
provider for the new modular PuppetCa module. This should fix the current nightlies.This is a part of #433 but without the
token_whitelisting
provider's settings and with the suggested additions.This is a working copy of #434 because I sometimes am unable to work with git properly. See previous commit.