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

Respect containerd's debug config #2208

Merged
merged 1 commit into from
Nov 22, 2019
Merged

Conversation

choury
Copy link
Contributor

@choury choury commented Nov 18, 2019

Currently, There are too many logs from kata shim in containerd's log. We should handle it.

Fix #2209

@grahamwhaley
Copy link
Contributor

Hi @choury - thanks for the PR. Looks sane to me, but I will let others fully review....
you will however need to open an Issue and use that Issue number in your commit for the CI to be happy - see the https://github.com/kata-containers/community/blob/master/CONTRIBUTING.md#patch-format for details....
Can I ask, when you open the Issue, can you add details (snippets of the logs), showing the before (too much noise) and after (less noise) examples?

@choury choury force-pushed the patch-1 branch 5 times, most recently from 1969419 to d30f210 Compare November 18, 2019 14:52
There are too many logs from kata shim in containerd's log.
Fixes kata-containers#2209

Signed-off-by: choury <[email protected]>
@lifupan
Copy link
Member

lifupan commented Nov 21, 2019

/test

@codecov
Copy link

codecov bot commented Nov 21, 2019

Codecov Report

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

@@            Coverage Diff            @@
##             master    #2208   +/-   ##
=========================================
  Coverage          ?   49.04%           
=========================================
  Files             ?      111           
  Lines             ?    16063           
  Branches          ?        0           
=========================================
  Hits              ?     7878           
  Misses            ?     7211           
  Partials          ?      974

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 @choury!

lgtm

@GabyCT GabyCT merged commit 7938cd8 into kata-containers:master Nov 22, 2019
@choury choury deleted the patch-1 branch November 22, 2019 15:16
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.

Too many noisy logs from containerd-shim-kata-v2
5 participants