Bug 1381330

Summary: Unrelated error pops up when changing the settings in Configuration->Workers
Product: Red Hat CloudForms Management Engine Reporter: Aparna Karve <akarve>
Component: UI - OPSAssignee: Harpreet Kataria <hkataria>
Status: CLOSED CURRENTRELEASE QA Contact: Niyaz Akhtar Ansari <nansari>
Severity: unspecified Docs Contact:
Priority: high    
Version: 5.7.0CC: cpelland, hkataria, jhardy, mpovolny, nansari, obarenbo
Target Milestone: GAKeywords: TestOnly
Target Release: 5.8.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 5.8.0.0 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1395853 (view as bug list) Environment:
Last Closed: 2017-06-12 17:29:03 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:
Bug Depends On:    
Bug Blocks: 1395853    
Attachments:
Description Flags
Unrelated Password error none

Description Aparna Karve 2016-10-03 17:50:16 UTC
Created attachment 1206949 [details]
Unrelated Password error

Description of problem:Unrelated error pops up when changing the settings in Configuration->Workers


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


How reproducible: Always


Steps to Reproduce:
1. Navigate to Configuration, go to the Workers tab
2. Change a value in any of the Worker dropdowns and hit Save
3. "Password/Verify Password do not match" error pops up

Actual results: Unrelated "Password/Verify Password do not match" error pops up

Expected results:Values should be saved without any issues



Additional info:

Comment 2 Harpreet Kataria 2016-10-03 21:18:17 UTC
This issue is caused by removal of rubyrep replication workers settings in https://github.com/ManageIQ/manageiq/pull/9696, some of the leftover code is causing the flash message to be displayed on screen.

Comment 3 Harpreet Kataria 2016-10-03 21:50:57 UTC
https://github.com/ManageIQ/manageiq/pull/11668

Comment 4 CFME Bot 2016-10-03 22:31:47 UTC
New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/f64de7d1432ec0b6a776f1cc43292fd8e7a9a1e3

commit f64de7d1432ec0b6a776f1cc43292fd8e7a9a1e3
Author:     Harpreet Kataria <hkataria>
AuthorDate: Mon Oct 3 17:47:33 2016 -0400
Commit:     Harpreet Kataria <hkataria>
CommitDate: Mon Oct 3 17:47:33 2016 -0400

    Removed leftover rubyrep passowrd checking code.
    
    Removed leftover checks that were causing a flash error message to display on worlkers settings screen, introduced in https://github.com/ManageIQ/manageiq/pull/9696
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1381330

 app/controllers/ops_controller/settings/common.rb | 9 ---------
 1 file changed, 9 deletions(-)

Comment 6 Niyaz Akhtar Ansari 2017-03-08 11:52:53 UTC
Verified in version 5.8.0.4