From 9967eb8adbcf471d652337ed77add05480773a1a Mon Sep 17 00:00:00 2001
From: Rob Crittenden <rcritten@redhat.com>
Date: Thu, 28 Apr 2022 08:57:38 -0400
Subject: [PATCH 8/9] Relocate eval of debug/verbose in case they are set in
config file
Since the configuration file allows options to be set we need
to evaluate them after the merge.
Leaving version handling pre-config load since it makes no sense
within the config file.
https://bugzilla.redhat.com/show_bug.cgi?id=2079861
Signed-off-by: Rob Crittenden <rcritten@redhat.com>
---
src/ipahealthcheck/core/core.py | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/ipahealthcheck/core/core.py b/src/ipahealthcheck/core/core.py
index c04a6ee..6e6a928 100644
--- a/src/ipahealthcheck/core/core.py
+++ b/src/ipahealthcheck/core/core.py
@@ -346,12 +346,6 @@ class RunChecks:
if rval is not None:
return rval
- if options.verbose:
- logger.setLevel(logging.INFO)
-
- if options.debug:
- logger.setLevel(logging.DEBUG)
-
if options.config is not None:
config = read_config(options.config)
else:
@@ -366,6 +360,12 @@ class RunChecks:
self.options = config
options = config
+ if options.verbose:
+ logger.setLevel(logging.INFO)
+
+ if options.debug:
+ logger.setLevel(logging.DEBUG)
+
# pylint: disable=assignment-from-none
rval = self.pre_check()
# pylint: enable=assignment-from-none
--
2.31.1