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

(TEST) Apply Florian's patch to 2.30.x #4204

Conversation

TomAugspurger
Copy link
Member

@TomAugspurger TomAugspurger commented Oct 30, 2020

Trying out #4176 against 2.30 to check if the failures on that PR are only present on master.

If this passes, I don't think we merge this PR. Instead, we should merge #4176 despite the failing tests and then just backport the merged PR to 2.30.x. That should play more nicely with versioneer.

@TomAugspurger TomAugspurger changed the title (TEST) Apply Florien's patch to 2.30.x (TEST) Apply Florian's patch to 2.30.x Oct 30, 2020
@TomAugspurger
Copy link
Member Author

The test failure on windows 3.7 was a know flaky test that's now skipped in #4166. It can be ignored.

@jrbourbeau
Copy link
Member

I opened a separate PR over in #4212 to see what test failures we currently have in the main branch and saw there are many failures on both Windows and Travis CI builds. Lots of the test failures (maybe all?) look the same as the ones we're seeing in this PR, but it's difficult to directly compare the builds when there are > 20 failures.

I ended up not releasing 2.30.1 last Friday as, I think, the Travis builds weren't completed late in the evening and then on Saturday I saw more tests were failing than expected. Given the current state of our test suite, I'm inclined to not release right now and instead focus on fixing tests. Thoughts @TomAugspurger?

Also, I can cross-post this over in the release issue on the community tracker

@TomAugspurger
Copy link
Member Author

Given the current state of our test suite, I'm inclined to not release right now and instead focus on fixing tests. Thoughts @TomAugspurger?

Agreed.

@jrbourbeau
Copy link
Member

Woo, tests pass 🎉 I'll carry on with releasing 2.30.1

@jrbourbeau jrbourbeau closed this Nov 3, 2020
@TomAugspurger
Copy link
Member Author

Great, thanks!

@TomAugspurger TomAugspurger deleted the connect-retry-if-handshake-fails-patch branch November 3, 2020 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants