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

cli: make check subcommand more tolerant to failures #3086

Merged
merged 2 commits into from
Nov 27, 2020

Conversation

devimc
Copy link

@devimc devimc commented Nov 19, 2020

No description provided.

@devimc devimc requested a review from jodh-intel November 19, 2020 21:55
cli/kata-check.go Outdated Show resolved Hide resolved
@devimc devimc force-pushed the 2020-11-19/fixKataCheck branch from 6b312a6 to 901eb01 Compare November 20, 2020 14:35
@devimc
Copy link
Author

devimc commented Nov 20, 2020

@amshinde change applied, thanks

Copy link
Contributor

@jodh-intel jodh-intel left a comment

Choose a reason for hiding this comment

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

Thanks @devimc! A bunch of nits, so...

lgtm

cli/kata-check.go Outdated Show resolved Hide resolved
cli/kata-check.go Outdated Show resolved Hide resolved
cli/release.go Show resolved Hide resolved
cli/release.go Outdated Show resolved Hide resolved
@devimc devimc force-pushed the 2020-11-19/fixKataCheck branch from 901eb01 to dd1e867 Compare November 20, 2020 15:19
@devimc
Copy link
Author

devimc commented Nov 20, 2020

@jodh-intel changes applied, thanks

/test

Don't fail if rate limit is exceeded since this is a
limitation/restriction of Github not a problem in the host.
Print a warning when the rate limit is exceeded.

For more information about Github's rate limit, see
https://developer.github.com/v3/#rate-limiting

Signed-off-by: Julio Montes <julio.montes@intel.com>
@devimc devimc force-pushed the 2020-11-19/fixKataCheck branch from dd1e867 to aa26753 Compare November 25, 2020 16:36
@devimc
Copy link
Author

devimc commented Nov 25, 2020

/test

Before loading a module, the check subcommand should check if the
current user can load it.

fixes kata-containers#3085

Signed-off-by: Julio Montes <julio.montes@intel.com>
@devimc devimc force-pushed the 2020-11-19/fixKataCheck branch from aa26753 to c963777 Compare November 25, 2020 17:16
@devimc
Copy link
Author

devimc commented Nov 25, 2020

/test

@codecov
Copy link

codecov bot commented Nov 25, 2020

Codecov Report

Merging #3086 (c963777) into master (806cc56) will decrease coverage by 0.05%.
The diff coverage is 54.54%.

@@            Coverage Diff             @@
##           master    #3086      +/-   ##
==========================================
- Coverage   50.42%   50.37%   -0.06%     
==========================================
  Files         120      120              
  Lines       15910    15923      +13     
==========================================
- Hits         8023     8021       -2     
- Misses       6796     6812      +16     
+ Partials     1091     1090       -1     

@devimc devimc added the forward-port Change from an older branch / repository label Nov 26, 2020
@devimc devimc merged commit 2a98f43 into kata-containers:master Nov 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
forward-port Change from an older branch / repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants