|
 |
cead9d |
From 04b824ebfcf80c648d5855f10bc30fde45fd62eb Mon Sep 17 00:00:00 2001
|
|
|
cead9d |
From: Sunny Kumar <sunkumar@redhat.com>
|
|
|
cead9d |
Date: Thu, 26 Mar 2020 10:46:16 +0000
|
|
|
cead9d |
Subject: [PATCH 361/362] snap_scheduler: python3 compatibility and new test
|
|
|
cead9d |
case
|
|
|
cead9d |
MIME-Version: 1.0
|
|
|
cead9d |
Content-Type: text/plain; charset=UTF-8
|
|
|
cead9d |
Content-Transfer-Encoding: 8bit
|
|
|
cead9d |
|
|
|
cead9d |
Problem:
|
|
|
cead9d |
"snap_scheduler.py init" command failing with the below traceback:
|
|
|
cead9d |
|
|
|
cead9d |
[root@dhcp43-104 ~]# snap_scheduler.py init
|
|
|
cead9d |
Traceback (most recent call last):
|
|
|
cead9d |
File "/usr/sbin/snap_scheduler.py", line 941, in <module>
|
|
|
cead9d |
sys.exit(main(sys.argv[1:]))
|
|
|
cead9d |
File "/usr/sbin/snap_scheduler.py", line 851, in main
|
|
|
cead9d |
initLogger()
|
|
|
cead9d |
File "/usr/sbin/snap_scheduler.py", line 153, in initLogger
|
|
|
cead9d |
logfile = os.path.join(process.stdout.read()[:-1], SCRIPT_NAME + ".log")
|
|
|
cead9d |
File "/usr/lib64/python3.6/posixpath.py", line 94, in join
|
|
|
cead9d |
genericpath._check_arg_types('join', a, *p)
|
|
|
cead9d |
File "/usr/lib64/python3.6/genericpath.py", line 151, in _check_arg_types
|
|
|
cead9d |
raise TypeError("Can't mix strings and bytes in path components") from None
|
|
|
cead9d |
TypeError: Can't mix strings and bytes in path components
|
|
|
cead9d |
|
|
|
cead9d |
Solution:
|
|
|
cead9d |
|
|
|
cead9d |
Added the 'universal_newlines' flag to Popen to support backward compatibility.
|
|
|
cead9d |
|
|
|
cead9d |
Added a basic test for snapshot scheduler.
|
|
|
cead9d |
|
|
|
cead9d |
Backport Of:
|
|
|
cead9d |
|
|
|
cead9d |
>Upstream Patch: https://review.gluster.org/#/c/glusterfs/+/24257/
|
|
|
cead9d |
>Change-Id: I78e8fabd866fd96638747ecd21d292f5ca074a4e
|
|
|
cead9d |
>Fixes: #1134
|
|
|
cead9d |
>Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
|
|
|
cead9d |
|
|
|
cead9d |
BUG: 1817369
|
|
|
cead9d |
Change-Id: I78e8fabd866fd96638747ecd21d292f5ca074a4e
|
|
|
cead9d |
Signed-off-by: Sunny Kumar <sunkumar@redhat.com>
|
|
|
cead9d |
Reviewed-on: https://code.engineering.redhat.com/gerrit/196482
|
|
|
cead9d |
Tested-by: RHGS Build Bot <nigelb@redhat.com>
|
|
|
cead9d |
Reviewed-by: Sunil Kumar Heggodu Gopala Acharya <sheggodu@redhat.com>
|
|
|
cead9d |
---
|
|
|
cead9d |
extras/snap_scheduler/snap_scheduler.py | 2 +-
|
|
|
cead9d |
tests/basic/volume-snap-scheduler.t | 49 +++++++++++++++++++++++++++++++++
|
|
|
cead9d |
2 files changed, 50 insertions(+), 1 deletion(-)
|
|
|
cead9d |
create mode 100644 tests/basic/volume-snap-scheduler.t
|
|
|
cead9d |
|
|
|
cead9d |
diff --git a/extras/snap_scheduler/snap_scheduler.py b/extras/snap_scheduler/snap_scheduler.py
|
|
|
cead9d |
index a66c5e3..5a29d41 100755
|
|
|
cead9d |
--- a/extras/snap_scheduler/snap_scheduler.py
|
|
|
cead9d |
+++ b/extras/snap_scheduler/snap_scheduler.py
|
|
|
cead9d |
@@ -149,7 +149,7 @@ def initLogger():
|
|
|
cead9d |
sh.setFormatter(formatter)
|
|
|
cead9d |
|
|
|
cead9d |
process = subprocess.Popen(["gluster", "--print-logdir"],
|
|
|
cead9d |
- stdout=subprocess.PIPE)
|
|
|
cead9d |
+ stdout=subprocess.PIPE, universal_newlines=True)
|
|
|
cead9d |
logfile = os.path.join(process.stdout.read()[:-1], SCRIPT_NAME + ".log")
|
|
|
cead9d |
|
|
|
cead9d |
fh = logging.FileHandler(logfile)
|
|
|
cead9d |
diff --git a/tests/basic/volume-snap-scheduler.t b/tests/basic/volume-snap-scheduler.t
|
|
|
cead9d |
new file mode 100644
|
|
|
cead9d |
index 0000000..a638c5c
|
|
|
cead9d |
--- /dev/null
|
|
|
cead9d |
+++ b/tests/basic/volume-snap-scheduler.t
|
|
|
cead9d |
@@ -0,0 +1,49 @@
|
|
|
cead9d |
+#!/bin/bash
|
|
|
cead9d |
+
|
|
|
cead9d |
+. $(dirname $0)/../include.rc
|
|
|
cead9d |
+. $(dirname $0)/../volume.rc
|
|
|
cead9d |
+
|
|
|
cead9d |
+cleanup;
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST glusterd;
|
|
|
cead9d |
+TEST pidof glusterd;
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST $CLI volume create $V0 replica 2 $H0:$B0/${GMV0}{1,2,3,4};
|
|
|
cead9d |
+TEST $CLI volume start $V0
|
|
|
cead9d |
+
|
|
|
cead9d |
+## Create, start and mount meta_volume as
|
|
|
cead9d |
+## snap_scheduler expects shared storage to be enabled.
|
|
|
cead9d |
+## This test is very basic in nature not creating any snapshot
|
|
|
cead9d |
+## and purpose is to validate snap scheduling commands.
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST $CLI volume create $META_VOL replica 3 $H0:$B0/${META_VOL}{1,2,3};
|
|
|
cead9d |
+TEST $CLI volume start $META_VOL
|
|
|
cead9d |
+TEST mkdir -p $META_MNT
|
|
|
cead9d |
+TEST glusterfs -s $H0 --volfile-id $META_VOL $META_MNT
|
|
|
cead9d |
+
|
|
|
cead9d |
+##function to check status
|
|
|
cead9d |
+function check_status_scheduler()
|
|
|
cead9d |
+{
|
|
|
cead9d |
+ local key=$1
|
|
|
cead9d |
+ snap_scheduler.py status | grep -F "$key" | wc -l
|
|
|
cead9d |
+}
|
|
|
cead9d |
+
|
|
|
cead9d |
+##Basic snap_scheduler command test init/enable/disable/list
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST snap_scheduler.py init
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST snap_scheduler.py enable
|
|
|
cead9d |
+
|
|
|
cead9d |
+EXPECT 1 check_status_scheduler "Enabled"
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST snap_scheduler.py disable
|
|
|
cead9d |
+
|
|
|
cead9d |
+EXPECT 1 check_status_scheduler "Disabled"
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST snap_scheduler.py list
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST $CLI volume stop $V0;
|
|
|
cead9d |
+
|
|
|
cead9d |
+TEST $CLI volume delete $V0;
|
|
|
cead9d |
+
|
|
|
cead9d |
+cleanup;
|
|
|
cead9d |
--
|
|
|
cead9d |
1.8.3.1
|
|
|
cead9d |
|