Skip to content

Disable flaky test_invalid_names in test_rpc.py#25916

Closed
mrshenli wants to merge 1 commit intopytorch:masterfrom
mrshenli:names
Closed

Disable flaky test_invalid_names in test_rpc.py#25916
mrshenli wants to merge 1 commit intopytorch:masterfrom
mrshenli:names

Conversation

@mrshenli
Copy link
Copy Markdown
Contributor

@pietern discovered that test_invalid_names is flaky on master. #25656 is potentially the fix. Disable this test for now and will try to add it again when #25656 is in.

@mrshenli mrshenli requested a review from pietern September 10, 2019 14:14
@pietern
Copy link
Copy Markdown
Contributor

pietern commented Sep 10, 2019

What about the initialization/destruction order could make this segfault?

@mrshenli
Copy link
Copy Markdown
Contributor Author

I am not sure what happened here. The error was thrown from Pair::read(), but this test should not trigger any communication on process groups except init_process_group.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@mrshenli has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@mrshenli merged this pull request in 2655b27.

laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
pietern discovered that `test_invalid_names` is flaky on master. pytorch#25656 is potentially the fix. Disable this test for now and will try to add it again when pytorch#25656 is in.
Pull Request resolved: pytorch#25916

Differential Revision: D17287496

Pulled By: mrshenli

fbshipit-source-id: 9313958d3480c2bab20cd2341837c7821e3bb1b5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants