diff options
| author | Valerii Ponomarov <vponomar@redhat.com> | 2019-12-16 22:35:12 +0530 |
|---|---|---|
| committer | Bala Konda Reddy M <bmekala@redhat.com> | 2019-12-17 09:58:27 +0000 |
| commit | c0fa7cfb52803a75af7faf374c155517f9d5e644 (patch) | |
| tree | 7570ca8408a9c5aed70b5d79f817a5a9d70699b4 /glustolibs-gluster | |
| parent | 099438eec88274839411f51cd214f6ff7fdac87c (diff) | |
Fix volume cleanup in 'gluster_base_class' module
One of the recent changes [1] broke volume cleanup logic
which is located in base class.
Fix it by proper handling of the cleanup function results
where '0' return code is 'success' and not 'failure'
as it was considered in that change.
[1] 6cd137615aec29dade5b41975fcbdae06852cf53
Change-Id: I674493369202ceabc6983fae0b3834e3b0708bf1
Signed-off-by: Valerii Ponomarov <kiparis.kh@gmail.com>
Diffstat (limited to 'glustolibs-gluster')
| -rw-r--r-- | glustolibs-gluster/glustolibs/gluster/gluster_base_class.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/glustolibs-gluster/glustolibs/gluster/gluster_base_class.py b/glustolibs-gluster/glustolibs/gluster/gluster_base_class.py index 15d96748e..858741abd 100644 --- a/glustolibs-gluster/glustolibs/gluster/gluster_base_class.py +++ b/glustolibs-gluster/glustolibs/gluster/gluster_base_class.py @@ -414,7 +414,7 @@ class GlusterBaseClass(TestCase): cmd = ('rm -rf %s' % mount_obj.mountpoint) ret, _, err = g.run( mount_obj.client_system, cmd, user=mount_obj.user) - if not ret: + if ret: g.log.error( "failed to delete the directory path used for " "mounting %s: %s" % (mount_obj.mountpoint, err)) |