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

trace: Add missing log message#1815

Merged
jcvenegas merged 1 commit intokata-containers:masterfrom
jodh-intel:log-trace-msg
Jun 20, 2019
Merged

trace: Add missing log message#1815
jcvenegas merged 1 commit intokata-containers:masterfrom
jodh-intel:log-trace-msg

Conversation

@jodh-intel
Copy link
Copy Markdown

Add a log message for every trace span created, required by the tracing tests to validate tracing is working.

Fixes: #1814.

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

Add a log message for every trace span created, required by the tracing
tests to validate tracing is working.

Fixes: kata-containers#1814.

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

/test

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 19, 2019

Codecov Report

Merging #1815 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1815   +/-   ##
=======================================
  Coverage   54.59%   54.59%           
=======================================
  Files         106      106           
  Lines       13035    13035           
=======================================
  Hits         7117     7117           
  Misses       5071     5071           
  Partials      847      847

@egernst
Copy link
Copy Markdown
Member

egernst commented Jun 19, 2019

opened issue to track the CI failures: kata-containers/tests#1745

@jcvenegas
Copy link
Copy Markdown
Member

E0619 11:33:09.400132  108168 reset.go:192] [reset] Failed to remove containers: failed to stop running pod 5be64495f71897c17815be9e1860e305d3157638ed014eb057ccd8e1ae4b38e8: output: time="2019-06-19T11:32:59Z" level=fatal msg="stopping the pod sandbox \"5be64495f71897c17815be9e1860e305d3157638ed014eb057ccd8e1ae4b38e8\" failed: rpc error: code = Unknown desc = failed to stop sandbox container \"5be64495f71897c17815be9e1860e305d3157638ed014eb057ccd8e1ae4b38e8\" in '\\x01' state: wait sandbox container \"5be64495f71897c17815be9e1860e305d3157638ed014eb057ccd8e1ae4b38e8\" stop timeout"
, error: exit status 1

restarting cri-o job not related with this PR the error

centos failed due to timeout,

Downloading packages:
Delta RPMs disabled because /usr/bin/applydeltarpm not installed.
Build timed out (after 5 minutes). Marking the build as aborte

@jodh-intel is this something you consider should be a fix for stable branches ? I dont think this is critical
so I say no, but let me know.

@jcvenegas jcvenegas merged commit a5b8ff4 into kata-containers:master Jun 20, 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.

Add missing trace log message

4 participants