From 11c033c222bde7122b974464dbfb8d3c8734b19b Mon Sep 17 00:00:00 2001 From: Tony Asleson Date: Fri, 9 Sep 2022 10:03:35 -0500 Subject: [PATCH] lvmdbusd: Include lvm debug output for lvmshell Move the option to add the debug file into lvm_full_report_json so that we collect the debug data when we fork & exec lvm and when we use lvm shell. --- daemons/lvmdbusd/cmdhandler.py | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/daemons/lvmdbusd/cmdhandler.py b/daemons/lvmdbusd/cmdhandler.py index 1a7943f01..0e344d3ca 100644 --- a/daemons/lvmdbusd/cmdhandler.py +++ b/daemons/lvmdbusd/cmdhandler.py @@ -121,12 +121,6 @@ def call_lvm(command, debug=False, line_cb=None, command.insert(0, cfg.LVM_CMD) command = add_no_notify(command) - # If we are running the fullreport command, we will ask lvm to output the debug - # data, so we can have the required information for lvm to debug the fullreport failures. - if "fullreport" in command: - fn = cfg.lvmdebug.setup() - add_config_option(command, "--config", "log {level=7 file=%s syslog=0}" % fn) - process = Popen(command, stdout=PIPE, stderr=PIPE, close_fds=True, env=os.environ) @@ -618,6 +612,11 @@ def lvm_full_report_json(): '--reportformat', 'json' ]) + # We are running the fullreport command, we will ask lvm to output the debug + # data, so we can have the required information for lvm to debug the fullreport failures. + fn = cfg.lvmdebug.setup() + add_config_option(cmd, "--config", "log {level=7 file=%s syslog=0}" % fn) + rc, out, err = call(cmd) # When we have an exported vg the exit code of lvs or fullreport will be 5 if rc == 0 or rc == 5: