Skip to content

Skip parallel analysis node tests which hang in pytest full runs#75

Closed
mgrange1998 wants to merge 1 commit intofacebookresearch:mainfrom
mgrange1998:export-D83752642
Closed

Skip parallel analysis node tests which hang in pytest full runs#75
mgrange1998 wants to merge 1 commit intofacebookresearch:mainfrom
mgrange1998:export-D83752642

Conversation

@mgrange1998
Copy link
Contributor

Summary:
The PrivacyGuard Github actions began timing out on the parallel analysis nodes compute_outputs test.

Changes

  • Skips the two timing out tests
  • Adds a overall timeout to the Github

Investigation

I investigated into the root cause, and found the following

  1. The tests fail only when the full test suite is called. Running pytest on the specific file does not cause failure
  2. These tests hang when both running pytest and pytest --timeout=120

Differential Revision: D83752642

Summary:
The PrivacyGuard Github actions began timing out on the parallel analysis nodes compute_outputs test.

## Changes


- Skips the two timing out tests
- Adds a overall timeout to the Github 



## Investigation

 I investigated into the root cause, and found the following
1. The tests fail only when the full test suite is called. Running pytest on the specific file does not cause failure
2. These tests hang when both running `pytest` and `pytest --timeout=120`

Differential Revision: D83752642
@meta-cla meta-cla bot added the CLA Signed This label is managed by the Meta Open Source bot. label Oct 2, 2025
@meta-codesync
Copy link

meta-codesync bot commented Oct 2, 2025

@mgrange1998 has exported this pull request. If you are a Meta employee, you can view the originating Diff in D83752642.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 0cce9e9.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot. fb-exported Merged meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants