Skip to content

Conversation

@atalman
Copy link
Contributor

@atalman atalman commented Oct 24, 2023

Pins all Docker images used for building to Sept 19 timeframe (time of final RC build for release 2.1.0)

@atalman atalman requested a review from a team as a code owner October 24, 2023 22:01
@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Oct 24, 2023
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 24, 2023

🔗 Helpful Links

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

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

❌ 4 New Failures, 3 Unrelated Failures

As of commit 7c0c64c with merge base 138e289 (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

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

@atalman atalman changed the title [release only] Fix docker release build for 2.1.1 - use generate_pytorch_version to extract version [release only] Pin Docker images for release. Fix docker release build Oct 25, 2023
@atalman atalman added ciflow/binaries_conda ciflow/binaries_wheel Trigger binary build and upload jobs for wheel on the PR ciflow/binaries_libtorch Trigger binary build and upload jobs for libtorch on the PR labels Oct 25, 2023
@atalman atalman changed the title [release only] Pin Docker images for release. Fix docker release build [release only] 1 - Pin Docker images for release. 2 - Fix docker release build Oct 25, 2023
@atalman atalman force-pushed the fix_release_docker_build branch from 286bfcf to da8f317 Compare October 25, 2023 17:17
@atalman atalman force-pushed the fix_release_docker_build branch from da8f317 to 3f9118e Compare October 25, 2023 17:17
@atalman atalman changed the title [release only] 1 - Pin Docker images for release. 2 - Fix docker release build [release only] Pin Docker images for release. Oct 25, 2023
@malfet
Copy link
Contributor

malfet commented Oct 25, 2023

IMO we should tag those, otherwise they might be recycled after some period of time

@atalman atalman merged commit ab5b919 into pytorch:release/2.1 Oct 25, 2023
pytorchmergebot pushed a commit that referenced this pull request Nov 2, 2023
This will help prevent a commit like pytorch/builder@7790132 pushing to release branch from overwrite the Docker images used in main.  In addition, the `DEFAULT_TAG` can be easily updated to `2.1` for example when doing branch cut release.  This basically pins the Docker images like #111971

Pull Request resolved: #112692
Approved by: https://github.com/malfet
xuhancn pushed a commit to xuhancn/pytorch that referenced this pull request Nov 7, 2023
This will help prevent a commit like pytorch/builder@7790132 pushing to release branch from overwrite the Docker images used in main.  In addition, the `DEFAULT_TAG` can be easily updated to `2.1` for example when doing branch cut release.  This basically pins the Docker images like pytorch#111971

Pull Request resolved: pytorch#112692
Approved by: https://github.com/malfet
Skylion007 pushed a commit to Skylion007/pytorch that referenced this pull request Nov 14, 2023
This will help prevent a commit like pytorch/builder@7790132 pushing to release branch from overwrite the Docker images used in main.  In addition, the `DEFAULT_TAG` can be easily updated to `2.1` for example when doing branch cut release.  This basically pins the Docker images like pytorch#111971

Pull Request resolved: pytorch#112692
Approved by: https://github.com/malfet
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries_libtorch Trigger binary build and upload jobs for libtorch on the PR ciflow/binaries_wheel Trigger binary build and upload jobs for wheel on the PR topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants