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

feat(build): add arm builds to travis #296

Merged
merged 1 commit into from
Apr 2, 2020

Conversation

kmova
Copy link

@kmova kmova commented Mar 31, 2020

Ref: openebs/openebs#1295

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

@kmova kmova added the pr/hold-review PR currently under development label Mar 31, 2020
@kmova kmova requested review from pawanpraka1 and mynktl March 31, 2020 19:10
@kmova kmova added pr/release-note-alpha and removed pr/hold-review PR currently under development labels Mar 31, 2020
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>
Copy link
Member

@mynktl mynktl left a 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.

@kmova
Copy link
Author

kmova commented Apr 2, 2020

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.

Copy link
Member

@pawanpraka1 pawanpraka1 left a 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.

@mynktl
Copy link
Member

mynktl commented Apr 2, 2020

I've created an issue openebs/openebs#2984 to track the pending changes.

@mynktl mynktl merged commit 2cb54f2 into mayadata-io:develop Apr 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants