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

[3.11] gh-88177: Revert the new asyncio.sslproto implementation #95663

Closed
wants to merge 2 commits into from

Conversation

ambv
Copy link
Contributor

@ambv ambv commented Aug 4, 2022

This reverts #31275 and #31597 for Python 3.11 which enters Release Candidate stage tomorrow while an unresolved issue is present in the new code.

See #95573 for details.

@ambv ambv requested review from 1st1 and asvetlov as code owners August 4, 2022 16:53
@ambv ambv added the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Aug 4, 2022
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @ambv for commit 722e189 🤖

If you want to schedule another build, you need to add the ":hammer: test-with-buildbots" label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-buildbots Test PR w/ buildbots; report in status section label Aug 4, 2022
@ambv
Copy link
Contributor Author

ambv commented Aug 4, 2022

This PR will hopefully not be necessary. I did stress-test it with -j8 -F on my M1 Max and it looks solid:

== Tests result: INTERRUPTED ==
Test suite interrupted by signal SIGINT.

1696 tests OK.

Total duration: 4 hour 42 min
Tests result: INTERRUPTED

________________________________________________________
Executed in  282.18 mins    fish           external
   usr time  1180.99 mins   43.00 micros  1180.99 mins
   sys time   34.76 mins  366.00 micros   34.76 mins

@ambv ambv closed this Aug 5, 2022
@ambv ambv deleted the revert-gh-31275 branch August 5, 2022 07:36
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.

2 participants