-
Notifications
You must be signed in to change notification settings - Fork 18
Conversation
Note the |
kubernetes qa-failed 👎 |
cmd/cc-log-parser/README.md
Outdated
To merge together all logs: | ||
|
||
1. Enable global logging in the runtime | ||
1. Enable debug logging for the proxy. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1.
again?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah! 😄 if you uses 1.
, it triggers markdown to auto-number. If you click on the [View] button, you'll see they get magically updated ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ouch! yep sorry
f07205c
to
71fcba5
Compare
The |
kubernetes qa-passed 👍 |
Hi @iphutch - please could you review the README on this PR (https://github.com/clearcontainers/tests/pull/590/files#diff-c294cba5b0c553706556ef34aa2e7429)? |
@jodh-intel, tried this tool and got an error when merging logs from proxy and runtime:
I have the |
Hi @chavafg - thanks for testing. I think the problem you are seeing is because the latest ... but that was from 23 August: @jcvenegas, @gorozco1 - am I getting confused here or do we need to update the image (I thought we were doing it weekly as part of the release process)? Which reminds me: clearcontainers/runtime#632 :) |
@jodh-intel the image was not updated last week, the agent was not tagged for a release, I updated to master anyway on Friday, let me send a PR to updated. |
Thanks @jcvenegas! |
@jodh-intel and ouch let me write a docuement about image update. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An indentation, wording suggestion, and note. Indentation and wording is all that needs to be changed.
cmd/cc-log-parser/README.md
Outdated
|
||
The primary two logs that are passed to the tool are the [runtime global | ||
log](https://github.com/clearcontainers/runtime#debugging) and the [proxy | ||
log](https://github.com/clearcontainers/proxy#debugging). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note:
This wrapping doesn't break anything because it is within the rendered text, however, note that you can ignore 78char rules when it is a link taking you over the max. It's fine as is but should the 78 char mark hit outside of the square brackets, you would wrap text before or after the full
[runtime global log](https://github.com/clearcontainers/runtime#debugging)
cmd/cc-log-parser/README.md
Outdated
automatically added to the runtimes global log. | ||
|
||
The [agent](https://github.com/clearcontainers/agent) logs are encoded inside | ||
the proxies log. The tool automatically unpacks these and displays only the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/The tool/The cc-log-parser
tool
OR
s/The tool/cc-log-parser
cmd/cc-log-parser/README.md
Outdated
$ sudo cp /var/lib/clear-containers/runtime/runtime.log ./runtime.log | ||
``` | ||
1. Run the script | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indent this code block
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for reviewing @iphutch - branch updated.
9548146
to
34e9873
Compare
kubernetes qa-passed 👍 |
1 similar comment
kubernetes qa-passed 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The README looks great
Hi @iphutch - branch updated. |
Failing on F26 due to the
|
34e9873
to
51fa1ac
Compare
kubernetes qa-passed 👍 |
51fa1ac
to
4fe3add
Compare
Branch updated now that the shim also produces structured logs (clearcontainers/shim#84). |
kubernetes qa-passed 👍 |
As mentioned on #603, unless you use https://github.com/clearcontainers/osbuilder, you'll need to use |
4fe3add
to
d87a293
Compare
Found a bug - the agent unpack doesn't work if you have hypervisor debug enabled in the runtime config file, so you'll need to ensure that I'll see if there is a clean way to add a new field for hypervisor output that can be added so |
kubernetes qa-passed 👍 |
d87a293
to
502bfe2
Compare
kubernetes qa-failed 👎 |
hi @jodh-intel , did you find a way to solve the issue you described above? or should we add in the README a statement to indicate not use with |
502bfe2
to
5c59ecb
Compare
Hi @chavafg - yes, I've found a way and updated the branch. To test it fully, you'll need to use the latest shim built from source (
But alternatively, just don't pass |
5c59ecb
to
940627a
Compare
kubernetes qa-passed 👍 |
1 similar comment
kubernetes qa-passed 👍 |
Add a "cc-log-parser" tool that reads multiple logfmt [*] logfiles and writes their output in time order showing time differences between log entries. Fixes clearcontainers#578. [*] - https://brandur.org/logfmt Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
940627a
to
058c6f7
Compare
kubernetes qa-passed 👍 |
Hi @chavafg - does this work for you now? |
Hi @jodh-intel, unfortunately still have issues: I am running with
Taking a look at the log files, the only file which has 516 lines is the proxy, and the error is because of this line:
|
kubernetes qa-failed 👎 |
kubernetes qa-passed 👍 |
kubernetes qa-passed 👍 |
1 similar comment
kubernetes qa-passed 👍 |
I'm going to close this PR as the plan is now to move the tool to https://github.com/kata-containers/tests/. |
Replaced by kata-containers/tests#98. |
metrics: Add bidirectional iperf network bandwidth
Add a "cc-log-parser" tool that reads multiple logfmt [*] logfiles and
writes their output in time order showing time differences between log
entries.
Fixes #578.
[*] - https://brandur.org/logfmt
Signed-off-by: James O. D. Hunt james.o.hunt@intel.com