Skip to content

migrate to linux_job_v2 and manylinux 2_28 #1302

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

Merged
merged 7 commits into from
Nov 18, 2024
Merged

Conversation

HDCharles
Copy link
Contributor

@HDCharles HDCharles commented Nov 18, 2024

this is to handle the migration of dev-infra from manylinux_2_26 to 2_28: pytorch/pytorch#123649

this required us update to linux_job_v2 for everything using nightly pytorch.
float8 tests: change to linux_job_v2 and remove binutils (they don't seem to be needed anymore)
nightly smoke tests: change to linux_job_v2
regression tests: only the nightlies need to use the new linux_job_v2, the rest needs to remain unchanged. We are currentlly pinning the pytorch version due to the bad_alloc tests but when that gets unpinned in #1283 the same changes for float8 test will need to be made to the nightly regression tests

these changes are working if CI is passing here

wheel builds: we update the manylinux version to 2_28, this was tested with https://github.com/pytorch/ao/actions/runs/11901019589 where all the cpu/cuda jobs pass, though we are still seeing failures with rocm jobs, that is a larger issue, unrelated to this PR, fixes for that are ongoing in pytorch/pytorch#140631

Copy link

pytorch-bot bot commented Nov 18, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/1302

Note: Links to docs will display an error until the docs builds have been completed.

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ No Failures

As of commit 6b5f811 with merge base d4ca98f (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-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 18, 2024
@HDCharles HDCharles added the topic: not user facing Use this tag if you don't want this PR to show up in release notes label Nov 18, 2024
@HDCharles HDCharles changed the title Update float8_test.yml to use linux_job_v2 migrate to linux_job_v2 and manylinux 2_28 Nov 18, 2024
Copy link
Contributor

@atalman atalman left a comment

Choose a reason for hiding this comment

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

lgtm

@HDCharles HDCharles merged commit bce2abb into main Nov 18, 2024
18 checks passed
sunjiweiswift pushed a commit to sunjiweiswift/ao that referenced this pull request Nov 25, 2024
* Update float8_test.yml to use linux_job_v2

* Update nightly_smoke_test.yml

* Update float8_test.yml no binutils

* Update post_build_script.sh

* Update post_build_script.sh

* Update regression_test.yml

* Update regression_test.yml
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. topic: not user facing Use this tag if you don't want this PR to show up in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants