Bug 1159182
Summary: | some useless conf files are created by openshift-routing-daemon | ||
---|---|---|---|
Product: | OpenShift Container Platform | Reporter: | Johnny Liu <jialiu> |
Component: | Node | Assignee: | chris alfonso <calfonso> |
Status: | CLOSED ERRATA | QA Contact: | libra bugs <libra-bugs> |
Severity: | medium | Docs Contact: | |
Priority: | high | ||
Version: | 2.2.0 | CC: | adellape, bleanhar, calfonso, hbrock, jokerman, libra-onpremise-devel, mmccomas |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | rubygem-openshift-origin-routing-daemon-0.20.2.4-1.el6op | Doc Type: | Bug Fix |
Doc Text: |
In OpenShift Enterprise environments using the routing daemon with an nginx router, nginx configuration files were created for non-scalable applications, however they are not necessary as the routing daemon only routes for scalable applications. In addition, a hard-coded prefix of "ha-" was added to configuration files for scalable applications when they did not have an alias configured, which could cause confusion. This bug fix updates the routing daemon to no longer create nginx configuration files for non-scalable applications. In addition, the /etc/openshift/routing-daemon.conf file now includes the HA_DNS_PREFIX parameter, which allows administrators to define a prefix for scalable applications, if desired. This prefix must match the prefix that is set in the HA_DNS_PREFIX parameter in the /etc/openshift/broker.conf file, as well.
|
Story Points: | --- |
Clone Of: | Environment: | ||
Last Closed: | 2014-12-10 13:24:56 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
Johnny Liu
2014-10-31 07:11:23 UTC
Thanks for testing this Johnny. The naming of the conf files are a bit weird, but the way it works is we create a pool file when the application is created even if there are no members in the pool yet. The alias file does not only represent an alias created with RHC, it's also the alias created for any ha app. These config files will be removed when the application is removed. Also, thanks for catching the hard-coded ha- prefix! This is now fixed as part of https://github.com/openshift/enterprise-server/commit/f7043cae2dcac8631c8ee5319c82a433c9ac294b Retest this bug with rubygem-openshift-origin-routing-daemon-0.20.2.3-1.el6op.noarch. The issue found in scenarios 1 is fixed now, when creating non-scalable app, no nginx config files are generated. The hard-coded ha- prefix issue found in scenarios 2 is not fixed yet, the prefix should be allow user to set it in /etc/openshift/routing-daemon.conf There is no reason to have any nginx config files for non-scalable apps. I missed the ha-prefix hard coding that was mentioned. I'll fix that. PR open upstream. Will merge to enterprise-server when it's ready Verified this bug with rubygem-openshift-origin-routing-daemon-0.20.2.4-1.el6op.noarch, and PASS. The lines mentioned in comment 7 is removed now. The following config options are added in /etc/openshift/routing-daemon.conf: HA_DNS_PREFIX="ha-" After I change it to HA_DNS_PREFIX="hhh-" Then create a scalable app, the following config file is generated. # cat alias_pool_ose_scaruby18app_jialiu_80_hhh-scaruby18app-jialiu.example.com.conf server { listen 80; server_name hhh-scaruby18app-jialiu.example.com; location / { proxy_pass http://pool_ose_scaruby18app_jialiu_80; } } server { listen 443 ssl; #ssl_certificate_template #ssl_certificate_key_template server_name hhh-scaruby18app-jialiu.example.com; location / { proxy_pass http://pool_ose_scaruby18app_jialiu_80; } } The modified config option takes effect now. 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://rhn.redhat.com/errata/RHBA-2014-1979.html |