Skip to content

cri,nri: pass linux sysctl to plugins.#12766

Merged
dmcgowan merged 1 commit intocontainerd:mainfrom
klihub:devel/main/nri-input/sysctl
Jan 9, 2026
Merged

cri,nri: pass linux sysctl to plugins.#12766
dmcgowan merged 1 commit intocontainerd:mainfrom
klihub:devel/main/nri-input/sysctl

Conversation

@klihub
Copy link
Member

@klihub klihub commented Jan 9, 2026

Implement missing support for passing any container linux sysctl parameters as input to NRI plugins.

Pass linux sysctl to plugins

@klihub klihub requested review from mikebrow and samuelkarp January 9, 2026 17:28
@github-project-automation github-project-automation bot moved this to Needs Triage in Pull Request Review Jan 9, 2026
@dosubot dosubot bot added area/cri Container Runtime Interface (CRI) area/nri Node Resource Interface (NRI) labels Jan 9, 2026
@samuelkarp
Copy link
Member

Implement missing support for passing any container POSIX rlimits as input to NRI plugins.

PR description looks to be a copy/paste error

Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

LGTM

@github-project-automation github-project-automation bot moved this from Needs Triage to Review In Progress in Pull Request Review Jan 9, 2026
@dmcgowan dmcgowan added this pull request to the merge queue Jan 9, 2026
Merged via the queue into containerd:main with commit 8e2aa0b Jan 9, 2026
52 checks passed
@github-project-automation github-project-automation bot moved this from Review In Progress to Done in Pull Request Review Jan 9, 2026
@klihub klihub deleted the devel/main/nri-input/sysctl branch January 11, 2026 16:20
@dmcgowan dmcgowan added impact/changelog and removed area/cri Container Runtime Interface (CRI) labels Mar 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/nri Node Resource Interface (NRI) impact/changelog size/S

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants