Skip to content

Conversation

@amoghrajesh
Copy link
Contributor

from airflow.sdk.configuration import conf is too long of a path for users and providers to consume conf from. It would be much more convenient if it were from airflow.sdk import conf

Tested that below works as expected:

from airflow.sdk import conf
conf.get("core", "security")
Out[3]: ''
conf.get("api", "port")
Out[4]: '8080'

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@amoghrajesh amoghrajesh requested a review from uranusjr December 4, 2025 07:24
@amoghrajesh amoghrajesh added this to the Airflow 3.2.0 milestone Dec 4, 2025
@amoghrajesh amoghrajesh self-assigned this Dec 4, 2025
@amoghrajesh amoghrajesh requested a review from uranusjr December 5, 2025 06:52
@amoghrajesh amoghrajesh merged commit 64102a0 into apache:main Dec 5, 2025
172 of 173 checks passed
@amoghrajesh amoghrajesh deleted the shorter-conf-import-path branch December 5, 2025 16:54
Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

amoghrajesh added a commit to astronomer/airflow that referenced this pull request Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants