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

logger: Fix syslog identifier#36

Merged
sameo merged 1 commit into
kata-containers:masterfrom
jodh-intel:fix-syslog-identifier
Jan 31, 2018
Merged

logger: Fix syslog identifier#36
sameo merged 1 commit into
kata-containers:masterfrom
jodh-intel:fix-syslog-identifier

Conversation

@jodh-intel

Copy link
Copy Markdown

Ensure log entries specify the name of the shim, not its path.

Fixes #35.

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

@jodh-intel

Copy link
Copy Markdown
Author

Related: kata-containers/proxy#29.

@codecov

codecov Bot commented Jan 22, 2018

Copy link
Copy Markdown

Codecov Report

Merging #36 into master will increase coverage by 2.67%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
+ Coverage    41.7%   44.38%   +2.67%     
==========================================
  Files           5        4       -1     
  Lines         199      178      -21     
==========================================
- Hits           83       79       -4     
+ Misses        108       91      -17     
  Partials        8        8
Impacted Files Coverage Δ
main.go 0% <0%> (ø) ⬆️
terminal_linux.go

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 8908929...42d7d7a. Read the comment docs.

@grahamwhaley grahamwhaley left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@bergwolf

bergwolf commented Jan 25, 2018

Copy link
Copy Markdown
Member

lgtm
Please rebase to solve conflict so that this can land.

Approved with PullApprove

Ensure log entries specify the name of the shim, not its path.

Fixes kata-containers#35.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
@jodh-intel jodh-intel force-pushed the fix-syslog-identifier branch from f06f639 to 42d7d7a Compare January 29, 2018 15:19
@jodh-intel

Copy link
Copy Markdown
Author

Rebased.

@jodh-intel

Copy link
Copy Markdown
Author

Hi @kata-containers/shim - could someone look at this one word code fix please? :)

@sameo

sameo commented Jan 31, 2018

Copy link
Copy Markdown

LGTM

Approved with PullApprove

@sameo sameo merged commit b8f7336 into kata-containers:master Jan 31, 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