-
Notifications
You must be signed in to change notification settings - Fork 565
tests: Enable AArch64 Jenkins CI with unit tests #1351
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
tests: Enable AArch64 Jenkins CI with unit tests #1351
Conversation
b677051 to
6cf6835
Compare
|
@rbradford I think the Jenkins pipeline can be deployed to the AArch64 node now. I am not very sure though however I can see AArch64 related building and testing logs. However I think some other issues with musl toolchain on arm64 needs to be fixed first. It seems that I can always reproduce this either in the container or in my development machine or the CI machine. Error (only occur when building with musl toolchain): This error happens at the last step, i.e. linking the cloud-hypervisor binary. Root cause see here: https://www.openwall.com/lists/musl/2014/11/05/3 |
bc37920 to
1f38391
Compare
|
Also I noticed that the |
I think this is normal with the SSH launcher. It comes back when the node is needed. |
If it is normal that is very good, but I am worrying a little bit if you do this in your local development machine: I checked the |
|
Can you do something like: https://www.jenkins.io/blog/2019/07/05/jenkins-pipeline-stage-result-visualization-improvements/ So the the Jenkins build doesn't fail. We need the Jenkins builds to be stable as part of our PR merging criteria. |
2fcce41 to
0f9af69
Compare
This commit enables the AArch64 Jenkins CI with build and running unit tests for GNU toolchain. Signed-off-by: Henry Wang <[email protected]>
0f9af69 to
9f20523
Compare
cloud-hypervisor#1225 introduces a hypervisor abstraction crate, which breaks some of the unit test cases on AArch64. This commit fixes related test cases. Signed-off-by: Henry Wang <[email protected]>
9f20523 to
189be82
Compare
|
Hi @rbradford, I have struggled for some weeks to fix the musl toolchain on arm, but sadly none of these methods worked...So in order to let the CI online and let the enablement of integration test unblocked, I will firstly get the GNU toolchain working in this PR and propose following PR for the musl toolchain. rust-lang/rust#73493 created to track this musl issue.
I am not sure if I misunderstood, but I think in this case it seems that this |
Okay with me.
This was just if you wanted to keep the musl rules in there and have them fail but not block the build. |
This commit enables the AArch64 Jenkins CI with build and running unit tests for the GNU toolchain.
Signed-off-by: Henry Wang [email protected]