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.

Sign up for GitHub

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

Jump to bottom

v0.0.2 and v0.0.3 image tags only have arm64 arch. #35

Open
LS80 opened this issue Nov 13, 2023 · 5 comments · May be fixed by #36
Open

v0.0.2 and v0.0.3 image tags only have arm64 arch. #35

LS80 opened this issue Nov 13, 2023 · 5 comments · May be fixed by #36
Assignees
@ChrisHirsch
Labels
request requests that don't fall under something else

Comments

@LS80
Copy link
Contributor

LS80 commented Nov 13, 2023

It looks like the v0.0.2 image tag was re-published but only for arm64. There is no support for the amd64 arch. This results in exec /app/storage-check: exec format error if the checker pod is scheduled to an amd64 node.
@github-actions GitHub Actions
Copy link

👋 Welcome to Kuberhealthy Storage Check. Thanks for opening your first issue.

@ChrisHirsch ChrisHirsch self-assigned this Nov 13, 2023
@ChrisHirsch
Copy link
Owner

Thank you for bringing this to my attention. This should now be fixed and I'll be bringing in a more proper pipeline to ensure this doesn't happen in the future.

@LS80
Copy link
Contributor Author

LS80 commented Nov 13, 2023

The images now only have the amd64 architecture. Could you publish multi-arch images that have support for both arm64 and amd64?

@ChrisHirsch
Copy link
Owner

Hi @LS80 I've pushed 0.0.2 and 0.0.3 with the amd64 and arm64 images. Please let me know if this works for you.

@ChrisHirsch ChrisHirsch reopened this Nov 16, 2023
ChrisHirsch pushed a commit that referenced this issue Nov 16, 2023
ci: 🎡 Refactor the makefile for multiarch docker build & push
2f35cb7 
✅ Closes: #35
@ChrisHirsch ChrisHirsch linked a pull request Nov 16, 2023 that will close this issue
Open
@LS80
Copy link
Contributor Author

LS80 commented Nov 16, 2023

The manifest looks strange. There are two extra images that have unknown architecture. It does appear to work though.

docker manifest inspect chrishirsch/kuberhealthy-storage-check:v0.0.3

{
   "schemaVersion": 2,
   "mediaType": "application/vnd.oci.image.index.v1+json",
   "manifests": [
      {
         "mediaType": "application/vnd.oci.image.manifest.v1+json",
         "size": 861,
         "digest": "sha256:82eb0c2487ba450bbccb57b0358581d296c8c812c69d2bfb536f27450ff992bd",
         "platform": {
            "architecture": "amd64",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.oci.image.manifest.v1+json",
         "size": 861,
         "digest": "sha256:9656453f07ccdf74742848b6ad183b871212a03708e7ceb513955911dafe04a6",
         "platform": {
            "architecture": "arm64",
            "os": "linux"
         }
      },
      {
         "mediaType": "application/vnd.oci.image.manifest.v1+json",
         "size": 566,
         "digest": "sha256:8b04a1da4e21292d07f3a14c9741634a42d8da7c30da8a80d5d386985acb065a",
         "platform": {
            "architecture": "unknown",
            "os": "unknown"
         }
      },
      {
         "mediaType": "application/vnd.oci.image.manifest.v1+json",
         "size": 566,
         "digest": "sha256:db3b29b3fd4c13a887ebafabb3d2e73f03cbfd692b51ff3897b9d5fb7d7474af",
         "platform": {
            "architecture": "unknown",
            "os": "unknown"
         }
      }
   ]
}

@mgoodhart5 mgoodhart5 added the request requests that don't fall under something else label Jan 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ChrisHirsch ChrisHirsch

Labels
request requests that don't fall under something else
Projects
None yet
Milestone
No milestone
3 participants
@ChrisHirsch @LS80 @mgoodhart5