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

FC: log out the firecracker's console when debug enabled #2202

Merged
merged 2 commits into from
Nov 19, 2019

Conversation

lifupan
Copy link
Member

@lifupan lifupan commented Nov 14, 2019

When hypervisor's debug option set, log out the firecracker's
console output which contains the kernel boot logs; thus it
would be easy for system panic debugging.

When agent debug was enabled by passing "agent.log=debug" to
kernel parameter, it will also log out the agent logs from
the console output.

Fixes: #2201

Signed-off-by: lifupan [email protected]

@lifupan lifupan force-pushed the watch_firecracker_console branch 2 times, most recently from 07a746b to 3f14ea1 Compare November 15, 2019 02:16
@lifupan
Copy link
Member Author

lifupan commented Nov 15, 2019

/test

@codecov
Copy link

codecov bot commented Nov 15, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@35f54fd). Click here to learn what that means.
The diff coverage is 8.51%.

@@            Coverage Diff            @@
##             master    #2202   +/-   ##
=========================================
  Coverage          ?   51.12%           
=========================================
  Files             ?      110           
  Lines             ?    15152           
  Branches          ?        0           
=========================================
  Hits              ?     7747           
  Misses            ?     6448           
  Partials          ?      957

Add a 'sateful' variable to FC to indicate
when it is called with cli or shimv2.

Signed-off-by: lifupan <[email protected]>
For shimv2 case, when hypervisor's debug option set, log out
the firecracker's console output which contains the kernel boot
logs; thus it would be easy for system panic debugging.

When agent debug was enabled by passing "agent.log=debug" to
kernel parameter, it will also log out the agent logs from
the console output.

Fixes: kata-containers#2201

Signed-off-by: lifupan <[email protected]>
@lifupan lifupan force-pushed the watch_firecracker_console branch from 3f14ea1 to a2b6afc Compare November 15, 2019 03:54
@lifupan
Copy link
Member Author

lifupan commented Nov 15, 2019

/test

@lifupan
Copy link
Member Author

lifupan commented Nov 19, 2019

ping @kata-containers/runtime

@bergwolf bergwolf merged commit be6110d into kata-containers:master Nov 19, 2019
@bergwolf bergwolf mentioned this pull request Nov 19, 2019
bergwolf added a commit to bergwolf/kata-runtime that referenced this pull request Nov 19, 2019
PR kata-containers#2202 changed createSandbox() interface but didn't get a chance
to match with cloud hypervisor change.

Fixes: kata-containers#2213

Signed-off-by: Peng Tao <[email protected]>
@bergwolf bergwolf mentioned this pull request Nov 19, 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.

log out the firecracker's console for debugging when hypervisor debug enabled
3 participants