-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Support arm64 architecture for tests. #525
Conversation
|
||
services: | ||
- docker | ||
language: minimal |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Travis assumes Ruby by default and thus flagged builds for installing rvm and a default ruby. Using minimal
reduces that overhead while still providing basic utilities like Docker out of the box.
@kylemanna my only failing tests are the following:
I'm really not sure why it would have a problem with TTY? Could there be something missing in the base arm image from travis? |
I am also seeing this for the
|
8fdba7c
to
306d467
Compare
That OTP test hanging is due to a bug in the EasyRSA lib. I am going to add the fix into my PR here as a discrete commit. Some other PR's here have the temporary workaround as well. Ultimately it'll JustWork™️ once |
Are there any news regarding this issue? |
I would still like this to be completed but I ran into issues getting these tests to pass. The actual arm64 builds work just fine. I need to circle back on this and see if the underlying issues are now fixed. |
Hi folks, any update on this? |
Unfortunately no. I decided to move to WireGuard as the existing containers for it worked out-of-the-box and supported ARM builds better. There just doesn't seem to be core interest in supporting ARM for this project (not a complaint, just a statement of observation). It would be good to leave this open for someone else to pick up. |
How about this pr? I want use this image on arm. |
I have successfully built and tested the image for both amd64 and arm64 architecture and also pushed it through Travis-ci on dockerhub. Commit Link - pheobe07@b67f766 Travis Link - https://travis-ci.com/github/pheobe07/docker-openvpn/builds/209627748 Dockerhub Link - https://hub.docker.com/repository/docker/odidev/openvpn I have used arm64-graviton which is added for travis-ci.com as per https://blog.travis-ci.com/2020-09-11-arm-on-aws. So to use arm64-graviton2 in travis we need to migrate this from travis-ci.org to travis-ci.com. Please share your thoughts on this and it will be really helpful if arm64 image is released for openvpn. |
Any progress. Still its not merged. Can you please provide docker image for arm64 directly on docker hub. Thanks. |
Can you please have a look at my changes? If the changes looks good I can raise a separate PR for releasing image for arm64. It will be really helpful if arm64 image is released. May I know when are you planning on releasing the images? |
I have a couple failing tests on arm64. I am looking into them but overall this seems to be working great and should allow official builds for arm.
Supersedes pull requests:
This will probably close: