Skip to content
This repository has been archived by the owner on May 6, 2020. It is now read-only.

update virtcontainers #829

Closed
devimc opened this issue Nov 24, 2017 · 0 comments · Fixed by #830
Closed

update virtcontainers #829

devimc opened this issue Nov 24, 2017 · 0 comments · Fixed by #830
Assignees

Comments

@devimc
Copy link

devimc commented Nov 24, 2017

update virtcontainers to have support for hotplugging in Qemu Q35

@devimc devimc self-assigned this Nov 24, 2017
devimc pushed a commit to devimc/runtime that referenced this issue Nov 24, 2017
this new version of virtcontainers has support for hotplugging
in Qemu Q35

shortlog:
aa2ba14 Remove redundant "if" check
37dc3c6 cni: Assign the netpair for virtual endpoints after scan
8f846a2 cni: Fix bug while updating endpoints after scanning net namespace
eb39def qemu: Provide unique QMP sockets names
6026603 qemu: Don't wait for the end of the VM
eeacf8b qemu: Fix disconnectCh issue and simplify it
711783f agent: hyperstart: Tear down properly the pod
b20ba87 pod: Improve pod create performances
d449ece api: Refactor RunPod()
c3663b2 pod: Run post-start hooks outside netns
42df5a6 hypervisor: add hot plugging support for Qemu Q35

fixes clearcontainers#829

Signed-off-by: Julio Montes <[email protected]>
mcastelino pushed a commit to mcastelino/runtime that referenced this issue Dec 6, 2018
Got "docker: Error response from daemon: OCI runtime create failed:
QMP command failed: unknown." when "docker run --privileged" with kata.
In qemu part, it got:
"Could not open '/dev/sr0': Read-only file system"
or
"No medium found"
The cause is qemu need open block device to get its status.
But /dev/sr0 is a CDROM that cannot be opened.

This patch let newContainer doesn't attach device if it is a CDROM
to handle the issue.

Fixes clearcontainers#829

Signed-off-by: Hui Zhu <[email protected]>
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 a pull request may close this issue.

2 participants