summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorkshithijiyer <kshithij.ki@gmail.com>2019-12-17 11:49:26 +0530
committerBala Konda Reddy M <bmekala@redhat.com>2020-01-16 06:23:19 +0000
commitd0aee85958ff13d15eaf75468fe9af0698b06f2e (patch)
treec45a072b5c261424fc268509b43f1c3ca23fa51e
parent86d16d2a98af540ba458721edc2ac8a530c01635 (diff)
[TC] Snapshot + glusterd: Status of snapshot after glusterd restart
Checking the status of snapshot informations before and after restarting glusterd. Test case: 1. Create volume 2. Create two snapshots with description 3. Check snapshot status informations with snapname, volume name and without snap name/volname. 4. Restart glusterd on all nodes 5. Follow step3 again and validate snapshot History of the patch: The testcase was failing in CentOS-ci due to bug [1], However this bug was defered for fix. Post updating the code with the workaround mentioned below, it was observed that glusterd restart was failing due to bug [2]. And now as the both the bugs are fixed this patch is safe to merge. Workaround: Now the only possible workaround for this is to modify the function get_snap_info_by_volname() to not use --xml option and directly run the command which will dump the output as string which can be directly used in the testcase. This modification in the library function will not impact any other testcase. Links: [1] https://bugzilla.redhat.com/show_bug.cgi?id=1590797 [2] https://bugzilla.redhat.com/show_bug.cgi?id=1782200 Change-Id: I26ac0aaa5f6c8849fd9de41f506d6d13fc55e166 Co-authored-by: srivickynesh <sselvan@redhat.com> Signed-off-by: srivickynesh <sselvan@redhat.com> Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
-rw-r--r--glustolibs-gluster/glustolibs/gluster/snap_ops.py63
-rw-r--r--tests/functional/snapshot/test_snap_status_glusterd_restart.py162
2 files changed, 173 insertions, 52 deletions
diff --git a/glustolibs-gluster/glustolibs/gluster/snap_ops.py b/glustolibs-gluster/glustolibs/gluster/snap_ops.py
index 01a7df3..1143663 100644
--- a/glustolibs-gluster/glustolibs/gluster/snap_ops.py
+++ b/glustolibs-gluster/glustolibs/gluster/snap_ops.py
@@ -304,7 +304,7 @@ def get_snap_status_by_snapname(mnode, snapname):
return None
-def get_snap_status_by_volname(mnode, volname):
+def snap_status_by_volname(mnode, volname):
"""Parse the output of 'gluster snapshot status' command
for the given volume.
@@ -313,59 +313,18 @@ def get_snap_status_by_volname(mnode, volname):
volname (str): snapshot name
Returns:
- NoneType: None if command execution fails, parse errors.
- list: list of dicts on success.
-
- Examples:
- >>> get_snap_status_by_volname('abc.lab.eng.xyz.com',
- 'testvol')
- [{'volCount': '1', 'volume': {'brick': [{'path': '10.70.47.11:
- testvol_brick0', 'pid': '26747', 'lvUsage': '3.52', 'volumeGroup':
- 'RHS_vg0', 'lvSize': '9.95g'}, {'path': '10.70.47.16:/testvol_brick1',
- 'pid': '25497', 'lvUsage': '3.52', 'volumeGroup': 'RHS_vg0',
- 'lvSize': '9.95g'}], 'brickCount': '2'}, 'name': 'snap2', 'uuid':
- '56a39a92-c339-47cc-a8b2-9e54bb2a6324'}, {'volCount': '1', 'volume':
- {'brick': [{'path': '10.70.47.11:testvol_next_brick0', 'pid': '26719',
- 'lvUsage': '4.93', 'volumeGroup': 'RHS_vg1', 'lvSize': '9.95g'}],
- 'brickCount': '1'}, 'name': 'next_snap1',
- 'uuid': 'dcf0cd31-c0db-47ad-92ec-f72af2d7b385'}]
- """
-
- cmd = "gluster snapshot status volume %s --xml" % volname
- ret, out, _ = g.run(mnode, cmd)
- if ret != 0:
- g.log.error("Failed to execute 'snapshot status' on node %s. "
- "Hence failed to get the snapshot status.", mnode)
- return None
-
- try:
- root = etree.XML(out)
- except etree.ParseError:
- g.log.error("Failed to parse the gluster snapshot "
- "status xml output.")
- return None
+ tuple: Tuple containing three elements (ret, out, err).
+ The first element 'ret' is of type 'int' and is the return value
+ of command execution.
- snap_status_list = []
- for snap in root.findall("snapStatus/snapshots/snapshot"):
- snap_status = {}
- for element in snap.getchildren():
- if element.tag == "volume":
- status = {}
- status["brick"] = []
- for elmt in element.getchildren():
- if elmt.tag == "brick":
- brick_info = {}
- for el in elmt.getchildren():
- brick_info[el.tag] = el.text
- status["brick"].append(brick_info)
- else:
- status[elmt.tag] = elmt.text
+ The second element 'out' is of type 'str' and is the stdout value
+ of the command execution.
- snap_status[element.tag] = status
- else:
- snap_status[element.tag] = element.text
- snap_status_list.append(snap_status)
- return snap_status_list
+ The third element 'err' is of type 'str' and is the stderr value
+ of the command execution.
+ """
+ cmd = "gluster snapshot status volume %s" % volname
+ return g.run(mnode, cmd)
def snap_info(mnode, snapname="", volname=""):
diff --git a/tests/functional/snapshot/test_snap_status_glusterd_restart.py b/tests/functional/snapshot/test_snap_status_glusterd_restart.py
new file mode 100644
index 0000000..a378737
--- /dev/null
+++ b/tests/functional/snapshot/test_snap_status_glusterd_restart.py
@@ -0,0 +1,162 @@
+# Copyright (C) 2020 Red Hat, Inc. <http://www.redhat.com>
+#
+# This program is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2 of the License, or
+# any later version.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with this program; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+
+"""
+Description:
+
+Test Cases in this module tests the
+snapshot Status when glusterd is restarted.
+
+"""
+from glusto.core import Glusto as g
+from glustolibs.gluster.exceptions import ExecutionError
+from glustolibs.gluster.gluster_base_class import GlusterBaseClass
+from glustolibs.gluster.gluster_base_class import runs_on
+from glustolibs.gluster.gluster_init import (restart_glusterd,
+ is_glusterd_running)
+from glustolibs.gluster.snap_ops import (snap_create,
+ get_snap_status,
+ get_snap_status_by_snapname,
+ snap_status_by_volname)
+
+
+@runs_on([['replicated', 'distributed-replicated', 'dispersed',
+ 'distributed', 'distributed-dispersed'],
+ ['glusterfs']])
+class TestSnapshotGlusterdRestart(GlusterBaseClass):
+
+ @classmethod
+ def setUpClass(cls):
+ cls.get_super_method(cls, 'setUpClass')()
+ cls.snapshots = [('snap-test-snap-status-gd-restart-%s-%s'
+ % (cls.volname, i))for i in range(0, 2)]
+
+ def setUp(self):
+
+ # SettingUp volume and Mounting the volume
+ self.get_super_method(self, 'setUp')()
+
+ ret = self.setup_volume()
+ if not ret:
+ raise ExecutionError("Failed to setup volume %s" % self.volname)
+ g.log.info("Volume %s has been setup successfully", self.volname)
+
+ def tearDown(self):
+
+ # Unmount and cleanup original volume
+
+ ret = self.cleanup_volume()
+ if not ret:
+ raise ExecutionError("Failed to umount the vol & cleanup Volume")
+ g.log.info("Successful in umounting the volume and Cleanup")
+
+ def test_snap_status_glusterd_restart(self):
+ # pylint: disable=too-many-statements, too-many-branches
+ """
+ Test Case:
+ 1. Create volume
+ 2. Create two snapshots with description
+ 3. Check snapshot status informations with snapname, volume name and
+ without snap name/volname.
+ 4. Restart glusterd on all nodes
+ 5. Follow step3 again and validate snapshot
+ """
+
+ # Creating snapshot with description
+ for snap in self.snapshots:
+ ret, _, _ = snap_create(self.mnode, self.volname, snap,
+ description='$p3C!@l C#@R@cT#R$')
+ self.assertEqual(ret, 0, ("Failed to create snapshot for volume %s"
+ % self.volname))
+ g.log.info("Snapshot %s created successfully"
+ " for volume %s", snap, self.volname)
+
+ # Validate snapshot status information
+ # Check snapshot status
+ snap_stat = get_snap_status(self.mnode)
+ self.assertIsNotNone(snap_stat, "failed to get snap status")
+ snap_count = 0
+ for snap in self.snapshots:
+ self.assertEqual(snap_stat[snap_count]['name'],
+ snap, "Failed to show snapshot status")
+ snap_count += 1
+ g.log.info("Successfully checked snapshot status")
+
+ # Check snapshot status using snap name
+ snap_status = get_snap_status_by_snapname(self.mnode,
+ self.snapshots[0])
+ self.assertIsNotNone(snap_status, "failed to get snap status")
+ self.assertEqual(snap_status['name'], "%s" % self.snapshots[0],
+ "Failed to show snapshot "
+ "status for %s" % self.snapshots[0])
+ g.log.info("Successfully checked snapshot status for %s",
+ self.snapshots[0])
+
+ # Check snapshot status using volname
+ ret, snap_vol_status, _ = snap_status_by_volname(self.mnode,
+ self.volname)
+ self.assertEqual(ret, 0, ("Failed to get snapshot statue "
+ "by volume name"))
+ self.assertIsNotNone(snap_vol_status, "failed to get snap status")
+ for snap in self.snapshots:
+ self.assertIn(snap, snap_vol_status,
+ "Failed to validate snapshot name")
+ g.log.info("Successfully validated snapshot status for %s",
+ self.volname)
+
+ # Restart Glusterd on all node
+ ret = restart_glusterd(self.servers)
+ self.assertTrue(ret, "Failed to stop glusterd")
+ g.log.info("Successfully stopped glusterd on all node")
+
+ # Check Glusterd status
+ ret = is_glusterd_running(self.servers)
+ self.assertEqual(ret, 0, "glusterd running on node ")
+ g.log.info("glusterd is not running")
+
+ # Validate snapshot status information
+ # Check snapshot status
+ snap_stat = get_snap_status(self.mnode)
+ self.assertIsNotNone(snap_stat, "failed to get snap status")
+ snap_count = 0
+ for snap in self.snapshots:
+ self.assertEqual(snap_stat[snap_count]['name'],
+ snap, "Failed to show snapshot status")
+ snap_count += 1
+ g.log.info("Successfully checked snapshot status")
+
+ # Check snapshot status using snap name
+ snap_status = get_snap_status_by_snapname(self.mnode,
+ self.snapshots[0])
+ self.assertIsNotNone(snap_status, "failed to get snap status")
+ self.assertEqual(snap_status['name'], "%s" % self.snapshots[0],
+ "Failed to show snapshot "
+ "status for %s" % self.snapshots[0])
+ g.log.info("Successfully checked snapshot status for %s",
+ self.snapshots[0])
+
+ # Check snapshot status using volname
+ ret, snap_vol_status, _ = snap_status_by_volname(self.mnode,
+ self.volname)
+ self.assertEqual(ret, 0, ("Failed to get snapshot statue "
+ "by volume name"))
+ self.assertIsNotNone(snap_vol_status, "failed to get snap status")
+ for snap in self.snapshots:
+ self.assertIn(snap, snap_vol_status,
+ "Failed to validate snapshot status "
+ "using volume name")
+ g.log.info("Successfully validated snapshot status for %s",
+ self.volname)