Skip to content

Commit

Permalink
re-enable ECR push
Browse files Browse the repository at this point in the history
  • Loading branch information
lsy323 committed Oct 9, 2023
1 parent 8a7b8c4 commit 343f669
Showing 1 changed file with 12 additions and 12 deletions.
24 changes: 12 additions & 12 deletions .github/workflows/_build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
type: string
description: Base image for builds
ecr-docker-image-base:
required: false
required: true
type: string
description: Container registry to upload image to
runner:
Expand Down Expand Up @@ -64,17 +64,17 @@ jobs:
- name: Download docker image from GCR
shell: bash
run: docker pull "${GCR_DOCKER_IMAGE}"
# - name: Stage image to ECR
# shell: bash
# run: |
# # This is to stage PyTorch/XLA base image for use in the upstream.
# # To allow the upstream workflow to access PyTorch/XLA build images, we
# # need to have them in the ECR. This is not expensive, and only pushes it
# # if image layers are not present in the repo.
# # Note: disable the following 2 lines while testing a new image, so we do not
# # push to the upstream.
# docker tag "${GCR_DOCKER_IMAGE}" "${ECR_DOCKER_IMAGE_BASE}:v1.1.1-lite-test" >/dev/null
# docker push "${ECR_DOCKER_IMAGE_BASE}:v1.1.1-lite-test" >/dev/null
- name: Stage image to ECR
shell: bash
run: |
# This is to stage PyTorch/XLA base image for use in the upstream.
# To allow the upstream workflow to access PyTorch/XLA build images, we
# need to have them in the ECR. This is not expensive, and only pushes it
# if image layers are not present in the repo.
# Note: disable the following 2 lines while testing a new image, so we do not
# push to the upstream.
docker tag "${GCR_DOCKER_IMAGE}" "${ECR_DOCKER_IMAGE_BASE}:v1.1.2-lite-test" >/dev/null
docker push "${ECR_DOCKER_IMAGE_BASE}:v1.1.2-lite-test" >/dev/null
- name: Start the container
shell: bash
run: |
Expand Down

0 comments on commit 343f669

Please sign in to comment.