Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Conversation

jeffdaily
Copy link
Contributor

No description provided.

@jeffdaily jeffdaily marked this pull request as ready for review July 5, 2022 16:31
@jeffdaily jeffdaily requested a review from jithunnair-amd July 5, 2022 16:38
Copy link
Contributor

@janeyx99 janeyx99 left a comment

Choose a reason for hiding this comment

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

Could I get some context about moving to 5.2?

@janeyx99
Copy link
Contributor

Also, before we merge this, can we verify it passes CI?

@jithunnair-amd
Copy link
Contributor

Could I get some context about moving to 5.2?

It's the latest ROCm version that was released. We always strive to keep the nightly binary builds on the latest ROCm release.

@jithunnair-amd
Copy link
Contributor

Also, before we merge this, can we verify it passes CI?

rocm_binary_builds

I think the ROCm CI builds already passed.

@jithunnair-amd
Copy link
Contributor

@janeyx99 Please let us know if we need something more in this PR to merge it. The pytorchbot command doesn't work on builder repo I think.
cc @seemethere

@janeyx99 janeyx99 merged commit ff781a1 into pytorch:main Jul 25, 2022
@janeyx99
Copy link
Contributor

merged

@jeffdaily
Copy link
Contributor Author

@janeyx99 On the corresponding pytorch PR I'm getting build failures for rocm 5.2 binaries because the docker image is missing.

pytorch/manylinux-builder:rocm5.2

How can we trigger a build and push of this image? I thought this current PR should have done that.

@janeyx99
Copy link
Contributor

@jeffdaily it looks liek the corresponding actions timed out https://github.com/pytorch/builder/actions/runs/2733195759

@jeffdaily
Copy link
Contributor Author

@jeffdaily it looks liek the corresponding actions timed out https://github.com/pytorch/builder/actions/runs/2733195759

Looks like rocm base image builds need more time. Can we increase the timeout? We need these images for nightly pytorch binaries.

@jeffdaily
Copy link
Contributor Author

@janeyx99 but the latest CI runs were all successful. Not sure why they timed out for the run you linked to.

https://github.com/pytorch/builder/pull/1033/checks?check_run_id=7201495074

jeffdaily added a commit to ROCm/pytorch that referenced this pull request Aug 8, 2022
pytorchmergebot pushed a commit to ROCm/pytorch that referenced this pull request Aug 10, 2022
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Aug 10, 2022
### Description
Updates ROCm arches for nightly binaries to 5.1.1 and 5.2, dropping 5.0.

Corresponding builder PR:
pytorch/builder#1033

### Testing
PR CI with extra label to enable binary builds.
Pull Request resolved: #82353
Approved by: https://github.com/janeyx99, https://github.com/malfet
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Aug 11, 2022
### Description
Updates ROCm arches for nightly binaries to 5.1.1 and 5.2, dropping 5.0.

Corresponding builder PR:
pytorch/builder#1033

### Testing
PR CI with extra label to enable binary builds.
Pull Request resolved: #82353
Approved by: https://github.com/janeyx99, https://github.com/malfet
facebook-github-bot pushed a commit to pytorch/pytorch that referenced this pull request Aug 11, 2022
Summary:
### Description
Updates ROCm arches for nightly binaries to 5.1.1 and 5.2, dropping 5.0.

Corresponding builder PR:
pytorch/builder#1033

### Testing
PR CI with extra label to enable binary builds.

Pull Request resolved: #82353
Approved by: https://github.com/janeyx99, https://github.com/malfet

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/2dae93b212de21b2dafc76327ba9c5ac90fa6be6

Reviewed By: seemethere

Differential Revision: D38624007

fbshipit-source-id: e52f7e7e0a307cac0becd26cd55984481ca6b12f
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants