Skip to content
This repository has been archived by the owner on Mar 22, 2021. It is now read-only.

testrunner: pass through /dev/kvm to containers #88

Closed
wants to merge 1 commit into from

Conversation

jlebon
Copy link
Collaborator

@jlebon jlebon commented Feb 23, 2018

For reasons I can't quite figure out, oci-kvm-hook doesn't work on F25
(though it does on F27). The nodes are currently stuck at F25 because of
severe issues with docker cp in newer version (rhbz#1489505). So let's
just explicitly pass /dev/kvm into the container, which is all
oci-kvm-hook is trying to do anyway. I've verified this works.

This will unblock ostreedev/ostree#1462.

For reasons I can't quite figure out, `oci-kvm-hook` doesn't work on F25
(though it does on F27). The nodes are currently stuck at F25 because of
severe issues with `docker cp` in newer version (rhbz#1489505). So let's
just explicitly pass /dev/kvm into the container, which is all
`oci-kvm-hook` is trying to do anyway. I've verified this works.

This will unblock ostreedev/ostree#1462.
@cgwalters
Copy link
Member

@rh-atomic-bot r+ 9fbdf13

@rh-atomic-bot
Copy link

⌛ Testing commit 9fbdf13 with merge 3617c4b...

rh-atomic-bot pushed a commit that referenced this pull request Feb 23, 2018
For reasons I can't quite figure out, `oci-kvm-hook` doesn't work on F25
(though it does on F27). The nodes are currently stuck at F25 because of
severe issues with `docker cp` in newer version (rhbz#1489505). So let's
just explicitly pass /dev/kvm into the container, which is all
`oci-kvm-hook` is trying to do anyway. I've verified this works.

This will unblock ostreedev/ostree#1462.

Closes: #88
Approved by: cgwalters
@rh-atomic-bot
Copy link

💔 Test failed - status-papr

@jlebon
Copy link
Collaborator Author

jlebon commented Feb 23, 2018

Ahh right. This is fixed in #87. Let's get that one in first?

@jlebon
Copy link
Collaborator Author

jlebon commented Feb 23, 2018

@rh-atomic-bot retry

@rh-atomic-bot
Copy link

⌛ Testing commit 9fbdf13 with merge d8d93a5...

@rh-atomic-bot
Copy link

☀️ Test successful - status-papr
Approved by: cgwalters
Pushing d8d93a5 to master...

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.

3 participants