This repository has been archived by the owner on Jul 10, 2018. It is now read-only.
0020: Converge Katello and Foreman Ports #25
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sean797
reviewed
Oct 13, 2016
|
|
||
| ### Custom Port Ranges | ||
|
|
||
| One idea is to move all services to a set of well known ports and choose a port range that allows adding new services to a well defined location. For example: |
There was a problem hiding this comment.
Although this is a nice idea, I think this well create unnecessary pain for users upgrading -1
There was a problem hiding this comment.
For what users? We need to change at least one port anyway, so some upgrade path must be there anyway.
I like the idea of doing in for once but correctly. And also it looks like egrep ^90 /etc/services is empty.
ohadlevy
reviewed
Oct 30, 2016
|
|
||
| The Katello reverse proxy deployed on Capsules to achieve isolation feature currently runs on 8443. This means that subscription-manager is configured to point at and go through 8443 and is yet another port that must be managed. | ||
|
|
||
| * Move reverse proxy to 443 |
There was a problem hiding this comment.
easy to break if there is any default apache ssl setup? we should not use common known ports.
ohadlevy
reviewed
Oct 30, 2016
|
|
||
| ### Capsule vs Smart Proxy Ports | ||
|
|
||
| The Foreman Proxy uses 8443 by default on installation while the Capsule install for Katello uses 9090. This was originally done due to Candlepin's use of 8443; however, given Candlepin is a local installation the port it uses matters less. |
There was a problem hiding this comment.
so in all in one default install, subscription manager clients connect directly to 8443(candlepin) or rather it get proxied via katello itself?
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.