Bug 1654287

Summary: [REGRESSION] Authentication settings cannot be changed
Product: Red Hat CloudForms Management Engine Reporter: Ievgen Zapolskyi <izapolsk>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED CURRENTRELEASE QA Contact: Ievgen Zapolskyi <izapolsk>
Severity: high Docs Contact: Red Hat CloudForms Documentation <cloudforms-docs>
Priority: high    
Version: 5.9.6CC: bmidwood, dmetzger, hkataria, lavenel, mpovolny, obarenbo, simaishi
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.9.6   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.9.6.5 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2019-01-07 16:24:36 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:
Attachments:
Description Flags
no appliance name error
none
appliance logs none

Description Ievgen Zapolskyi 2018-11-28 12:04:16 UTC 
Created attachment 1509475 [details]
no appliance name error

Description of problem:
When I try to change Authentication settings and save my changes, I get "Appliance name must be entered" error.
I suppose it is caused by changes introduced in  https://bugzilla.redhat.com/show_bug.cgi?id=1639364

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


How reproducible:
100%

Steps to Reproduce:
1. Open Configuration
2. Click on server's -> Authentication tab
3. Change Mode to anything. F.e. External (httpd)
4. Click Save

Actual results:
"Appliance name must be entered" error


Expected results:
No such error

Additional info:
Comment 2 Ievgen Zapolskyi 2018-11-28 12:06:10 UTC 
Created attachment 1509476 [details]
appliance logs
Comment 3 CFME Bot 2018-11-29 22:36:47 UTC 
https://github.com/ManageIQ/manageiq-ui-classic/pull/5015
Comment 4 CFME Bot 2018-11-29 23:26:57 UTC 
New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili:

https://github.com/ManageIQ/manageiq-ui-classic/commit/10ce1a166c16593cc9fe4a68456f567956a94351
commit 10ce1a166c16593cc9fe4a68456f567956a94351
Author:     Harpreet Kataria <hkataria>
AuthorDate: Thu Nov 29 17:26:03 2018 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Thu Nov 29 17:26:03 2018 -0500

    Check for Appliance name only when validating settings tab.

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

 app/controllers/ops_controller/settings/common.rb | 1 +
 1 file changed, 1 insertion(+)
Comment 5 Ievgen Zapolskyi 2018-12-05 08:56:08 UTC 
Fixed in 5.9.6.5