Skip to content

Conversation

@aelavender
Copy link

@aelavender aelavender commented Nov 13, 2025

Most CI has not been running in this repo. This PR re-enables it.

  1. We had a bunch of conditionals in our jobs like if github.repository == 'pytorch/data'. Since we moved to meta-pytorch, these checks have failed and jobs are skipped. Update to point at the correct repo.
  2. Jobs testing against python 3.9 were failing to install torch. This is because torch trunk no longer supports py3.9 and they have stopped building nightlies for it. Update to remove p3.9 from our matrix.
  3. Similar to the above: update linter and mypy to use py3.13 instead of py3.9.

The failed job Run Nodes Tests / test (windows-latest, 3.11) is a legit test failure. This test appears to rely on some timeouts, so might be flaky.

As of this writing, still waiting on StatefulDataLoader tests to complete (these take a long time), but they are at least kicked off correctly, which is an improvement over them being skipped.

Note: currently we do NOT have CI running against python 3.14 or 3.14t. We should probably add this.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 13, 2025
@aelavender aelavender changed the title [ci] fix hardcoded repo owner [ci] Fix CI Nov 13, 2025
@aelavender aelavender marked this pull request as ready for review November 13, 2025 21:53
Copy link
Contributor

@divyanshk divyanshk left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you.

@meta-codesync
Copy link

meta-codesync bot commented Nov 13, 2025

@aelavender has imported this pull request. If you are a Meta employee, you can view this in D87016927.

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 Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants