Bug 1506502
| Summary: | [TSB] Should not show openshift/templates on Catalog after TSB enabled | |||
|---|---|---|---|---|
| Product: | OpenShift Container Platform | Reporter: | Yadan Pei <yapei> | |
| Component: | Installer | Assignee: | Scott Dodson <sdodson> | |
| Status: | CLOSED ERRATA | QA Contact: | Yadan Pei <yapei> | |
| Severity: | medium | Docs Contact: | ||
| Priority: | medium | |||
| Version: | 3.7.0 | CC: | aos-bugs, ewolinet, jmatthew, jokerman, mmccomas, sdodson, spadgett, wzheng, xtian, yapei | |
| Target Milestone: | --- | Keywords: | Reopened | |
| Target Release: | 3.7.0 | |||
| Hardware: | Unspecified | |||
| OS: | Unspecified | |||
| Whiteboard: | ||||
| Fixed In Version: | Doc Type: | No Doc Update | ||
| Doc Text: |
undefined
|
Story Points: | --- | |
| Clone Of: | ||||
| : | 1534316 (view as bug list) | Environment: | ||
| Last Closed: | 2017-11-28 22:19:38 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: | ||||
| Bug Depends On: | ||||
| Bug Blocks: | 1534316 | |||
|
Description
Yadan Pei
2017-10-26 08:55:23 UTC
Jessica, I thought this might be a web-console issue, sound related to issue you raised in this mornings meeting. I thought the ansible install was supposed to be setting up the console extension. That said we need to rename the config option the console uses either way based on the discussion this morning. The flag is getting fixed in ansible in https://github.com/openshift/openshift-ansible/pull/5912 The build this bug was opened against was from before we changed the flag name, we are concerned the installer is not properly laying down / configuring the extension for the console. Need to investigate this more. The ansible fix is merged and I can't see anything else wrong with the tasks for setting up the TSB. Can you re-test when there is a build that includes both of these PRs: https://github.com/openshift/origin-web-console/pull/2387 https://github.com/openshift/openshift-ansible/pull/5912 If this is still happening we need to know: 1) The content of this file on disk on the masters /etc/origin/master/openshift-ansible-catalog-console.js 2) The master-config.yaml setting for assetConfig -> extensionScripts 3) The content of the compiled extensions.js file loaded into the browser by the web console After collecting the above, if openshift-ansible-catalog-console.js contains content that looks like: window.OPENSHIFT_CONSTANTS.ENABLE_TECH_PREVIEW_FEATURE.template_service_broker = true; AND assetConfig -> extensionScripts is set to pull that file AND extensions.js does not have the same content, then: 1) Clear the browser cache and see if the content of extensions.js changes 2) If not, restart the master and see if the content of extensions.js changes Re-tested based on comment 6, openshift-ansible-catalog-console.js and compiled extensions.js has the same content which is an old one. window.OPENSHIFT_CONSTANTS.ENABLE_TECH_PREVIEW_FEATURE.template_service_broker = true; Although pr 5912 is merged to openshift-ansible, not built into rpm package yet, that's probably why the issue happens. Will test when we get new ansible packages. Ok thats good news, since the JS file is getting setup and served correctly then getting the ansible update for the file contents should fix this. Checked on v3.7.0-0.190.0 with registry.reg-aws.openshift.com:443/openshift3/ose-service-catalog v3.7.0-0.190.0 2a40aff211de 17 hours ago 266 MB Templates are not shown this time Move to VERIFIED The issue reproduces on v3.7.5 registry.reg-aws.openshift.com:443/openshift3/ose-service-catalog v3.7 ca30f0c47ab3 22 hours ago 266 MB 1. content of openshift-ansible-catalog-console.js [root@host-172-16-120-76 ~]# cat /etc/origin/master/openshift-ansible-catalog-console.js window.OPENSHIFT_CONSTANTS.TEMPLATE_SERVICE_BROKER_ENABLED = true; 2. compiled extensions.js // empty file so that the master-config can still point to a file that exists // this file will be replaced by the template service broker role if enabled ; The v3.7.5 env is launched with openshift-ansible at master branch If you look at master-config.yaml, is openshift-ansible-catalog-console.js listed in the asset config extension scripts? I have a suspicion that the updated extension file is copied after the master has started, which means it isn't being picked up. Yeah, extension scripts is set correctly extensionScripts: - /etc/origin/master/openshift-ansible-catalog-console.js Extensions are only read on server startup. Right now the installer is copying an empty file and updating it later if TSB is enabled. The updated file appears to be copied after the master has started, however, which is why it's not taking effect. A workaround would be to restart master again. @Sam, Is the fix then for the template_service_broker role to restart the master after we update the extensions js file? Eric -- Yes, that should fix it. https://github.com/openshift/openshift-ansible/pull/6101 in progress to fix that Fixed on latest ansible openshift-ansible-3.7.8-1 Move to VERIFIED 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 |