-
Notifications
You must be signed in to change notification settings - Fork 30
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
feat(build): add arm builds to travis #296
Conversation
Refactored the travis yaml to automate the arm builds. The major changes include: - introducing new env variables for running ztest and test_uzfs - the arm build vm on travis doesn't have permissions to run in sudo. Hence skipping the tests for now. - installing test packages only on amd - hack for installing the right linux headers based on the cpu arch - pushing the code coverage only when enabled Signed-off-by: kmova <kiran.mova@mayadata.io>
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.
We need to verify ARM binaries for each release. ARM containers are not having privileged access so test_dup_zrepl
won't work but we can have some check to disable that test execution. But other test we need to verify, may be through travis or e2e.
Sounds good @mynktl - can we go ahead with this PR to get the images generated in alpha. and possibly raise another PR to start enabling the tests that can work. |
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.
looks good. We probably need to make changes in maya repo also to use cstor-base-arm64 image as base image, so that we can test it.
I've created an issue openebs/openebs#2984 to track the pending changes. |
Ref: openebs/openebs#1295
Refactored the travis yaml to automate the arm builds.
The major changes include:
Signed-off-by: kmova kiran.mova@mayadata.io