From 82f3bfa803d674c6ff61166e676ed82a17f0c70d Mon Sep 17 00:00:00 2001 From: anthony sottile Date: Thu, 18 Apr 2024 16:24:19 -0400 Subject: [PATCH] ref: dual-write docker image to artifact registry as part of OPS-3966 we're migrating off of deprecated gcr --- .github/workflows/ci.yml | 4 ++-- cloudbuild.yaml | 6 +++++- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 401fe11bdc..29b45b2d89 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -434,7 +434,7 @@ jobs: - name: Pull the test image id: image_pull env: - IMAGE_URL: us.gcr.io/sentryio/snuba:${{ github.sha }} + IMAGE_URL: us-central1-docker.pkg.dev/sentryio/snuba/image:${{ github.sha }} shell: bash run: | echo "We poll for the Docker image that the GCB/GHA build produces until it succeeds or this job times out." @@ -455,7 +455,7 @@ jobs: shell: bash env: SHORT_SHA: ${{ steps.short_sha.outputs.sha }} - IMAGE_URL: us.gcr.io/sentryio/snuba:${{ github.sha }} + IMAGE_URL: us-central1-docker.pkg.dev/sentryio/snuba/image:${{ github.sha }} run: | # only login if the password is set if [[ "${{ secrets.DOCKER_HUB_RW_TOKEN }}" ]]; then echo "${{ secrets.DOCKER_HUB_RW_TOKEN }}" | docker login --username=sentrybuilder --password-stdin; fi diff --git a/cloudbuild.yaml b/cloudbuild.yaml index 47180d2df2..dc30d31bbb 100644 --- a/cloudbuild.yaml +++ b/cloudbuild.yaml @@ -9,6 +9,7 @@ steps: "--build-arg", "SOURCE_COMMIT=$COMMIT_SHA", "--destination=us.gcr.io/$PROJECT_ID/$REPO_NAME:$COMMIT_SHA", + "--destination=us-central1-docker.pkg.dev/$PROJECT_ID/snuba/image:$COMMIT_SHA", "--target=application", "-f", "./Dockerfile", @@ -23,7 +24,10 @@ steps: # This is needed for Freight to find matching builds -images: ['us.gcr.io/$PROJECT_ID/$REPO_NAME:$COMMIT_SHA'] +images: [ + 'us.gcr.io/$PROJECT_ID/$REPO_NAME:$COMMIT_SHA', + 'us-central1-docker.pkg.dev/$PROJECT_ID/snuba/image:$COMMIT_SHA', +] timeout: 2640s options: # We need more memory for Webpack builds & e2e self-hosted tests