summaryrefslogtreecommitdiffstats
path: root/commands/storage/lmi
diff options
context:
space:
mode:
authorMichal Minar <miminar@redhat.com>2014-02-13 08:43:56 +0100
committerMichal Minar <miminar@redhat.com>2014-02-13 14:41:32 +0100
commitfc9c45fa75cb13693781e68407fcd7c8b051ec95 (patch)
tree42f5cf64253b886fc0b7770375d3274cd90b1438 /commands/storage/lmi
parent7a795f8d6a5772a85b3b03deb140ac314ed5a10b (diff)
downloadopenlmi-scripts-fc9c45fa75cb13693781e68407fcd7c8b051ec95.tar.gz
openlmi-scripts-fc9c45fa75cb13693781e68407fcd7c8b051ec95.tar.xz
openlmi-scripts-fc9c45fa75cb13693781e68407fcd7c8b051ec95.zip
turned logging messages into sentences
Which means that all messages have first letter uppercased and are terminated with a dot.
Diffstat (limited to 'commands/storage/lmi')
-rw-r--r--commands/storage/lmi/scripts/storage/common.py8
-rw-r--r--commands/storage/lmi/scripts/storage/mount.py4
2 files changed, 6 insertions, 6 deletions
diff --git a/commands/storage/lmi/scripts/storage/common.py b/commands/storage/lmi/scripts/storage/common.py
index 5fc6989..57824cf 100644
--- a/commands/storage/lmi/scripts/storage/common.py
+++ b/commands/storage/lmi/scripts/storage/common.py
@@ -88,7 +88,7 @@ def str2device(ns, device):
if len(devices) > 1:
raise LmiFailed("Too many devices with name '%s' found" % (device,))
- LOG().debug("String %s translated to device '%s'",
+ LOG().debug("String %s translated to device '%s'.",
device, devices[0].DeviceID)
return devices[0]
@@ -120,11 +120,11 @@ def str2vg(ns, vg):
% {'vg': escape_cql(vg)}
vgs = ns.wql(query)
if not vgs:
- raise LmiFailed("Volume Group '%s' not found" % (vg,))
+ raise LmiFailed("Volume Group '%s' not found." % (vg,))
if len(vgs) > 1:
- raise LmiFailed("Too many volume groups with name '%s' found" % (vg,))
+ raise LmiFailed("Too many volume groups with name '%s' found." % (vg,))
- LOG().debug("String %s translated to Volume Group '%s'",
+ LOG().debug("String %s translated to Volume Group '%s'.",
vg, vgs[0].InstanceID)
return vgs[0]
diff --git a/commands/storage/lmi/scripts/storage/mount.py b/commands/storage/lmi/scripts/storage/mount.py
index f0066b2..baf10a1 100644
--- a/commands/storage/lmi/scripts/storage/mount.py
+++ b/commands/storage/lmi/scripts/storage/mount.py
@@ -87,7 +87,7 @@ def get_setting_from_opts(ns, options, other_options):
(ret, outparams, _err) = cap.CreateSetting()
if ret != 0:
- raise LmiFailed('Could not create setting')
+ raise LmiFailed('Could not create setting.')
setting = outparams['setting'].to_instance()
@@ -109,7 +109,7 @@ def get_setting_from_opts(ns, options, other_options):
# insist on using a number with FileSystemCheckOrder
if opts[0] == 'FileSystemCheckOrder':
if not opts[1].isdigit():
- raise LmiFailed('Value of FileSystemCheckOrder must be a number')
+ raise LmiFailed('Value of FileSystemCheckOrder must be a number.')
opts[1] = int(opts[1])
else:
# otherwise check for true/false possibilities