Skip to content

Conversation

@justinpakzad
Copy link
Contributor

@justinpakzad justinpakzad commented Jan 1, 2026

Updated standard, elasticsearch, opensearch, and microsoft providers to use airflow.sdk.configuration.conf instead of airflow.configuration.conf. One thing worth mentioning is that for elastic search I needed to add the elasticsearch_configs to the unit_tests.cfg file and fallback="_all" for the index_patterns. Without those some of the tests were failing and unable to read the necessary config.

related: #60000


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

@justinpakzad
Copy link
Contributor Author

The test failure seems to be unrelated to any of the changes I made: "Login output not found before command output for 'airflowctl dags update --dag-id=example_bash_operator --no-is-paused'". I rebased before pushing as well to make sure my branch is up to date.

@justinpakzad justinpakzad force-pushed the update-providers-to-sdk-config branch from bf5d75d to 06dea76 Compare January 2, 2026 17:28
@potiuk potiuk merged commit 88aad6a into apache:main Jan 3, 2026
128 checks passed
chirodip98 pushed a commit to chirodip98/airflow-contrib that referenced this pull request Jan 5, 2026
chirodip98 pushed a commit to chirodip98/airflow-contrib that referenced this pull request Jan 8, 2026
chirodip98 pushed a commit to chirodip98/airflow-contrib that referenced this pull request Jan 9, 2026
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
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