Skip to content
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 #31662: Set enable_http to ensure pub dir is deployed to Pulp v… #315

Merged
merged 1 commit into from
Jan 20, 2021

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Jan 19, 2021

…host

This should be cherry picked to 3.18. This affects only content proxies deployed in Katello 3.18 (or current nightly, all that changes once we go Pulp 3 only) that have enable_puppet set to false. The reason is this chunk of code: https://github.com/theforeman/puppet-pulp/blob/master/manifests/apache.pp#L23

@evgeni
Copy link
Member

evgeni commented Jan 20, 2021

By "pub dir" you mean /var/www/pub/ which hosts all those pulp bits, NOT forman.example.com/pub which only hosts ca cert etc, right? If so, then ACK

@ehelms
Copy link
Member Author

ehelms commented Jan 20, 2021

Nope, I mean foreman.example.com/pub where bootstrap RPM and such live. The user issue thread if it helps to understand -- https://community.theforeman.org/t/content-proxy-not-listing-to-port-80-anymore/21996

@evgeni
Copy link
Member

evgeni commented Jan 20, 2021

Ah, the issue IS actually about the cert...

@ehelms ehelms merged commit 2dc0d72 into theforeman:master Jan 20, 2021
@ekohl ekohl added the Bug label Jan 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants