[go: nahoru, domu]

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

utils: kata-manager: Ensure only one download URL #8374

Merged

Conversation

jodh-intel
Copy link
Contributor

Add an extra sanity check to ensure that only a single download URL is
found for the specified release version.

Fixes: #8364.

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

@katacontainersbot katacontainersbot added the size/small Small and simple task label Nov 3, 2023
@jodh-intel
Copy link
Contributor Author

/test

@jodh-intel jodh-intel force-pushed the kata-manager-check-dl-url-count branch from d36345c to 8705467 Compare November 7, 2023 17:48
@jodh-intel jodh-intel added area/testing Issues related to testing ok-to-test safe-to-test Add to PR after manually reviewing to allow certain extra checks to run and removed area/testing Issues related to testing labels Nov 7, 2023
@jodh-intel
Copy link
Contributor Author

/test

Improve some of the information messages a little by adding
more detail and quoting file names.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
Add an extra sanity check to ensure that only a single download URL is
found for the specified release version.

Fixes: kata-containers#8364.

Signed-off-by: James O. D. Hunt <james.o.hunt@intel.com>
@jodh-intel jodh-intel force-pushed the kata-manager-check-dl-url-count branch from 8705467 to c72a27e Compare November 10, 2023 08:28
Copy link
Member
@fidencio fidencio 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!

@fidencio
Copy link
Member

/test

@jodh-intel jodh-intel merged commit f588d31 into kata-containers:main Nov 10, 2023
137 of 142 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test safe-to-test Add to PR after manually reviewing to allow certain extra checks to run size/small Small and simple task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

utils: kata-manager: Ensure only one download URL
4 participants