ttanay commented on a change in pull request #4601: [AIRFLOW-3758] Fix circular import in WasbTaskHandler URL: https://github.com/apache/airflow/pull/4601#discussion_r262467410
########## File path: tests/test_logging_config.py ########## @@ -251,6 +251,14 @@ def test_1_9_config(self): finally: conf.remove_option('core', 'task_log_reader', remove_default=False) + def test_loading_remote_logging_with_wasb_handler(self): + """Test if logging can be configured successfully for Azure Blob Storage""" + from airflow.logging_config import configure_logging + conf.set('core', 'remote_logging', 'True') + conf.set('core', 'remote_log_conn_id', 'some_wasb') + conf.set('core', 'remote_base_log_folder', 'wasb://some-folder') + configure_logging() Review comment: Hey @ashb I tried this after `configure_logging` is called in the change above, and also in a DAG: ``` import logging logger = logging.getLogger('airflow.task.x') ``` But, the `logger.handlers` was an empty list `[]`. Am I missing something? ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services