amoghrajesh commented on code in PR #54943:
URL: https://github.com/apache/airflow/pull/54943#discussion_r2311786093
##########
task-sdk/src/airflow/sdk/definitions/dag.py:
##########
@@ -1096,10 +1096,10 @@ def test(
from contextlib import ExitStack
from airflow import settings
- from airflow.configuration import secrets_backend_list
from airflow.models.dag import DAG as SchedulerDAG,
_get_or_create_dagrun
from airflow.models.dagrun import DagRun
from airflow.sdk import timezone
+ from airflow.sdk._shared.configuration import secrets_backend_list
Review Comment:
That, I kind of did intentionally. I have added a new
module(task-sdk/src/airflow/sdk/configuration.py_) which can be a public facing
API in case users / dag authors need to use it in dags etc, they could do
`airflow.sdk.configuration.conf` but for anything else, which is not consumed
by end users / dag authors, real path has to be used/
--
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]
