Skip to content

go.{mod,sum}: bump CDI deps to v.0.8.1.#11449

Merged
fuweid merged 1 commit intocontainerd:mainfrom
klihub:fixes/main/cdi-emfile-sigsegv
Feb 27, 2025
Merged

go.{mod,sum}: bump CDI deps to v.0.8.1.#11449
fuweid merged 1 commit intocontainerd:mainfrom
klihub:fixes/main/cdi-emfile-sigsegv

Conversation

@klihub
Copy link
Copy Markdown
Member

@klihub klihub commented Feb 27, 2025

Update CDI dependency to v0.8.1. That pulls in a fix for a SIGSEGV/panic, which in principle could be triggered if we're out of file descriptors while we're instantiating the CDI cache.

@dosubot dosubot Bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Feb 27, 2025
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah 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!

@github-project-automation github-project-automation Bot moved this from Needs Triage to Review In Progress in Pull Request Review Feb 27, 2025
@AkihiroSuda AkihiroSuda added this pull request to the merge queue Feb 27, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch Feb 27, 2025
@thaJeztah
Copy link
Copy Markdown
Member

Looks like this needs a rebase, @klihub

Signed-off-by: Krisztian Litkey <[email protected]>
@klihub klihub force-pushed the fixes/main/cdi-emfile-sigsegv branch from c52967f to 22d568f Compare February 27, 2025 16:10
@fuweid fuweid added this pull request to the merge queue Feb 27, 2025
Merged via the queue into containerd:main with commit 4d36c2d Feb 27, 2025
@github-project-automation github-project-automation Bot moved this from Review In Progress to Done in Pull Request Review Feb 27, 2025
@klihub klihub deleted the fixes/main/cdi-emfile-sigsegv branch February 28, 2025 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file go Pull requests that update Go code size/S

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

6 participants