Skip to content
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

Add Additional Delivery to Google Artifact Repo #3004

Merged
merged 4 commits into from
Nov 12, 2024

Conversation

corbanCloud
Copy link
Contributor

@corbanCloud corbanCloud commented Nov 6, 2024

Adds GAR (Google Artifact Repository) as a second delivery source for GLAM images

Functionality was test in a temporary branch, which produced this artifact

The pipeline steps here are intentionally not DRY'd up between the existing GCR delivery as those steps can simply be deleted after V2 Migration is complete. Additionally GAR delivery was added for both the main glam app container and the auth0 proxy which creates some additional cruft during the transition

@corbanCloud corbanCloud marked this pull request as ready for review November 8, 2024 00:51
@corbanCloud corbanCloud changed the title stub out changes for Dockerhub -> GAR migration Enable Parallel Delivery to Google Artifact Repo Nov 12, 2024
@corbanCloud corbanCloud changed the title Enable Parallel Delivery to Google Artifact Repo Add Additional Delivery to Google Artifact Repo Nov 12, 2024
Copy link
Member

@whd whd left a comment

Choose a reason for hiding this comment

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

This looks like it won't work to me for the ancillary images but I might be reading it wrong. As long as images get pushed to all the right places 👍

.circleci/config.yml Outdated Show resolved Hide resolved
# this could be done by re-tagging the existing image but for ease of migration
# I'm duplicating it fully with the intent of removing the above steps entirely
# /moz-fx-glam-prod/glam-prod/glam@sha256:705f53c5511c5803c6ea58896a93a97a30dc8defdef19c5d2243c3171a13be25
- run:
Copy link
Member

Choose a reason for hiding this comment

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

Do you need to do this if you're just using the gcp-gcr orb? You're not actually using the GCP CLI after you set it up. It seems like you should be able to get away with a simpler config than this that's just gcp-gcr/build-and-push-image with the right environment variables set and with use_oidc set to true. But if this works then that's fine.

image: auth0-proxy
tag: $CIRCLE_SHA1

- gcp-gcr/push-image:
Copy link
Member

Choose a reason for hiding this comment

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

Has this been tested yet? I don't see any images in https://console.cloud.google.com/artifacts/docker/moz-fx-glam-prod/us/glam-prod?project=moz-fx-glam-prod other than the glam image.

Co-authored-by: whd <whd@users.noreply.github.com>
@corbanCloud corbanCloud merged commit 9a3a198 into main Nov 12, 2024
4 checks passed
@corbanCloud corbanCloud deleted the migrate-to-v2-use-google-app-repo branch November 12, 2024 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants