Bug 1420543
Summary: | The --ports flag does not modify dc environment variables | ||
---|---|---|---|
Product: | OpenShift Container Platform | Reporter: | Steven Walter <stwalter> |
Component: | Networking | Assignee: | Phil Cameron <pcameron> |
Networking sub component: | router | QA Contact: | zhaozhanqi <zzhao> |
Status: | CLOSED ERRATA | Docs Contact: | |
Severity: | low | ||
Priority: | low | CC: | aos-bugs, atragler, bbennett, dlbewley, pcameron, stwalter |
Version: | 3.3.1 | ||
Target Milestone: | --- | ||
Target Release: | 3.7.0 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2017-11-28 21:52:23 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Steven Walter
2017-02-08 22:56:26 UTC
I think this is "functions as designed ... but confusing". The exposed ports are set up independently of the config for port binding. In fact the exposed ports really aren't used for much since typically you are using host networking so can't remap the ports anyway. I'm dropping te priority to "low" so we can clean up the command line help to make it clear that you need to set this and the environment variables. As you said, these are host ports, so I was expecting the `--ports` arg to make it all the way into the haproxy bind config by whatever means. The goal was to run a second proxy (router) on a unique port while continuing to leverage the same ipfailover VIPs and DNS records. We have the following documentation on this topic. Please review it. I can make changes to clear up inaccurate or confusing discussions. Documentation: https://docs.openshift.com/container-platform/3.5/ Architecture: Environment variables: ROUTER_SERVICE_HTTPS_PORT ROUTER_SERVICE_HTTP_PORT architecture/core_concepts/routes.html#routers architecture/core_concepts/routes.html#env-variables How to set it up: install_config/router/default_haproxy_router.html#customizing-the-router-service-ports -------- # oadm router --help --ports='80:80,443:443': A comma delimited list of ports or port pairs to expose on the router pod. The default is set for HAProxy. Port pairs are applied to the service and to host ports (if specified). The above oadm help is confusing (if not wrong). I will change it. Ok, the docs you link make sense. I think as long as we clarify the oadm help info, that would be sufficient from my pov. origin PR 13816 https://github.com/openshift/origin/pull/13816 Commit pushed to master at https://github.com/openshift/origin https://github.com/openshift/origin/commit/0d4d887a808e95bdd2f0d4a71e747ceeb6d48557 The --ports flag does not modify dc env variables --ports is not intended to modify the env variables. That can be done using oc env or by editing the router's dc. Rather it sets the ports that are exposed in the router's service. The help message is confusing and misleading. This change rewords the help text. bug 1420543 https://bugzilla.redhat.com/show_bug.cgi?id=1420543 verified this bug on v3.7.0-0.143.1 oadm router --help --ports='80:80,443:443': A comma delimited list of ports or port pairs that set the port in the router pod containerPort and hostPort. It also sets service port and targetPort to expose on the router pod. This does not modify the env variables. That can be done using oc env or by editing the router's dc. This is used when host-network=false. Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2017:3188 |