Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1584175

Summary: The gdeploy conf file is overwritten during the expand cluster operation.
Product: [oVirt] cockpit-ovirt Reporter: Mugdha Soni <musoni>
Component: GdeployAssignee: Gobinda Das <godas>
Status: CLOSED NOTABUG QA Contact: SATHEESARAN <sasundar>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 0.11.20CC: bugs, musoni, rhs-bugs, sabose, sankarshan, sasundar
Target Milestone: ---Flags: musoni: devel_ack?
musoni: testing_ack?
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1584174 Environment:
Last Closed: 2018-06-02 02:38:57 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Gluster RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1584174    
Attachments:
Description Flags
screenshot of the over written file name none

Description Mugdha Soni 2018-05-30 12:32:39 UTC 
Created attachment 1445830 [details]
screenshot of the over written file name

+++ This bug was initially created as a clone of Bug #1584174 +++

Description of problem:
---------------------------------------
The gdeploy conf file is over written in expand cluster and create volume operations.The conf file should not be overwritten as the previous data will be lost whicj might be useful.


Version-Release number of selected component (if applicable):
---------------------------------------------------------------
cockpit-ovirt-dashboard-0.11.999-0.0.master.el7.noarch

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

Steps to Reproduce:
----------------------------------------
1.Successfully deploy gluster and hosted engine.
2.Reach out to the gluster management wizard under hosted engine and click on ' expand cluster ' button .
3.Fill in all the parametrs required in  tab host , packages,volume and bricks.
4.The issue can be found in the gdeploy conf file under the review tab.

Actual results:
-----------------------------------------
The gdeploy conf file is overwritten.

Expected results:
-----------------------------------------
The gdeploy conf file should not use the same name everytime and should not be overwritten.

Additional info:
Comment 1 Gobinda Das 2018-05-31 09:12:24 UTC 
Mugdha,
 There should be the backup file named with gdeploy-<imestamp>.conf
Before creating new conf file cockpit will take backup of exiting one for further reference.
Can you please check backup file present or not?
Comment 2 SATHEESARAN 2018-06-02 02:38:57 UTC 
(In reply to Gobinda Das from comment #1)
> Mugdha,
>  There should be the backup file named with gdeploy-<imestamp>.conf
> Before creating new conf file cockpit will take backup of exiting one for
> further reference.
> Can you please check backup file present or not?

Yes, its available. This conf file is ok for the future reference.
Based on that closing this bug.