Skip to content
This repository has been archived by the owner on Jan 20, 2022. It is now read-only.

Run tests for the s390x build #79

Merged
merged 1 commit into from
Dec 3, 2018

Conversation

markdryan
Copy link
Contributor

It turns out it is possible to run the unit tests for the s390x build
on travis by renaming the s390x specific files, so that their
inclusion in the build is determined only by tags and not by filename,
and by introducing a new tag s390x_test that we can use to force
their inclusion into a build by using this tag. The .travis file is
then updated to include the line

go test --tags s390x_test ./...

This creates a build on travis that includes the s390x specific
files and runs the unit tests.

Signed-off-by: Mark Ryan mark.d.ryan@intel.com

It turns out it is possible to run the unit tests for the s390x build
on travis by renaming the s390x specific files, so that their
inclusion in the build is determined only by tags and not by filename,
and by introducing a new tag s390x_test that we can use to force
their inclusion into a build by using this tag.  The .travis file is
then updated to include the line

go test --tags s390x_test ./...

This creates a build on travis that includes the s390x specific
files and runs the unit tests.

Signed-off-by: Mark Ryan <mark.d.ryan@intel.com>
@markdryan
Copy link
Contributor Author

@alicefr Could you test this on a s390x and make sure that govmm still works on that architecture with this patch applied?

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 80.335% when pulling d617307 on markdryan:s390x-tests into 09923e8 on intel:master.

@alicefr
Copy link
Member

alicefr commented Dec 3, 2018

@markdryan yup! It worked!

@markdryan markdryan merged commit d31bc8d into kata-containers:master Dec 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants