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

qemu: clean up qmp channel#429

Merged
sboeuf merged 1 commit intokata-containers:masterfrom
bergwolf:qmp
Jun 20, 2018
Merged

qemu: clean up qmp channel#429
sboeuf merged 1 commit intokata-containers:masterfrom
bergwolf:qmp

Conversation

@bergwolf
Copy link
Member

We only need one qmp channel and it is qemu internal detail thus
sandbox.go does not need to be aware of it.

This was part of #303 and but does not really relate to vm factory so I take it out.

@katacontainersbot
Copy link
Contributor

PSS Measurement:
Qemu: 144175 KB
Proxy: 4681 KB
Shim: 8920 KB

Memory inside container:
Total Memory: 2045972 KB
Free Memory: 2007580 KB

@jodh-intel
Copy link

jodh-intel commented Jun 20, 2018

lgtm

The CI is failing due to lint issues. I don't yet understand why since:

  • we're pinning gometalinter to a particular commit (and that hasn't been changed in a while).
  • this PR doesn't touch the files that the linters are complaining about.

Approved with PullApprove

@jodh-intel
Copy link

Ugh - I've think I've just worked it out: kata-containers/tests#439.

We only need one qmp channel and it is qemu internal detail thus
sandbox.go does not need to be aware of it.

Fixes: kata-containers#428

Signed-off-by: Peng Tao <bergwolf@gmail.com>
@katacontainersbot
Copy link
Contributor

PSS Measurement:
Qemu: 149935 KB
Proxy: 4663 KB
Shim: 8833 KB

Memory inside container:
Total Memory: 2045972 KB
Free Memory: 2007308 KB

@codecov
Copy link

codecov bot commented Jun 20, 2018

Codecov Report

Merging #429 into master will increase coverage by 0.21%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #429      +/-   ##
==========================================
+ Coverage   63.77%   63.98%   +0.21%     
==========================================
  Files          87       87              
  Lines        8823     8794      -29     
==========================================
  Hits         5627     5627              
+ Misses       2592     2563      -29     
  Partials      604      604
Impacted Files Coverage Δ
virtcontainers/sandbox.go 66.83% <ø> (ø) ⬆️
virtcontainers/qemu.go 20.48% <0%> (+1.13%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c324b55...8f329db. Read the comment docs.

@sboeuf sboeuf merged commit fca7eb8 into kata-containers:master Jun 20, 2018
lifupan pushed a commit to hyperhq/kata-runtime that referenced this pull request Jun 22, 2018
@bergwolf bergwolf deleted the qmp branch September 13, 2018 03:27
zklei pushed a commit to zklei/runtime that referenced this pull request Jun 13, 2019
…c1-branch-bump

# Kata Containers 1.5.0-rc1
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.

5 participants