flannel icon indicating copy to clipboard operation
flannel copied to clipboard

Tests failing on ARM64 platform

Open odidev opened this issue 2 years ago • 9 comments

Hi Team, I have been working on running and testing Flannel on arm64 platform but the make test command fails on arm64 platform with the following error:

etcd cluster is unavailable or misconfigured

Since the make test command works fine on amd64 platform I changed etcd version in functional-test.sh to latest 3.5.1 which gave the following error:

Couldn't fetch network config: client: response is invalid json. The endpoint is probably not valid etcd cluster endpoint

I followed  recommendations given here but following the recommendations makes the test command stuck in infinite loop. If you would be able to give some pointers on how to resolve the issue on arm64 platform it would be very helpful.

Are you interested in enabling testing on arm64 platform in GitHub Actions?

odidev avatar Dec 28 '21 08:12 odidev

Hi Team, A gentle reminder to take a look at above issue and provide some pointers for resolving the issue on arm64 platform.

odidev avatar Jan 12 '22 11:01 odidev

@odidev hi we're taking a look

luthermonson avatar Feb 17 '22 17:02 luthermonson

Hi @luthermonson, Any updates on above issue?

odidev avatar Mar 25 '22 10:03 odidev

Sorry, was quite busy and forgot to send MR =( Since whole build process was done with assumption it will be running on x86 platform to compile and test other platform binaries (using statically compiled qemu binary) there is no good way of fixing it without full build system redesign. The commit above solves problem with builds on ARM64 (and probably other architectures like arm,ppc64le, s390x, and mips64le) using explicit definition of ARCH env variable for make command (like ARCH=arm64 make test). There is no good way of building cross-arch binaries on the other than x86 platform because that functionality is limited by using statically compiled qemu which is usually compiled for x86 only but in this case all the permutations of all the ARCHs are required. @odidev Please confirm it works for you.

AleksandrNull avatar Mar 27 '22 19:03 AleksandrNull

Hi @AleksandrNull,

Thanks for your help, the issue is fixed. The tests were passing after some modifications in test files but now the ipsec and vxlan tests are failing with same error as here.

Would you be interested in adding ARM64 tests in GitHub Actions using self-hosted arm64 runners. If interested, Please have a look at  worksonarm, can get access to ARM H/W through this.

odidev avatar Apr 27 '22 16:04 odidev

Hi @luthermonson @AleksandrNull , Any updates on the above issue?

odidev avatar May 24 '22 05:05 odidev

Thanks for your help, the issue is fixed as the etcd version has been upgraded to 3. The tests were passing after some modifications that can be seen in the PR.

PR Link:- https://github.com/flannel-io/flannel/pull/1626

Would you be interested in adding ARM64 tests in GitHub Actions using self-hosted arm64 runners. If interested, Please have a look at  worksonarm, can get access to ARM H/W through this.

odidev avatar Aug 03 '22 15:08 odidev

Hi @luthermonson @AleksandrNull , Kindly take a look on above query.

odidev avatar Aug 17 '22 08:08 odidev

Hi @luthermonson @AleksandrNull , Kindly take a look on above query.

odidev avatar Aug 31 '22 11:08 odidev

Hi @luthermonson @AleksandrNull , Gentle reminder to take a look on above query.

odidev avatar Dec 12 '22 11:12 odidev

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[bot] avatar Jun 10 '23 14:06 stale[bot]