Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

No longer configure pulp2 global proxy #297

Merged
merged 1 commit into from
Aug 13, 2019

Conversation

jlsherrill
Copy link
Contributor

since we're now configuring the repositories directly

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is also the CDN config in katello.yml but that's still relevant?

@jlsherrill
Copy link
Contributor Author

It is, thats currently how we pass the installer values for --katello-proxy to the application.

The idea is that eventually we remove those options from the installer completely and just configure it within the application, but the thought was to let all that bake for a couple releases until we remove the installer options (as lots of docs/scripting depend on the installer)

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good. We also have pulp_proxy_* parameters in puppet-foreman_proxy_content. It'd be great if you could also take a look at those to see whether they are now redundant.

@ekohl ekohl merged commit 4e3df18 into theforeman:master Aug 13, 2019
@jlsherrill jlsherrill deleted the pulp_proxy branch August 13, 2019 15:01
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants