-
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
Fixes #30950 - Enable SmartProxy Registration module #619
Conversation
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.
Please also write in the compatibility section in README.md that users on older versions should disable the module since it doesn't exist.
You may also need to create a template for the settings.
Acceptance tests will fail until it's merged in the smart proxy, but at least the unit tests can pass before that.
ab5b448
to
cf15e0d
Compare
cf15e0d
to
f45e805
Compare
f45e805
to
e4d0d6f
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.
Overall it looks good. Just wondering if we should enforce HTTPS-only for security.
) { | ||
foreman_proxy::module { 'registration': | ||
enabled => $enabled, | ||
feature => 'Registration', |
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.
Note to self: confirmed that this matches Foreman's feature name (theforeman/foreman@31b0d19)
03d643a
to
3c387b0
Compare
After a quick clarification, the pending thing here is to enforce https for this module. Then should be good to go. |
3c387b0
to
e743e5a
Compare
e743e5a
to
8e16fd2
Compare
Waiting for: theforeman/smart-proxy#768
PR is part of Simple & automatic host registration WF, see: