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

Quick fix: Ignore python internal deprecation warning #6004

Merged
merged 3 commits into from
Sep 18, 2021

Conversation

Dreamsorcerer
Copy link
Member

A change to deprecation warnings in the last python 3.9 release has resulted in warnings appearing in some internal asyncio code. We'll just ignore them for now (think they are fixed in the next release).

For some bizarre reason, they only seem to happen on Windows though...

@Dreamsorcerer Dreamsorcerer added the bot:chronographer:skip This PR does not need to include a change note label Sep 14, 2021
@Dreamsorcerer
Copy link
Member Author

Not such a simple fix, as there are tests which assert the number of DeprecationWarnings...

setup.cfg Outdated Show resolved Hide resolved
@webknjaz
Copy link
Member

For some bizarre reason, they only seem to happen on Windows though...

In fact, this was happening on F34 as well so @bmbouter temporarily ignored this in #6002.

@Dreamsorcerer
Copy link
Member Author

Weird that it doesn't happen in CI unless it's Windows.

@webknjaz
Copy link
Member

@bmbouter what's your CPython version, again?

@bmbouter
Copy link
Contributor

@bmbouter what's your CPython version, again?

Python 3.9.7

Co-authored-by: Sviatoslav Sydorenko <sviat@redhat.com>
@Dreamsorcerer
Copy link
Member Author

Alternative (because this still fails some tests): #6006

@@ -53,6 +53,7 @@ addopts =
filterwarnings =
error
ignore:module 'ssl' has no attribute 'OP_NO_COMPRESSION'. The Python interpreter is compiled against OpenSSL < 1.0.0. Ref. https.//docs.python.org/3/library/ssl.html#ssl.OP_NO_COMPRESSION:UserWarning
ignore:The loop argument is deprecated since Python 3.8, and scheduled for removal in Python 3.10.:DeprecationWarning:asyncio.base_events
Copy link
Member

Choose a reason for hiding this comment

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

Could you add a comment with the explanation right above this line? Maybe with links too. And then, go ahead and merge it.

Copy link
Member Author

Choose a reason for hiding this comment

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

Is it worth merging this when it doesn't fix all the tests? Or should we just merge #6006 instead? Or both?

I'll probably just open an issue though to remind myself to revert the changes in a few weeks.

Copy link
Member

Choose a reason for hiding this comment

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

I think if it improves things at least a little, we should merge it. And yes, we could revert later. Plus I'd combine with #6006 too.

Copy link
Member Author

Choose a reason for hiding this comment

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

Right, you'll need to tweak the required status to get them merged.

@webknjaz webknjaz merged commit 0afb7dd into 3.8 Sep 18, 2021
@webknjaz webknjaz deleted the Dreamsorcerer-patch-5 branch September 18, 2021 11:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot:chronographer:skip This PR does not need to include a change note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants