Bug 1547814

Summary: gdeploy should stop execution once the VDO volume creation fails
Product: [Red Hat Storage] Red Hat Gluster Storage Reporter: SATHEESARAN <sasundar>
Component: gdeployAssignee: Ramakrishna Reddy Yekulla <rreddy>
Status: CLOSED ERRATA QA Contact: SATHEESARAN <sasundar>
Severity: high Docs Contact:
Priority: unspecified    
Version: rhgs-3.3CC: amukherj, dkota, rcyriac, rhinduja, rhs-bugs, rreddy, sabose, smohan, storage-qa-internal
Target Milestone: ---Keywords: ZStream
Target Release: RHGS 3.3.1 Async   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: gdeploy-2.0.2-26 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1547813 Environment:
Last Closed: 2018-06-21 03:33:15 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:    
Bug Blocks: 1547813, 1581561    

Description SATHEESARAN 2018-02-22 02:36:40 UTC
+++ This bug was initially created as a clone of Bug #1547813 +++

Description of problem:
-----------------------
When the creation of VDO volume fails, gdeploy should halt the execution.
The expectation here is the availability of the param 'ignore_vdo_failure' similar to the ones under 'vg' & 'lv'

Version-Release number of selected component (if applicable):
-------------------------------------------------------------
gdeploy-2.0.2-22.el7rhgs

How reproducible:
-----------------
Always

Steps to Reproduce:
-------------------
1. Provide a non-available device for VDO volume creation in gdeploy conf file
2. Execute the conf file

Actual results:
---------------
gdeploy ignore errors when VDO volume creation fails and proceeds to further tasks

Expected results:
----------------
gdeploy should provide an option 'ignore_vdo_errors' there gdeploy could halt the execution on encountering errors

Additional info:
----------------
For example this is the gdeploy conf file used:
[hosts]
host1

[vdo]
action=create
devices=sdz
names=vdovol1
logicalsize=100G
ignore_vdo_errors=no

And here is the console output with the above mentioned gedploy conf file:

# gdeploy -c vdocreate.conf 

PLAY [gluster_servers] ********************************************************************************************************************************************************************************************

TASK [Create VDO with specified size] *****************************************************************************************************************************************************************************
failed: [10.70.36.244] (item={u'disk': u'/dev/sdz', u'logicalsize': u'100G', u'name': u'vdovol1'}) => {"changed": false, "err": "vdo: ERROR -   Device /dev/sdz not found.\n", "item": {"disk": "/dev/sdz", "logicalsize": "100G", "name": "vdovol1"}, "msg": "Creating VDO vdovol1 failed.", "rc": 2}
	to retry, use: --limit @/tmp/tmpaanGEu/vdo-create.retry

PLAY RECAP ********************************************************************************************************************************************************************************************************
10.70.36.244               : ok=0    changed=0    unreachable=0    failed=1   

Ignoring errors...

Comment 3 Devyani Kota 2018-04-20 07:52:00 UTC
This commit[1] fixes this issue.

[1]https://github.com/gluster/gdeploy/pull/502/commits/99e6e0bb721d48a835ca4daec743dd202958c3a3

Thanks.

Comment 4 SATHEESARAN 2018-05-02 10:39:05 UTC
Tested with gdeploy-2.0.2-26.el7rhgs

When 'ignore_vdo_errors=no' is available in the VDO section, VDO halts execution on encoutering errors.

Comment 6 errata-xmlrpc 2018-06-21 03:33:15 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-2018:1958