New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix a log-related rename mixup #1095
Fix a log-related rename mixup #1095
Conversation
jenkins, test this please |
2 similar comments
jenkins, test this please |
jenkins, test this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to otherwise. For the test to run properly you need to do a rebase.
pyanaconda/kickstart.py
Outdated
@@ -1125,7 +1125,9 @@ def execute(self, *args): | |||
# not set from the command line | |||
level = anaconda_logging.logLevelMap[self.level] | |||
anaconda_logging.logger.loglevel = level | |||
anaconda_logging.setHandlersLevel(anaconda_logging, level) | |||
# set log level for the "anaconde" root logger |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: anaconde
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, thanks! :)
Turns out there are still cases where working directly with the "anaconda" root logger makes sense, such as setting it's log level.
In this case we want to set the log level of the "anaconda" root logger, but this apparently got mixed up during the anaconda_log -> anaconda_logging rename and subsequent switch to structured logging. So fix the mixup and correctly use the "anaconda" root logger in the setHandlersLevel() call.
23085c1
to
878ab21
Compare
The first commit adds a getter for the "anaconda" root logger and the second commit fixes the actual mixup.