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

CI: add main tag to Docker image #519

Merged
merged 1 commit into from
Jan 8, 2025
Merged

CI: add main tag to Docker image #519

merged 1 commit into from
Jan 8, 2025

Conversation

lalver1
Copy link
Member

@lalver1 lalver1 commented Jan 8, 2025

Currently, Apple M2 users see a no matching manifest for linux/arm64/v8 in the manifest list entries message when trying to pull the eligibility-server image from this repository's associated GHCR and the pull fails.

This PR publishes a linux/arm64/v8 Docker image so that Apple M2 users can directly pull the eligibility-server image.

I tested pulling

ghcr.io/cal-itp/eligibility-server:4e1d6c09239ceac913243c61f230fe1c045e2939@sha256:8335d87361c823ccc9eea7a92c28da3fed92c730c4dd33c3a0dbdf27b70f6264

using an Apple M2, and the pull succeeded. So, the issue is actually due to the Docker image missing the main tag. This PR adds a main tag to the Docker image so that it can be pulled directly from the Benefits application (see cal-itp/benefits#2616).

@lalver1 lalver1 self-assigned this Jan 8, 2025
Copy link

github-actions bot commented Jan 8, 2025

Coverage report

This PR does not seem to contain any modification to coverable code.

@lalver1 lalver1 force-pushed the ci/multiplatform-update branch from fb754cb to c0702a4 Compare January 8, 2025 16:22
@thekaveman
Copy link
Member

Can you please check out cal-itp/benefits#2616?

Can we add back the main tag too? That is part of the problem.

@lalver1 lalver1 force-pushed the ci/multiplatform-update branch from c0702a4 to 0ad958a Compare January 8, 2025 16:37
@lalver1 lalver1 changed the title CI: add Docker image for Apple M2 CI: add main tag to Docker image Jan 8, 2025
@lalver1
Copy link
Member Author

lalver1 commented Jan 8, 2025

I tested using an Apple M2 and I'm pretty sure that's the actual problem @thekaveman, thanks for bringing it up! I updated the PR's description to make this more clear.

@lalver1 lalver1 marked this pull request as ready for review January 8, 2025 16:57
@lalver1 lalver1 requested a review from a team as a code owner January 8, 2025 16:57
@lalver1 lalver1 force-pushed the ci/multiplatform-update branch from 0ad958a to 1cfe739 Compare January 8, 2025 17:20
@lalver1 lalver1 merged commit a1b076b into main Jan 8, 2025
6 checks passed
@lalver1 lalver1 deleted the ci/multiplatform-update branch January 8, 2025 17:21
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