diff options
author | Michal Minar <miminar@redhat.com> | 2014-02-13 08:43:56 +0100 |
---|---|---|
committer | Michal Minar <miminar@redhat.com> | 2014-02-13 14:41:32 +0100 |
commit | fc9c45fa75cb13693781e68407fcd7c8b051ec95 (patch) | |
tree | 42f5cf64253b886fc0b7770375d3274cd90b1438 /commands/storage/lmi | |
parent | 7a795f8d6a5772a85b3b03deb140ac314ed5a10b (diff) | |
download | openlmi-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.py | 8 | ||||
-rw-r--r-- | commands/storage/lmi/scripts/storage/mount.py | 4 |
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 |