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

kata-check: check for newer release #2976

Merged

Conversation

jodh-intel
Copy link
Contributor

Update kata-check to see if there is a newer version available for download. Useful for users installing static packages (without a package manager).

Fixes: #2975.

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

@jodh-intel jodh-intel added the backport Change from a newer branch / repository label Sep 21, 2020
@jodh-intel
Copy link
Contributor Author

Backport of kata-containers/kata-containers#735.

@jodh-intel
Copy link
Contributor Author

/test

@jodh-intel jodh-intel force-pushed the kata-check-compare-versions branch from 44c63dc to eb47b77 Compare September 22, 2020 08:06
@jodh-intel
Copy link
Contributor Author

/test

@codecov
Copy link

codecov bot commented Sep 22, 2020

Codecov Report

Merging #2976 into master will decrease coverage by 1.37%.
The diff coverage is 71.72%.

@@            Coverage Diff             @@
##           master    #2976      +/-   ##
==========================================
- Coverage   51.65%   50.27%   -1.38%     
==========================================
  Files         118      119       +1     
  Lines       17430    15758    -1672     
==========================================
- Hits         9003     7923    -1080     
+ Misses       7342     6753     -589     
+ Partials     1085     1082       -3     

Update `kata-check` to see if there is a newer version available for
download. Useful for users installing static packages (without a package
manager).

Fixes: kata-containers#2975.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
@jodh-intel jodh-intel force-pushed the kata-check-compare-versions branch from eb47b77 to a707608 Compare September 23, 2020 07:54
jodh-intel added a commit to jodh-intel/kata-containers that referenced this pull request Sep 23, 2020
Fix the linter errors caught in the `runtime` repos `master` branch [1],
but not in the `2.0-dev` branch [2]. See [3] for further details.

[1] - kata-containers/runtime#2976
[2] - kata-containers#735
[3] - kata-containers/tests#2870

Fixes: kata-containers#783.

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

Ping @kata-containers/runtime !

@bpradipt
Copy link
Contributor

Ping @kata-containers/runtime !

@jodh-intel started looking at it.. .Thanks

Name: "verbose, v",
Usage: "display the list of checks performed",
Name: "check-version-only",
Usage: "Only compare the current and latest available versions (requires network, non-root only)",
Copy link
Contributor

Choose a reason for hiding this comment

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

A minor nit.. It's not obvious from the description that it checks for release versions only. Although the subsequent options help with the clarification. Just noting down in case if it makes sense to make it more explicit.

Copy link
Contributor

@bpradipt bpradipt left a comment

Choose a reason for hiding this comment

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

/lgtm
Thanks @jodh-intel

Copy link

@jensfr jensfr left a comment

Choose a reason for hiding this comment

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

lgtm

@jodh-intel
Copy link
Contributor Author

Ping @kata-containers/runtime. This can land with one more qualifying ack (it's already landed in 2.x so really shouldn't take much time to review.... :)

Copy link

@devimc devimc left a comment

Choose a reason for hiding this comment

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

thanks @jodh-intel

@jodh-intel
Copy link
Contributor Author

/test

@amshinde amshinde merged commit fd448db into kata-containers:master Sep 29, 2020
egernst pushed a commit to egernst/kata-containers that referenced this pull request Oct 2, 2020
Fix the linter errors caught in the `runtime` repos `master` branch [1],
but not in the `2.0-dev` branch [2]. See [3] for further details.

[1] - kata-containers/runtime#2976
[2] - kata-containers#735
[3] - kata-containers/tests#2870

Fixes: kata-containers#783.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
egernst pushed a commit to kata-containers/kata-containers that referenced this pull request Oct 7, 2020
Fix the linter errors caught in the `runtime` repos `master` branch [1],
but not in the `2.0-dev` branch [2]. See [3] for further details.

[1] - kata-containers/runtime#2976
[2] - #735
[3] - kata-containers/tests#2870

Fixes: #783.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
c3d pushed a commit to kata-containers/kata-containers that referenced this pull request Apr 12, 2021
Fix the linter errors caught in the `runtime` repos `master` branch [1],
but not in the `2.0-dev` branch [2]. See [3] for further details.

[1] - kata-containers/runtime#2976
[2] - #735
[3] - kata-containers/tests#2870

Fixes: #783.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
c3d pushed a commit to c3d/kata-containers that referenced this pull request Apr 12, 2021
Fix the linter errors caught in the `runtime` repos `master` branch [1],
but not in the `2.0-dev` branch [2]. See [3] for further details.

[1] - kata-containers/runtime#2976
[2] - kata-containers#735
[3] - kata-containers/tests#2870

Fixes: kata-containers#783.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport Change from a newer branch / repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Backport] runtime: make kata-check check for newer release
5 participants