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

Upload container images for arm64 architecture #1851

Closed
der-eismann opened this issue Dec 9, 2019 · 34 comments · Fixed by #1906
Closed

Upload container images for arm64 architecture #1851

der-eismann opened this issue Dec 9, 2019 · 34 comments · Fixed by #1906

Comments

@der-eismann
Copy link

Now that AWS announced their way more powerful M6g instances it will start to get interesting to move from x86 to the ARM architecture. You already provide binaries on the releases page, but the docker images are for x86 only. Could you maybe adjust your build pipeline and provide images for ARM like prometheus does?

@bwplotka
Copy link
Member

bwplotka commented Dec 9, 2019

We can definitely consider this (:

Help wanted to set it up.

@der-eismann
Copy link
Author

I'd be happy to help, but unfortunately it seems like CircleCI doesn't support it (yet). There are some hacky workarounds, but I'm not sure if this is the correct way.

So actually I have no idea how to do this right now.

@bwplotka
Copy link
Member

I don't think we need any CI special feature for this:

Let's instead use the way Prometheus is doing this, so:
Docker build:

And Dockerfile taking binaries from promu (we already use promu for the build process):

(:

@stale
Copy link

stale bot commented Jan 17, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jan 17, 2020
@stale stale bot closed this as completed Jan 24, 2020
@bwplotka
Copy link
Member

Still valid, and in fact we have a PR ready - it's waiting on us to actually test it and update quay a bit.

@bwplotka bwplotka reopened this Jan 24, 2020
@stale stale bot removed the stale label Jan 24, 2020
@bwplotka bwplotka added pinned and removed pinned labels Jan 24, 2020
@stale
Copy link

stale bot commented Feb 27, 2020

This issue/PR has been automatically marked as stale because it has not had recent activity. Please comment on status otherwise the issue will be closed in a week. Thank you for your contributions.

@stale stale bot added the stale label Feb 27, 2020
@stale stale bot closed this as completed Mar 5, 2020
@zosorock
Copy link

I am also looking into starting to test with m6g instances. Are there any prediction of when it will be available?

@daixiang0 daixiang0 reopened this Jul 14, 2020
@stale stale bot removed the stale label Jul 14, 2020
@stale
Copy link

stale bot commented Aug 13, 2020

Hello 👋 Looks like there was no activity on this issue for last 30 days.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity for next week, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Aug 13, 2020
@stale
Copy link

stale bot commented Aug 20, 2020

Closing for now as promised, let us know if you need this to be reopened! 🤗

@stale stale bot closed this as completed Aug 20, 2020
@autarchprinceps
Copy link

Is this still being worked on? I just tested and the thanos-sidecar in Prometheus is one of only a few containers that still make issues.

@bwplotka bwplotka reopened this Nov 16, 2020
@stale stale bot removed the stale label Nov 16, 2020
@bwplotka
Copy link
Member

We had contributions to add the build/CI process. but I think there is some process of configuration on quay/dockerhub. To test and finish up.

cc @daixiang0

@daixiang0
Copy link
Member

While I have no permission to test it, please help @bwplotka !

@stale stale bot added the stale label Mar 16, 2021
@commixon
Copy link

Hello people! Is this still a valid thing? We would really like this as well (and are willing to help with anything as well)

@stale stale bot removed the stale label Mar 26, 2021
@bwplotka
Copy link
Member

Looks like it's getting quite a popular feature (: We already build such binaries - so you can easily build (make crossbuild) and upload to your own registry. Ideally we have someone who can configure docker registry for us 🙃 which is unlikely. For some reasons @daixiang0 had issues to access things from China I believe.

We will try to prioritize this on maintainer team (: Thanks!

@der-eismann
Copy link
Author

FYI this is how we did it with GitHub actions, works fine for us so far:
https://github.com/rebuy-de/aws-nuke/blob/main/.github/workflows/release.yaml#L48-L78
Basically you need QEMU & buildx for docker, then it's possible to build for all arches.

@roidelapluie
Copy link

roidelapluie commented Mar 27, 2021

I don't think thanos requires this, it could use standard go crossbuild. I am happy to see how we can share promu with thanos, but generic tools like goreleaser would work too.

@stale
Copy link

stale bot commented Jun 2, 2021

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Jun 2, 2021
@der-eismann
Copy link
Author

der-eismann commented Jun 2, 2021

Still needed.

@stale stale bot removed the stale label Jun 2, 2021
@stale
Copy link

stale bot commented Aug 3, 2021

Hello 👋 Looks like there was no activity on this issue for the last two months.
Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗
If there will be no activity in the next two weeks, this issue will be closed (we can always reopen an issue if we need!). Alternatively, use remind command if you wish to be reminded at some point in future.

@stale stale bot added the stale label Aug 3, 2021
@autarchprinceps
Copy link

Still needed.

@der-eismann
Copy link
Author

Remove the stale label please

@stale stale bot removed the stale label Aug 4, 2021
@hanzZzZzZ
Copy link

Just found this issue, but I can confirm:
Still needed.

@ismashal
Copy link

ismashal commented Oct 4, 2021

We really needed this ARM64 supported official image into the docker hub and quay.io/thanos/thanos. As AWS already started the support graviton2 type images into the EKS.
Thanks!

@avivgold098
Copy link

We will be very helpful:
Still needed. :)

@WeiTangLau
Copy link

2 years later. Still needed

@firefixmaarten
Copy link

firefixmaarten commented Dec 2, 2021

Still wanted :) +1, in my case armv7.

@GiedriusS
Copy link
Member

After #4947 is done, we will finally have this ready. @firefixmaarten could you please open a separate ticket for armv7?

@GiedriusS
Copy link
Member

GiedriusS commented Dec 14, 2021

https://quay.io/repository/thanos/thanos/manifest/sha256:ba0b9192a0db3693afcb220c4c1fc8578a1c25df20dc41697e57b3329fe1d7f8
https://hub.docker.com/layers/thanosio/thanos/main-2021-12-14-7793ab63/images/sha256-9dd0cee98ac7dc4f122e75d4fade9ba2452e8a9a2c0b8cb9a160e55ce1bae0f1?context=explore
We finally have arm64 images as well 🎉 everything should be transparent to the users. Please test them out.

@WeiTangLau
Copy link

WeiTangLau commented Dec 23, 2021

Hi @GiedriusS, given the latest release of v0.24.0 and v0.23.2 of thanos, the arm64 images are not built accordingly. Why is this so?

https://hub.docker.com/layers/thanosio/thanos/v0.24.0/images/sha256-b8f527c6750d76c2bd637c7047e95bfc3b4e42a2be454b2efef00cc3d9a55a15?context=explore

https://hub.docker.com/layers/thanosio/thanos/v0.23.2/images/sha256-1936fe5c5829756d5c94af8c9a497ae7f4ce107e7ff5c4876cb26016aa857cd3?context=explore

@jan-di
Copy link

jan-di commented Feb 6, 2022

@WeiTangLau Not a maintainer here, but i guess thats because the release candidates of those versions pre date the merge of this feature. See https://github.com/thanos-io/thanos/commits/v0.24.0. I think the next release (0.25) will have arm64 images.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.