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

shim/firecracker: Read agent's logs #210

Merged
merged 2 commits into from
Oct 7, 2019

Conversation

devimc
Copy link

@devimc devimc commented Oct 4, 2019

Add support for reading agent's logs from a hybrid vsock. This feature is
very useful for debugging hypervisors that don't have a socket connected to
/dev/console, like firecracker

fixes #209

Signed-off-by: Julio Montes [email protected]

@devimc devimc added the do-not-merge PR has problems or depends on another label Oct 4, 2019
@devimc devimc force-pushed the topic/logs/fcReadAgentLogs branch 2 times, most recently from 9d6c9be to 0a26055 Compare October 7, 2019 15:32
@devimc
Copy link
Author

devimc commented Oct 7, 2019

/test

@jodh-intel change applied, thanks

@devimc devimc removed the do-not-merge PR has problems or depends on another label Oct 7, 2019
Copy link
Contributor

@jodh-intel jodh-intel left a comment

Choose a reason for hiding this comment

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

Thanks @devimc.

lgtm

Julio Montes added 2 commits October 7, 2019 16:38
Add support for reading agent's logs from a hybrid vsock. This feature is
very useful for debugging hypervisors that don't have a socket connected to
`/dev/console`, like firecracker

fixes kata-containers#209

Signed-off-by: Julio Montes <[email protected]>
bring support for logging through a hybrid vsock

shortlog:
95be1c3 agent: add support for logging to a vsock port
a03e23b protocols/client: improve hybrid vsock parser
6a96997 protocols/client: make schemes and hybrid vsock dialer public
e01f23c network: Add a testcase for setupDNS
d733185 network: Setup DNS for sandbox

Signed-off-by: Julio Montes <[email protected]>
@devimc devimc force-pushed the topic/logs/fcReadAgentLogs branch from b8a8629 to 6aa2d82 Compare October 7, 2019 16:38
@codecov
Copy link

codecov bot commented Oct 7, 2019

Codecov Report

Merging #210 into master will decrease coverage by 3.54%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #210      +/-   ##
==========================================
- Coverage    37.7%   34.15%   -3.55%     
==========================================
  Files           7        7              
  Lines         366      404      +38     
==========================================
  Hits          138      138              
- Misses        215      253      +38     
  Partials       13       13

devimc pushed a commit to devimc/kata-runtime that referenced this pull request Oct 7, 2019
In firecracker, there is no socket connected to /dev/console, so let's
use a vsock port to get agent's logs

Depends-on: github.com/kata-containers/shim#210

fixes kata-containers#2103

Signed-off-by: Julio Montes <[email protected]>
@devimc
Copy link
Author

devimc commented Oct 7, 2019

/test

@devimc devimc merged commit 264c1e3 into kata-containers:master Oct 7, 2019
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.

firecracker: add support for reading agent's logs
3 participants