From: Vishvananda Ishaya <vishvananda@gmail.com>
Date: Wed, 26 Mar 2014 19:09:28 +0000 (-0700)
Subject: Pass the mirrorlog option as two arguments.
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=6868c7adb3a35cdda390dd87050d6661067c23a0;p=openstack-build%2Fcinder-build.git

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
---

diff --git a/cinder/brick/local_dev/lvm.py b/cinder/brick/local_dev/lvm.py
index 37dc1742f..0cca76af0 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)))