[AIRFLOW-1457] fix user-defined log import #2648
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We can use
json.load
to import a user defined json config for passing intologging.config.dictConfig
Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA
Description
user-defined logging configuration provided as part of AIRFLOW-1457 is broken- current code tries to import a string value pulled from the config. This always fails and results in the default config being loaded.
Since contents of
logging_config_path
will get passed tologging.config.DictConfig
as a dictionary, it should be json de/serializable and so we can usejson.load
to read the file instead.Tests
I can add tests at some point if you want, but settings.py currently does not have any unit tests and this is a pretty core functionality that is broken so I really just want to get it working :)
Commits