summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDanny Al-Gaaf <danny.al-gaaf@bisect.de>2013-03-20 14:36:58 +0100
committerDanny Al-Gaaf <danny.al-gaaf@bisect.de>2013-03-20 14:36:58 +0100
commit87691dc46edfba11c370592dbb533772190be4b2 (patch)
tree62afac3703f7c576ca91d1704f9c38dac8d2788d
parent3575feb7d2e5f4e35c5df193a1f8c9f08f88fcf4 (diff)
downloadceph-87691dc46edfba11c370592dbb533772190be4b2.tar.gz
ceph-disk: cast output of subprocess.Popen() to str()
Cast output of subprocess.Popen() to str() to be able to use str.split() and str.splitlines() without warnings from pylint. Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
-rwxr-xr-xsrc/ceph-disk6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/ceph-disk b/src/ceph-disk
index 6d98fc32ada..1cf8ba96596 100755
--- a/src/ceph-disk
+++ b/src/ceph-disk
@@ -373,7 +373,7 @@ def get_conf(cluster, variable):
return None
elif ret != 0:
raise Error('getting variable from configuration failed')
- value = out.split('\n', 1)[0]
+ value = str(out).split('\n', 1)[0]
# don't differentiate between "var=" and no var set
if not value:
return None
@@ -406,7 +406,7 @@ def get_conf_with_default(cluster, variable):
e,
)
- value = out.split('\n', 1)[0]
+ value = str(out).split('\n', 1)[0]
return value
@@ -552,7 +552,7 @@ def get_free_partition_index(dev):
if not lines:
raise Error('parted failed to output anything')
- lines = lines.splitlines(True)
+ lines = str(lines).splitlines(True)
if lines[0] not in ['CHS;\n', 'CYL;\n', 'BYT;\n']:
raise Error('weird parted units', lines[0])