Skip to content
This repository was archived by the owner on May 12, 2021. It is now read-only.

travis: use xenial#1500

Merged
chavafg merged 1 commit intokata-containers:masterfrom
chavafg:topic/travis-xenial
Apr 9, 2019
Merged

travis: use xenial#1500
chavafg merged 1 commit intokata-containers:masterfrom
chavafg:topic/travis-xenial

Conversation

@chavafg
Copy link
Copy Markdown
Contributor

@chavafg chavafg commented Apr 8, 2019

We have hit an issue using trusty in travis and go 1.11.7
Using xenial the issue is fixed.

Fixes: #1492.

Signed-off-by: Salvador Fuentes salvador.fuentes@intel.com

@chavafg
travis: use xenial
ea1df84 
We have hit an issue using trusty in travis and go 1.11.7
the issue is fixed using xenial.

Fixes: kata-containers#1492.

Signed-off-by: Salvador Fuentes <salvador.fuentes@intel.com>
@chavafg
Copy link
Copy Markdown
Contributor Author

chavafg commented Apr 8, 2019

/test

grahamwhaley
grahamwhaley approved these changes Apr 9, 2019
View reviewed changes
Copy link
Copy Markdown
Contributor

@grahamwhaley grahamwhaley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm
glad this works for us now.

@grahamwhaley
Copy link
Copy Markdown
Contributor

grahamwhaley commented Apr 9, 2019

Well, travis worked, but we had some other CI fails.
Now, this change should only effect travis, so I expect those are unrelated. Let's have a peek at them here, and then I'll probably merge this anyway...

fedora-vsocks

Summarizing 1 Failure:

[Fail] state container [It] with workload [true], timeWait 5 
/tmp/jenkins/workspace/kata-containers-runtime-fedora-vsocks-PR/go/src/github.com/kata-containers/tests/functional/state_test.go:45

Ran 8 of 8 Specs in 17.222 seconds
FAIL! -- 7 Passed | 1 Failed | 0 Pending | 0 Skipped --- FAIL: TestFunctional (17.22s)
FAIL

bunty 18

[1] Summarizing 1 Failure:
[1] 
[1] [Fail] Update CPU constraints Update CPU set [It] cpuset should be equal to 2 
[1] /tmp/jenkins/workspace/kata-containers-runtime-ubuntu-18-04-PR/go/src/github.com/kata-containers/tests/integration/docker/cpu_test.go:395
[1] 
[1] Ran 11 of 199 Specs in 208.739 seconds
[1] FAIL! -- 10 Passed | 1 Failed | 0 Pending | 188 Skipped --- FAIL: TestIntegration (208.75s)
[1] FAIL

bunty 18 initrd

+ test_create_container_with_vm_template
+ sleep 2
+ sudo docker run --runtime=kata-runtime -d --name kata-G1yi1C busybox tail -f /dev/null
b540e8aeaaf44046cb407bc21e81fddf1aed0b13f326d5d902f517c40e4e49a7
docker: Error response from daemon: OCI runtime create failed: rpc error: code = Unavailable desc = transport is closing: unknown.
Makefile:123: recipe for target 'vm-factory' failed
make: *** [vm-factory] Error 125
Build step 'Execute shell' marked build as failure

@grahamwhaley
Copy link
Copy Markdown
Contributor

grahamwhaley commented Apr 9, 2019

hmm, force merge doesn't seem to be available for me here. I've nudged a rebuild of bunty-18, as that is the 'required' CI that is blocking us...

@bergwolf bergwolf mentioned this pull request Apr 9, 2019
Closed
@chavafg chavafg merged commit 23c00ff into kata-containers:master Apr 9, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

3 more reviewers

@grahamwhaley grahamwhaley grahamwhaley approved these changes

@devimc devimc devimc approved these changes

@GabyCT GabyCT GabyCT approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

continuous-integration/travis-ci/pr always fail in x86

4 participants

@chavafg @grahamwhaley @devimc @GabyCT