[GitHub] [airflow] tirkarthi edited a comment on pull request #22588: Print configuration on scheduler startup.

[GitBox <gi...@apache.org>]

tirkarthi edited a comment on pull request #22588:
URL: https://github.com/apache/airflow/pull/22588#issuecomment-1084924730


   Thanks @jedcunningham and @potiuk for the feedback. I have modified the PR so that non-default configuration is displayed along with source in [] as below indicating whether it's from environment variable / airflow.cfg / cmd. For signal SIGUSR1 in non-daemon mode I have added a handler that prints all configuration including default. I have added tests to ensure "default" is filtered by default :)
   
   I have used print since `airflow config list` also used print. Let me know if this needs to be changed to logger and using `log.info`. I have also not added syntax highlighting. Let me know your feedback on the format so that I can modify accordingly.
   
   
   ```ini
   $ airflow scheduler
     ____________       _____________
    ____    |__( )_________  __/__  /________      __
   ____  /| |_  /__  ___/_  /_ __  /_  __ \_ | /| / /
   ___  ___ |  / _  /   _  __/ _  / / /_/ /_ |/ |/ /
    _/_/  |_/_/  /_/    /_/    /_/  \____/____/|__/
   [core]
   dags_folder = < hidden > [env var]
   hostname_callable = socket.getfqdn [airflow.cfg]
   default_timezone = utc [airflow.cfg]
   executor = < hidden > [env var]
   sql_alchemy_conn = < hidden > [env var]
   sql_engine_encoding = utf-8 [airflow.cfg]
   sql_alchemy_pool_enabled = True [airflow.cfg]
   ```


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org