Skip to content

Conversation

@KoviAnusha
Copy link
Contributor

@KoviAnusha KoviAnusha commented Oct 19, 2025

closes: #55889

Replaced DeprecationWarning with DeprecatedImportWarning across airflow-core in the following files

  1. airflow/init.py
  2. airflow/datasets/init.py
  3. airflow/datasets/metadata.py
  4. airflow/models/dag.py
  5. airflow/models/dagbag.py
  6. airflow/secrets/init.py
  7. airflow/timetables/datasets.py
  8. airflow/utils/dag_parsing_context.py

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

@KoviAnusha KoviAnusha changed the title Replace DeprecationWarning with DeprecatedImportWarning in airflow/__init__.py Replace DeprecationWarning with DeprecatedImportWarning across airflow-core Oct 19, 2025
@potiuk potiuk force-pushed the deprecation_changes branch from c7295f3 to abe78f8 Compare October 19, 2025 05:48
Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

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

Nice thanks @KoviAnusha!

Could you run a quick test by trying to run DAGs with some or all of those imports to see if you can see those errors in the task logs? I am OK to approve once thats done.

@KoviAnusha
Copy link
Contributor Author

@amoghrajesh, I have created a sample DAG, and I can see DeprecatedImportWarning in the warnings. Please let me know if this is okay?

Screenshot 2025-10-19 at 8 06 05 PM

@KoviAnusha KoviAnusha changed the title Replace DeprecationWarning with DeprecatedImportWarning across airflow-core Replace DeprecationWarning with DeprecatedImportWarning 8 files in across airflow-core Oct 20, 2025
@amoghrajesh
Copy link
Contributor

That looks good thanks!

@amoghrajesh amoghrajesh added the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Oct 20, 2025
@amoghrajesh amoghrajesh added this to the Airflow 3.1.1 milestone Oct 20, 2025
@amoghrajesh amoghrajesh merged commit 7d02211 into apache:main Oct 20, 2025
81 checks passed
@github-actions
Copy link

Backport failed to create: v3-1-test. View the failure log Run details

Status Branch Result
v3-1-test Commit Link

You can attempt to backport this manually by running:

cherry_picker 7d02211 v3-1-test

This should apply the commit to the v3-1-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Oct 20, 2025
@KoviAnusha KoviAnusha changed the title Replace DeprecationWarning with DeprecatedImportWarning 8 files in across airflow-core Replace DeprecationWarning with DeprecatedImportWarning 8 files across airflow-core Oct 21, 2025
@kaxil kaxil modified the milestones: Airflow 3.1.1, Airflow 3.2.0 Oct 21, 2025
TyrellHaywood pushed a commit to TyrellHaywood/airflow that referenced this pull request Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clean up usages of DeprecationWarning across airflow-core

4 participants