Skip to content
This repository was archived by the owner on Apr 7, 2022. It is now read-only.

[1LP][RFR] Fix setting logger level from config#10224

Merged
mshriver merged 1 commit intoManageIQ:masterfrom
tpapaioa:fix_logger_level
Jun 30, 2020
Merged

[1LP][RFR] Fix setting logger level from config#10224
mshriver merged 1 commit intoManageIQ:masterfrom
tpapaioa:fix_logger_level

Conversation

@tpapaioa
Copy link
Contributor

@tpapaioa tpapaioa commented Jun 29, 2020

The loggers created via cfme.utils.log.setup_logger were hardcoded to have the INFO log level, ignoring any settings in conf/env.local.yaml. The associated file handlers do get created with the level from the yaml, but because the logger's level is fixed at INFO, no DEBUG logging is possible. This PR uses the yaml settings when creating the logger too, so that settings like the following in conf/env.local.yaml will log debug statements in cfme.log.

logging:
    level: DEBUG

[or]

logging:
    cfme:
        level: DEBUG

@tpapaioa tpapaioa changed the title [WIPTEST] Fix setting of logger level from config [RFR] Fix setting logger level from config Jun 29, 2020
@jarovo jarovo changed the title [RFR] Fix setting logger level from config [1LP][RFR] Fix setting logger level from config Jun 29, 2020
@mshriver mshriver merged commit 5520b10 into ManageIQ:master Jun 30, 2020
@tpapaioa tpapaioa deleted the fix_logger_level branch June 30, 2020 13:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants