-
Notifications
You must be signed in to change notification settings - Fork 113
Bump the container toolkit dependency to fix a high severity CVE in opencontainers/runc module #807
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
Conversation
shivamerla
commented
Jan 8, 2026
- The nvidia-cdi-hook binary copied from the toolkit image have a high CVE: GHSA-qw9x-cqr3-wc7r
- Import from the latest commit which fixes this issue. This commit is used as a reference to obtain the toolkit image from ghcr
- Fix the script to parse module version with the rc releases
hack/toolkit-container-image.sh
Outdated
| if [[ "${TOOLKIT_VERSION}" =~ ^v[0-9]+\.[0-9]+\.[0-9]+-[0-9]{14}-([a-f0-9]{12})$ ]]; then | ||
| TOOLKIT_VERSION_SHA="${BASH_REMATCH[1]}" | ||
| SHORT_SHA="${TOOLKIT_VERSION_SHA:0:8}" | ||
| IMAGE_URL="ghcr.io/nvidia/container-toolkit:${SHORT_SHA}" | ||
| # Handle format vX.Y.Z-rc.A.B.time-commit | ||
| elif [[ "${TOOLKIT_VERSION}" =~ ^v[0-9]+\.[0-9]+\.[0-9]+-rc\.[0-9]+\.[0-9]+\.[0-9]{14}-([a-f0-9]{12,40})$ ]]; then | ||
| TOOLKIT_VERSION_SHA="${BASH_REMATCH[1]}" | ||
| SHORT_SHA="${TOOLKIT_VERSION_SHA:0:8}" | ||
| IMAGE_URL="ghcr.io/nvidia/container-toolkit:${SHORT_SHA}" |
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.
Can these not be combined into a single regex?
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.
Looks like the fix made it into v1.18.x branch, so i am going to bump to that instead which makes this change unnecessary. Will update the PR shortly.
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.
@klueska updated this to be a single regex. PTAL.
522fcd4 to
bc1acbe
Compare
|
@elezar has said that 1.18.2 will go out a soon as Monday. Let's wait for that to get the official tag in, rather than a 1-off sha. |
|
v1.18.2 has been released. I have updated the PR. |
eaaf162 to
a28cab2
Compare
thanks @elezar |
…ty CVE in opencontainers/runc module * The nvidia-cdi-hook binary copied from the toolkit image have a high CVE: GHSA-qw9x-cqr3-wc7r * Import from the latest commit which fixes this issue. This commit is used as a reference to obtain the toolkit image from ghcr * Fix the script to parse module version with the rc releases Signed-off-by: Shiva Krishna, Merla <smerla@nvidia.com> Signed-off-by: Evan Lezar <elezar@nvidia.com>
a28cab2 to
8db6b40
Compare
|
#825 landed in the meantime :). I think we can maybe close this one for now. If we ever need to regex proposed here, we can dig it up again. |
|
Yes. |