eladkal commented on code in PR #61257: URL: https://github.com/apache/airflow/pull/61257#discussion_r2753184632
########## airflow-core/docs/administration-and-deployment/logging-monitoring/advanced-logging-configuration.rst: ########## @@ -49,50 +49,64 @@ You can read more about standard python logging classes (Loggers, Handlers, Form Create a custom logging class ----------------------------- -Configuring your logging classes can be done via the ``logging_config_class`` option in ``airflow.cfg`` file. -This configuration should specify the import path to a configuration compatible with -:func:`logging.config.dictConfig`. If your file is a standard import location, then you should set a -:envvar:`PYTHONPATH` environment variable. -Follow the steps below to enable custom logging config class: +.. important:: -#. Start by setting environment variable to known directory e.g. ``~/airflow/`` + **Migration Note: Logging in Airflow 3.x and Later** Review Comment: Migration from 2 to 3 notes should be only in https://airflow.apache.org/docs/apache-airflow/stable/installation/upgrading_to_airflow3.html#upgrading-to-airflow-3 The airflow-core/docs/administration-and-deployment/logging-monitoring/advanced-logging-configuration.rst doc should explain how advanced logging should be set. It should not consider 2->3 steps. It should assume you are starting fresh directly on Airflow 3. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
