[lvm-devel] master - lvmdbusd: Improve error msg to include PV

Tony Asleson tasleson at sourceware.org
Wed Apr 26 13:36:37 UTC 2017


Gitweb:        https://sourceware.org/git/?p=lvm2.git;a=commitdiff;h=0f31f10ac5881b2780b0083f5e45fed59aeb8013
Commit:        0f31f10ac5881b2780b0083f5e45fed59aeb8013
Parent:        e50fb06792518ff9a5a299172b77e9d22105d372
Author:        Tony Asleson <tasleson at redhat.com>
AuthorDate:    Sat Apr 22 00:07:52 2017 -0500
Committer:     Tony Asleson <tasleson at redhat.com>
CommitterDate: Wed Apr 26 07:31:08 2017 -0500

lvmdbusd: Improve error msg to include PV

Include PV device path when we believe it already exists.
---
 daemons/lvmdbusd/manager.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/daemons/lvmdbusd/manager.py b/daemons/lvmdbusd/manager.py
index 5bcdbb8..1ae9c8b 100644
--- a/daemons/lvmdbusd/manager.py
+++ b/daemons/lvmdbusd/manager.py
@@ -48,7 +48,7 @@ class Manager(AutomatedProperties):
 		pv = cfg.om.get_object_path_by_uuid_lvm_id(device, device)
 		if pv:
 			raise dbus.exceptions.DBusException(
-				MANAGER_INTERFACE, "PV Already exists!")
+				MANAGER_INTERFACE, "PV %s Already exists!" % device)
 
 		rc, out, err = cmdhandler.pv_create(create_options, [device])
 		Manager.handle_execute(rc, out, err)




More information about the lvm-devel mailing list