Skip to content

Conversation

@klihub
Copy link
Member

@klihub klihub commented Nov 11, 2025

This PR bumps our opencontainers/runtime-spec dependency to v1.3.0.

In particular it

  • bumps core NRI runtime-spec and runtime-tools dependency to match runtime-spec v1.3.0
  • refreshes in-repo plugins' dependencies accordingly
  • updates the v010-adapter plugin to depend on NRI v0.10.0 and runtime-spec v1.1.0
  • updates v0.1.0 examples to depend on NRI v0.10.0 and runtime-spec v1.1.0

@klihub klihub force-pushed the devel/runtime-spec-v1.3.0 branch from ce6cad4 to f5b81c3 Compare November 11, 2025 12:49
@mikebrow
Copy link
Member

nice unwind... cypher just lgtm'd your 795 so this shouldn't need to be draft for long :)

Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
Instead of using the in-repo NRI version, lock it to v0.10.0,
the last tagged version with a runtime-spec dependency below
v1.3.0, to prevent the incompatible Linux PIDS Limit pointer
change from creeping in.

Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
Instead of using the in-repo NRI version, lock it to v0.10.0,
the last tagged version with a runtime-spec dependency below
v1.3.0, to prevent the incompatible Linux PIDS Limit pointer
change from creeping in.

Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
@klihub klihub force-pushed the devel/runtime-spec-v1.3.0 branch from f5b81c3 to 29c5811 Compare November 14, 2025 10:58
@klihub klihub marked this pull request as ready for review November 14, 2025 10:59
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 just a couple nits..

github.com/opencontainers/runtime-spec v1.1.0
github.com/opencontainers/runtime-tools v0.9.0
github.com/opencontainers/runtime-spec v1.3.0
github.com/opencontainers/runtime-tools v0.9.1-0.20251114084447-edf4cb3d2116
Copy link
Member

Choose a reason for hiding this comment

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

nit // runtime-tools has not tagged a release in six years have to move up to commit to resolve __


const (
// UnlimitedPidsLimit indicates unlimited Linux PIDs limit.
UnlimitedPidsLimit = -1
Copy link
Member

Choose a reason for hiding this comment

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

nit todo: define and retrieve from runtime-tools?

Copy link
Member

Choose a reason for hiding this comment

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

I guess runtime-tools might be eventually going to be deprecated, so this seems fine for now

@AkihiroSuda AkihiroSuda merged commit 3827d9d into containerd:main Nov 17, 2025
16 checks passed
@klihub klihub deleted the devel/runtime-spec-v1.3.0 branch November 17, 2025 10:19
@AkihiroSuda
Copy link
Member

Can we have a new git tag?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants