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

fix: minor fixes for GAR deploy CircleCI jobs #764

Merged
merged 9 commits into from
Sep 5, 2024
Merged

Conversation

emaydeck-mozilla
Copy link
Contributor

This now mounts the workspace so we can load the built image from that. Also, I fixed the image name in the gcp-gcr/push-image step.

This was tested successfully in https://app.circleci.com/pipelines/github/mozilla-services/autopush-rs/2873/workflows/9adebe2f-13e8-4e5d-a853-e78b7f6ca368

@emaydeck-mozilla emaydeck-mozilla self-assigned this Sep 4, 2024
@emaydeck-mozilla emaydeck-mozilla changed the title fix: fix GAR deploy CircleCI jobs fix: minor fixes for GAR deploy CircleCI jobs Sep 4, 2024
Copy link
Contributor

@taddes taddes left a comment

Choose a reason for hiding this comment

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

👍 Looks good! Just had a question about Docker image caching.

.circleci/config.yml Show resolved Hide resolved
@emaydeck-mozilla emaydeck-mozilla merged commit e0d8766 into master Sep 5, 2024
1 check passed
@emaydeck-mozilla emaydeck-mozilla deleted the RRM-181 branch September 5, 2024 17:31
pjenvey pushed a commit that referenced this pull request Sep 5, 2024
This now mounts the workspace so we can load the built image from that.
Also, I fixed the image name in the `gcp-gcr/push-image` step.

This was tested successfully in
https://app.circleci.com/pipelines/github/mozilla-services/autopush-rs/2873/workflows/9adebe2f-13e8-4e5d-a853-e78b7f6ca368
emaydeck-mozilla added a commit that referenced this pull request Sep 5, 2024
This now mounts the workspace so we can load the built image from that.
Also, I fixed the image name in the `gcp-gcr/push-image` step.

This was tested successfully in
https://app.circleci.com/pipelines/github/mozilla-services/autopush-rs/2873/workflows/9adebe2f-13e8-4e5d-a853-e78b7f6ca368
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.

3 participants