Fedora Account System
Red Hat Associate
Red Hat Customer
Description of problem: When session timeout or authentication mode are changed in Configuration, server name is changed to empty string. This affects many automation tests. Version-Release number of selected component (if applicable): 5.10.0.32, 5.9.8.0 How reproducible: 100% Steps to Reproduce: 1. deploy new appliance 2. Go to Configuration->Authentication tab 3. Change Session timeout value or Mode to External 4. Click Save 5. look at appearing alert or reopen Configuration and check server name in tree Actual results: Server name gets blank Expected results: Server name shouldn't become blank Additional info:
name is lost in rest as well after auth change.
Created a PR on UI side to not allow server name to be updated when changes are saved for Authentication tab https://github.com/ManageIQ/manageiq-ui-classic/pull/5194 ~Harpreet
New commits detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/9585e66ed476e329202a29d2d0a7bd60c2ebc6bb commit 9585e66ed476e329202a29d2d0a7bd60c2ebc6bb Author: Harpreet Kataria <hkataria> AuthorDate: Thu Jan 24 11:37:49 2019 -0500 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Jan 24 11:37:49 2019 -0500 Only update/save name of server record when on Settings/Server tab. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1668849 app/controllers/ops_controller/settings/common.rb | 2 +- spec/controllers/ops_controller/settings/common_spec.rb | 12 +- 2 files changed, 12 insertions(+), 2 deletions(-) https://github.com/ManageIQ/manageiq-ui-classic/commit/5ced8d0d782e7db453eb56681a46ce342b0575e5 commit 5ced8d0d782e7db453eb56681a46ce342b0575e5 Author: Harpreet Kataria <hkataria> AuthorDate: Thu Jan 24 13:47:37 2019 -0500 Commit: Harpreet Kataria <hkataria> CommitDate: Thu Jan 24 13:47:37 2019 -0500 Addressed code climate warning. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1668849 app/controllers/ops_controller/settings/common.rb | 3 +- 1 file changed, 2 insertions(+), 1 deletion(-)
Verified in 5.11.0.3