From 6868c7adb3a35cdda390dd87050d6661067c23a0 Mon Sep 17 00:00:00 2001 From: Vishvananda Ishaya Date: Wed, 26 Mar 2014 12:09:28 -0700 Subject: [PATCH] Pass the mirrorlog option as two arguments. We don't use shell=True when we Popen, so we have to keep all arguments as separate values. Change-Id: Ic35fad36be72386d3496920e259c32fe16625046 Closes-bug: 1298017 --- cinder/brick/local_dev/lvm.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cinder/brick/local_dev/lvm.py b/cinder/brick/local_dev/lvm.py index 37dc1742f8c..0cca76af0f4 100644 --- a/cinder/brick/local_dev/lvm.py +++ b/cinder/brick/local_dev/lvm.py @@ -461,7 +461,7 @@ class LVM(executor.Executor): if mirror_count > 0: cmd.extend(['-m', mirror_count, '--nosync', - '--mirrorlog mirrored']) + '--mirrorlog', 'mirrored']) terras = int(size_str[:-1]) / 1024.0 if terras >= 1.5: rsize = int(2 ** math.ceil(math.log(terras) / math.log(2)))