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

qemu: Pass sandboxID to agent for logging purposes #466

Merged

Conversation

jodh-intel
Copy link
Contributor

Add a kernel command-line option that the agent can read to determine
the sandbox ID of the VM. It can use this to create a sandbox= log
field for improved log analysis.

Fixes #465.

Signed-off-by: James O. D. Hunt james.o.hunt@intel.com

Add a kernel command-line option that the agent can read to determine
the sandbox ID of the VM. It can use this to create a `sandbox=` log
field for improved log analysis.

Fixes kata-containers#465.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
@jodh-intel
Copy link
Contributor Author

Related: #463.

Copy link
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.

Using the kernel arg is pretty novel - will be interesting to see this trickle through to the agent :-)
lgtm

@jodh-intel
Copy link
Contributor Author

The other half of this feature is: kata-containers/agent#292.

@katacontainersbot
Copy link
Contributor

PSS Measurement:
Qemu: 162083 KB
Proxy: 4720 KB
Shim: 8836 KB

Memory inside container:
Total Memory: 2045968 KB
Free Memory: 2007384 KB

@codecov
Copy link

codecov bot commented Jul 4, 2018

Codecov Report

Merging #466 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #466      +/-   ##
==========================================
- Coverage   64.24%   64.23%   -0.01%     
==========================================
  Files          87       87              
  Lines        8837     8838       +1     
==========================================
  Hits         5677     5677              
- Misses       2554     2555       +1     
  Partials      606      606
Impacted Files Coverage Δ
virtcontainers/qemu.go 20.44% <0%> (-0.05%) ⬇️

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 bf1cf68...793a220. Read the comment docs.

@devimc
Copy link

devimc commented Jul 4, 2018

lgtm

Approved with PullApprove

@jodh-intel jodh-intel merged commit 47dfb7d into kata-containers:master Jul 5, 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.

4 participants