From c0fa7cfb52803a75af7faf374c155517f9d5e644 Mon Sep 17 00:00:00 2001 From: Valerii Ponomarov Date: Mon, 16 Dec 2019 22:35:12 +0530 Subject: 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 --- glustolibs-gluster/glustolibs/gluster/gluster_base_class.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'glustolibs-gluster/glustolibs/gluster') 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)) -- cgit