-
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
Add Pulp 3 HTTP URLs #549
Add Pulp 3 HTTP URLs #549
Conversation
@@ -25,6 +25,10 @@ | |||
# | |||
# $pulp3_mirror:: Whether this pulp3 plugin acts as a mirror or another pulp node. A pulp3 mirror is the pulp3 equivalent of a pulpnode. | |||
# | |||
# $pulp3_api_url:: The URL to the Pulp 3 API | |||
# | |||
# $pulp3_content_url:: The URL to the Pulp 3 content |
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.
We've already used this convention in a few places, but in others we've identified that the use of pulp3
may come back to bite us and used pulpcore
.
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.
I don't mind renaming. Most of these predate the pulpcore convention. In https://github.com/theforeman/smart_proxy_pulp/ it's also the pulp3 module.
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.
Let's merge this and I'll raise an issue here to discuss further.
@jlsherrill these defaults make sense, right? Or should we default to the public URLs? |
This matches the pulp3.yml.example file with defaults that match what theforeman-foreman_proxy_content deploys.
Updated to the public URLs by default to match how we deploy this. |
This matches the pulp3.yml.example file with defaults that match the theforeman-pulpcore defaults.