xBis7 commented on PR #56187: URL: https://github.com/apache/airflow/pull/56187#issuecomment-3569188516
The traces and the metrics code needs to read the config but the `conf` object that holds the initialized config, isn't available under the shared configuration package. #57744 adds a duplicate `conf` object under the sdk. https://github.com/apache/airflow/pull/57744/files#diff-bd3970fc27aea7f03dbcc37f56f286b537ef981025c33b00d8ea6ff2fe599093R252-R257 I'm trying to avoid having the `observability` package depend on `airflow-core` or `task-sdk`. Ideally, the code that initializes and reads the configuration, will be moved under shared. That's what I tried to explore with https://github.com/xBis7/airflow/commit/f50a634e81558ffc1d62de9e2c29a4ded3db583c. I wanted to see how wide the scope is. -- 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]
