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

WIP: Build on arm64 too #153

Closed
wants to merge 1 commit into from

Conversation

kernle32dll
Copy link
Contributor

Well, it would be cool to have an arm64 docker image. That could e.g. allow loginsrv to run on an raspberry pi.

This is an WIP PR to get the travis build running.

Well, it would be cool to have an arm64 docker image. That could e.g. allow loginsrv to run on an raspberry pi.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.734% when pulling ce27ca5 on kernle32dll:build-arm into be2ae2c on tarent:master.

@kernle32dll
Copy link
Contributor Author

Hi, @J-Rocke @smancke @gereons

I wanted to kick off discussion if it would be worthwhile to provide an ARM64 Docker image of loginsrv.

At home, we have devices like the Raspberry Pi et all.. In the enterprise segment, we not only start to see cheap ARM based bare-metal servers, but also more and more cloud offerings, e.g. in AWS.

This PR proves that loginsrv builds flawlessly with ARM64. Next step would be to adjust the push pipeline, to build a multi-plattform image.

@g-w
Copy link
Contributor

g-w commented Dec 5, 2019

Hello @kernle32dll,
currently it is out of our scope to support arm64. Beside compiling the binaries for arm64, we have to maintain the software for the architecture and be able to fix bugs. Currently nobody in our team regularly uses golang on arm64 and hence we are not able to support the development.

Feel free to build your own arm64 images and publish them. But we won't.

Kind regards, Gregor

@g-w g-w closed this Dec 5, 2019
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