-
Notifications
You must be signed in to change notification settings - Fork 65
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
Added passthrough pulp mode #372
Conversation
efe5fbe
to
1e1ee92
Compare
This is an interesting idea. I have a few questions to help me think on it.
|
Thanks Eric for you question and your interest. |
I think I am not grasping the concept here. Let me recap my thoughts and you can steer me back on course. I interpreted this as a lightweight way to present content on a proxy so that hosts isolated could get content from the main server without having to have a full blown Pulp configured and syncing content. |
This is correct.
During installation, all pulp node services are still configured. If you want, you can still sync the content to the pulp node (capsule) - maybe only parts. In the end, the client decides if the passthrough mode is used (= proxy configuration on the hosts is configured) or the pulp node.
Of course, it doesn't make sense to sync all the content to the pulp node and then use the passthrough mode. The intention is to use the passthrough mode and don't sync the content so that the content (rpms, debs) doesn't need to be duplicated.
…---- Eric Helms schrieb ----
I think I am not grasping the concept here. Let me recap my thoughts and you can steer me back on course. I interpreted this as a lightweight way to present content on a proxy so that hosts isolated could get content from the main server without having to have a full blown Pulp configured and syncing content.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub<#372?email_source=notifications&email_token=AGCOEFOHMRT3QE4BN5CDQMTPZBL2PA5CNFSM4HTWEVCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXBM2KA#issuecomment-499305768>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AGCOEFLEV7EHV2O37K43QXTPZBL2PANCNFSM4HTWEVCA>.
|
1e1ee92
to
6cdfb71
Compare
ping @ehelms |
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.
What's the general design behind this? Looking at the configuration, this would still deploy a full Pulp instance, correct? Is that really needed?
@@ -517,6 +533,11 @@ | |||
Optional[String] $ldap_bind_dn = $pulp::params::ldap_bind_dn, | |||
Optional[String] $ldap_bind_password = $pulp::params::ldap_bind_password, | |||
String $ldap_remote_user_attribute = $pulp::params::ldap_remote_user_attribute, | |||
Boolean $enable_passthrough_pulp = $pulp::params::enable_passthrough_pulp, | |||
Stdlib::Port $passthrough_pulp_http_port = $pulp::params::passthrough_pulp_http_port, | |||
Optional[String] $passthrough_pulp_allowed_net = $pulp::params::passthrough_pulp_allowed_net, |
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 can enforce the data type with https://github.com/puppetlabs/puppetlabs-stdlib/blob/master/REFERENCE.md#stdlibipaddressv4cidr. You may want to consider https://github.com/puppetlabs/puppetlabs-stdlib/blob/master/REFERENCE.md#stdlibipaddressv6cidr as well.
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 See #372 (comment) regarding your answer about "full pulp instance deployment".
@@ -47,7 +71,7 @@ | |||
order => '60', | |||
config_entries => { | |||
'cache_peer' => "${streamer_host} parent ${streamer_port} 0 no-digest no-query originserver name=PulpStreamer", | |||
'cache_peer_access' => 'PulpStreamer allow all', | |||
'cache_peer_access' => 'PulpStreamer allow localhost', |
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 looks like a general thing that can be applied to all installations, correct? Only Apache/Pulp connect to squid.
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.
yes. I think so.
can you please have a look at it again @ekohl ? |
@sbernhard We have had a hard time wrapping our brains around this change and how it plays into the architecture. With Pulp 3 integration appearing in Katello releases, and these components no longer being present. Is this a change you still want to persue? |
@ehelms The passthrough mode is feature often used by our customers. They like it a lot because they use a foreman proxy to make sure provisioning works in a specific network but don't waste storage as they just forward the requests to the rpm/deb packages. For pulp3, we want to have similar / the same approach. Therefore, we will work on theforeman/puppet-foreman_proxy_content#205, too. |
This module is just for Pulp 2. No new features will be accepted in this module. It's only here for Katello 3.18 until that's EOL. https://github.com/theforeman/puppet-pulpcore is used for Pulp 3 and is completely different. For example, squid is not present. |
This change implements a passthrough mode on a pulpnode (=capsule) to pulp master using the already existing squid proxy.
The advantage is, that many customers don't want to duplicate the pulp content but still have separated subnets. Therefore, we configured a squid proxy on the capsule to route all requests for pulp content (rpm, apt) to pulp master (=katello)
For using this at katello:
proxy=http://capsule.dev:3129