Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

NRI: error on unsupported adjustment#51712

Merged
robmry merged 1 commit into
moby:mastermoby/moby:masterfrom
robmry:nri-error-on-unsupported-adjustmentrobmry/moby:nri-error-on-unsupported-adjustmentCopy head branch name to clipboard
Dec 15, 2025
Merged

NRI: error on unsupported adjustment#51712
robmry merged 1 commit into
moby:mastermoby/moby:masterfrom
robmry:nri-error-on-unsupported-adjustmentrobmry/moby:nri-error-on-unsupported-adjustmentCopy head branch name to clipboard

Conversation

@robmry

@robmry robmry commented Dec 12, 2025

Copy link
Copy Markdown
Contributor

- What I did

Return an error if NRI tries to make an update we don't support yet.

- How I did it

- How to verify it

- Human readable description for the release notes

Add experimental NRI support

- A picture of a cute animal (not mandatory but encouraged)

@robmry robmry added this to the 29.2.0 milestone Dec 12, 2025
@robmry robmry self-assigned this Dec 12, 2025
@robmry robmry added kind/feature Functionality or other elements that the project doesn't currently have. Features are new and shiny area/daemon Core Engine labels Dec 12, 2025
@robmry
robmry marked this pull request as draft December 12, 2025 17:42
@robmry
robmry force-pushed the nri-error-on-unsupported-adjustment branch 3 times, most recently from e6e8aaf to 9066c4b Compare December 15, 2025 09:42
@robmry
robmry marked this pull request as ready for review December 15, 2025 09:46

@austinvazquez austinvazquez left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, just had a quick question on one part of the check unsupported adjustments function.

Comment thread daemon/internal/nri/nri.go Outdated
Also error on eviction requests.

Signed-off-by: Rob Murray <rob.murray@docker.com>
@robmry
robmry force-pushed the nri-error-on-unsupported-adjustment branch from 9066c4b to 98d196b Compare December 15, 2025 14:08
@robmry
robmry merged commit 425f239 into moby:master Dec 15, 2025
268 of 271 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/daemon Core Engine impact/changelog kind/feature Functionality or other elements that the project doesn't currently have. Features are new and shiny

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.