diff options
authorTom Zanussi <tom.zanussi@linux.intel.com>2014-07-10 13:25:14 -0500
committerRichard Purdie <richard.purdie@linuxfoundation.org>2014-07-16 10:06:21 +0100
commitb1c68e5446080f0743f7f8a530f3b00f90c8717d (patch)
parentfa325e44f5b429b4038022b31285af9c94672943 (diff)
wic: Error on parted non-zero error code
The current code uses msger.debug() to note errors, effectively squelching them if --debug isn't used. Apparently this is because it can return non-zero for some loop device failures. We don't care about loop devices, and not paying attention to the error code actually results in invalid images, so error out on parted failures as we should be. Signed-off-by: Tom Zanussi <tom.zanussi@linux.intel.com> Signed-off-by: Saul Wold <sgw@linux.intel.com> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/lib/mic/utils/partitionedfs.py b/scripts/lib/mic/utils/partitionedfs.py
index 6607466a83a..593cf1f3174 100644
--- a/scripts/lib/mic/utils/partitionedfs.py
+++ b/scripts/lib/mic/utils/partitionedfs.py
@@ -272,7 +272,7 @@ class PartitionedMount(Mount):
# parted always fails to reload part table with loop devices. This
# prevents us from distinguishing real errors based on return
# code.
- msger.debug("WARNING: parted returned '%s' instead of 0" % rc)
+ msger.error("WARNING: parted returned '%s' instead of 0 (use --debug for details)" % rc)
def __create_partition(self, device, parttype, fstype, start, size):
""" Create a partition on an image described by the 'device' object. """