Skip to content

rename the pulp-only proxy#96

Closed
evgeni wants to merge 1 commit intomasterfrom
pulp-proxy
Closed

rename the pulp-only proxy#96
evgeni wants to merge 1 commit intomasterfrom
pulp-proxy

Conversation

@evgeni
Copy link
Copy Markdown
Member

@evgeni evgeni commented Mar 3, 2025

No description provided.

theforeman.foreman.smart_proxy:
name: "{{ ansible_fqdn }}"
name: "{{ ansible_fqdn }}-pulp"
url: "https://{{ ansible_fqdn }}:9090"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Pulp doesn't run on port 9090 so shouldn't you rename the other entry?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

There is no other entry ;)
The proxy - as deployed today by this repo - is pulp only. There are no other features enabled:
https://github.com/theforeman/foreman-quadlet/blob/726e4a59677219a0a4ec4a116e3de9ed9022b77c/container-images/foreman-proxy/Containerfile#L5

#81 is then flipping the implementation to be pure pulp-based, but that doesn't affect the featureset

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is there a point to this change before #81 then?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

mostly "splitting things up in smaller chunks", but I can add it to 81 if you prefer

@evgeni evgeni closed this Mar 13, 2025
@evgeni evgeni deleted the pulp-proxy branch March 13, 2025 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants