Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecation warning for cgitb in Plugins Manager #41732

Merged

Conversation

jscheffl
Copy link
Contributor

Recently when running Airflow (locally but probably also when deployed) a lot of warnings are emitted by Plugins Manager like:

/opt/airflow/airflow/plugins_manager.py:30 DeprecationWarning: 'cgitb' is deprecated and slated for removal in Python 3.13

This is caused by a import in plugins_manager.py - and surprisingly besides this import all other types of the specified list actually use HookLineageReader - unable to trace this down, this PR removes the import and traces. Less noise in logs.

As this also will bother all users in Airfow 2.10, I propose to fix this also in the 2.10 code-line

@potiuk
Copy link
Member

potiuk commented Aug 27, 2024

surprisingly besides this import all other types of the specified list actually use HookLineageReader

Likely it's simply a mistake,

@potiuk potiuk merged commit 83ba17f into apache:main Aug 27, 2024
52 checks passed
potiuk pushed a commit to potiuk/airflow that referenced this pull request Aug 27, 2024
potiuk added a commit that referenced this pull request Aug 27, 2024
utkarsharma2 pushed a commit that referenced this pull request Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:plugins type:bug-fix Changelog: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants