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

Support arm64 architecture for tests. #525

Closed
wants to merge 1 commit into from

Conversation

pho3nixf1re
Copy link

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:


services:
- docker
language: minimal
Copy link
Author

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.

@pho3nixf1re
Copy link
Author

@kylemanna my only failing tests are the following:

failed
	'dual-proto' [5/8]...the input device is not a TTY
failed
	'otp' [6/8]...the input device is not a TTY
failed
	'iptables' [7/8]...the input device is not a TTY
failed
	'revocation' [8/8]...the input device is not a TTY
failed
The command "test/run.sh "$image"" exited with 1.

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?

@pho3nixf1re
Copy link
Author

I am also seeing this for the amd64 builds:

passed
	'otp' [6/8]...
No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.
Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
The build has been terminated

@pho3nixf1re
Copy link
Author

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 alpine is upgraded.

@maurerle
Copy link

Are there any news regarding this issue?

@pho3nixf1re
Copy link
Author

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.

@juchong
Copy link

juchong commented Sep 23, 2020

Hi folks, any update on this?

@pho3nixf1re
Copy link
Author

pho3nixf1re commented Sep 23, 2020

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.

@yywing
Copy link

yywing commented Nov 26, 2020

How about this pr? I want use this image on arm.

@odidev
Copy link

odidev commented Dec 18, 2020

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.

@hppyworld
Copy link

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.

@odidev
Copy link

odidev commented Nov 1, 2021

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?

@pho3nixf1re pho3nixf1re closed this Jul 9, 2023
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.

None yet

6 participants