Bug 1836158

Summary:	the url setting for ui is nil in central region if set to https in a sub region
Product:	Red Hat CloudForms Management Engine	Reporter:	Felix Dewaleyne <fdewaley>
Component:	Appliance	Assignee:	Yuri Rudman <yrudman>
Status:	CLOSED ERRATA	QA Contact:	Tasos Papaioannou <tpapaioa>
Severity:	high	Docs Contact:	Red Hat CloudForms Documentation <cloudforms-docs>
Priority:	high
Version:	5.11.5	CC:	dmetzger, fdewaley, obarenbo, sigbjorn.lie, simaishi, tpapaioa, yrudman
Target Milestone:	GA	Keywords:	ZStream
Target Release:	5.11.7	Flags:	simaishi: cfme-5.11.z+
Hardware:	All
OS:	All
Whiteboard:
Fixed In Version:	5.11.7.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-08-06 14:32:54 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	CFME Core	Target Upstream Version:
Embargoed:

Description Felix Dewaleyne 2020-05-15 09:45:16 UTC

Description of problem:
If in a region the advanced setting is set with https instead of http it is nil on the central region but not in the region itself


Version-Release number of selected component (if applicable):
5.11.5

How reproducible:
all the time

Steps to Reproduce:
1. set up at least two region, regionone and central, with regionone syncing into central
2. in regionone set up a webui zone with a couple ui providers hit behind a load balancer
3. set up in regionone the advanced url setting under ui (see additional info [1])
4. connnect to the rails console in region one and run 
srv = MiqServer.in_region(1).recently_active.find_by(:has_active_userinterface => true)
srv.settings_for_resource.ui.url
5. run the same in central

Actual results:
in regionone it will display 
=> "https://regionone.example.com"
in central it will display
=> nil

Expected results:
in both it displays
=> "https://regionone.example.com"

Additional info:
[1] - the setting is 
:ui:
  :custom_menu: 
  :mark_translated_strings: false
  :display_ops_database: false
  :url: https://regionone.example.com

note : if the url is set to http://regionone.example.com both regions will display 
=> http://regionone.example.com
there is no indication that the url should be http instead of https and this affects the `connect to vm through region` button (in the details of a vm in the vm explorer, compute > infra > vms > click a vm), which would be used in the central region to connect to a vm's console.

Comment 2 Felix Dewaleyne 2020-05-15 09:57:16 UTC

not sure if this is appliance or ui ops.

Comment 19 CFME Bot 2020-06-20 16:50:18 UTC

https://github.com/ManageIQ/manageiq/pull/20299

Comment 20 CFME Bot 2020-06-29 15:45:22 UTC

New commit detected on ManageIQ/manageiq/master:

https://github.com/ManageIQ/manageiq/commit/86f34c36d1a92bebbf92b15cfb06ef6dcf6463c5
commit 86f34c36d1a92bebbf92b15cfb06ef6dcf6463c5
Author:     Yuri Rudman <yrudman>
AuthorDate: Sat Jun 20 16:37:49 2020 +0000
Commit:     Yuri Rudman <yrudman>
CommitDate: Thu Jun 25 18:47:10 2020 +0000

    MiqServer#miq_region should return actual region this server belongs to
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1836158

 app/models/miq_server.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comment 21 CFME Bot 2020-07-01 17:10:19 UTC

New commit detected on ManageIQ/manageiq/ivanchuk:

https://github.com/ManageIQ/manageiq/commit/a3251c7b642f777a716f179bd69f549f8946c34c
commit a3251c7b642f777a716f179bd69f549f8946c34c
Author:     Jason Frey <fryguy9>
AuthorDate: Mon Jun 29 15:42:52 2020 +0000
Commit:     Satoe Imaishi <simaishi>
CommitDate: Wed Jul  1 17:06:59 2020 +0000

    Merge pull request #20299 from yrudman/global-region-settings-fix

    Fixed Settings hierarchy on Global region

    (cherry picked from commit 68b086e235f11e3edd5a9826eb0a8f680a8da0b1)

    https://bugzilla.redhat.com/show_bug.cgi?id=1836158

 app/models/miq_server.rb | 2 +-
 spec/lib/vmdb/settings_spec.rb | 75 +
 spec/models/miq_server_spec.rb | 17 +
 3 files changed, 93 insertions(+), 1 deletion(-)

Comment 22 Tasos Papaioannou 2020-07-16 15:27:10 UTC

Verified on 5.11.7.0.

Comment 25 errata-xmlrpc 2020-08-06 14:32:54 UTC

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 (Critical: CloudForms 5.0.7 bug fix and enhancement update), 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-2020:3358