Bug 1666714

Summary:	Re-editing the tabs after failure doesn't reflect in gdeploy conf file.
Product:	[Red Hat Storage] Red Hat Gluster Storage	Reporter:	Mugdha Soni <musoni>
Component:	rhhi	Assignee:	Sahina Bose <sabose>
Status:	CLOSED ERRATA	QA Contact:	Mugdha Soni <musoni>
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	rhhiv-1.5	CC:	godas, rhs-bugs, sankarshan, sasundar
Target Milestone:	---	Keywords:	Regression
Target Release:	RHHI-V 1.6
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:	Previously, when the generated configuration file was edited by the user editing values in the tabs of the UI during the 'Review' step of deployment, the original contents of the generated configuration file were not updated. This has now been corrected and the file is updated with new values as expected.	Story Points:	---
Clone Of:
Clones:	1666847 (view as bug list)		Environment:
Last Closed:	2019-05-09 06:09:06 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:	1666847
Bug Blocks:	1672930

Description Mugdha Soni 2019-01-16 13:04:07 UTC

Description of problem:
-----------------------------
In gluster deployment if deployment fails due to wrong configuration of the parameters and user tries to re-edit the configuration in any tab i.e hosts,FQDN,volumes and bricks ,it is not reflected in the generated gdeploy conf file which leads to failed deployment .Even if reload button is clicked no changes are reflected.


Version-Release number of selected component:
----------------------------------------------
rhvh-4.2.8.0-0.20190108
glusterfs-server-3.12.2-32.el7rhgs

How reproducible:
-----------------
Everytime

Steps to Reproduce:
--------------------
1.Login to cockpit UI and start gluster deployment.
2.Configure all tabs and deploy.
3.If any error encountered try to re-edit the tabs and check the gdeploy conf file.

Actual results:
-------------------
Changes are not reflected in gdeploy conf file.


Expected results:
----------------
Any changes made should be reflected in new gdeploy conf file.

Comment 2 SATHEESARAN 2019-01-16 18:03:11 UTC

This defect is marked with 'Regresion' as the same functionality was working well with RHHI-V 1.5 (RHV 4.2.7) and is now broken with RHHI-V 1.5.z (RHV 4.2.8)

With this defect in place, if there are any errors, or wrong values provided, then the user had to close the cockpit deployment window and start the deployment from scratch

Comment 5 SATHEESARAN 2019-02-26 08:45:21 UTC

The dependent bug is already ON_QA and moving this bug too to ON_QA

Comment 6 SATHEESARAN 2019-02-26 08:45:38 UTC

Tested with cockpit-ovirt-dashboard-0.12.2 

1. Re-editing the values gets reflected in the gluster-ansible playbook

Comment 8 errata-xmlrpc 2019-05-09 06:09:06 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, 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/RHEA-2019:1121