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

scripts: Add agent version to collect script #1057

Conversation

jodh-intel
Copy link
Contributor

Add the agent version to the collect script output.

Fixes #1055.

Signed-off-by: James O. D. Hunt [email protected]

Make more variables local in the collect script.

Signed-off-by: James O. D. Hunt <[email protected]>
Drop support for the `help` and `version` sub-commands in favour of
their CLI options only.

Signed-off-by: James O. D. Hunt <[email protected]>
Switch to using the `getopt(1)` command for argument parsing as it
supports short and long options.

Signed-off-by: James O. D. Hunt <[email protected]>
@jodh-intel
Copy link
Contributor Author

This will "work", but won't actually show the correct agent version until we get a new image release containing clearcontainers/agent#213 (commit clearcontainers/agent@14c4a2d).

Tested with Kata too (run as sudo kata-collect-data.sh) which shows the following on my test box:


Agent

version:

kata-agent version 0.0.1-33eecb2a445f906811a5bc9713d2dafd10768d18-dirty

Add the agent version to the collect script output.

Fixes clearcontainers#1055.

Signed-off-by: James O. D. Hunt <[email protected]>
Add a `--debug` option to the data collection script that could be
useful in the case where the agent version cannot be established.

Signed-off-by: James O. D. Hunt <[email protected]>
@jodh-intel jodh-intel force-pushed the collect-script-show-agent-version branch from 692cb40 to 17ad8f6 Compare March 9, 2018 17:48
@sboeuf
Copy link
Contributor

sboeuf commented Mar 9, 2018

Thanks @jodh-intel for this PR.
LGTM

Copy link
Contributor

@grahamwhaley grahamwhaley left a comment

Choose a reason for hiding this comment

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

lgtm

@grahamwhaley grahamwhaley merged commit 043b0ba into clearcontainers:master Mar 12, 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.

3 participants