nri icon indicating copy to clipboard operation
nri copied to clipboard

api,adaptation,generate: allow adjusting linux net devices

Open klihub opened this issue 8 months ago • 13 comments

Allow plugins to adjust linux net devices.

Notes: This PR is marked a draft since it uses an untagged opencontainers/runtime-spec version v1.2.2-0.20250401095657-e935f995dd67. Once a new version is tagged, we can mark this PR ready for review. Testing this requires

klihub avatar Apr 08 '25 11:04 klihub

❤️

aojea avatar Apr 08 '25 18:04 aojea

/cc @aojea @tao12345666333 Here is the draft PR proposal #180 was asking for.

klihub avatar Jun 24 '25 11:06 klihub

This is fantastic @klihub , can we undraft it for final review

aojea avatar Jun 25 '25 05:06 aojea

This is fantastic @klihub , can we undraft it for final review

We can try to do that to get it reviewed. @mikebrow WDYT?

But I think we won't be able to get this merged before the necessary new bits in runtime-spec gets behind a tag. Both CRI-O and containerd main/HEAD is at [email protected] and I think we usually tend to stick to a tagged version in both even in main.

klihub avatar Jun 25 '25 06:06 klihub

ok, that is fair

aojea avatar Jun 25 '25 07:06 aojea

Fixes: #180

aojea avatar Jun 25 '25 11:06 aojea

Containerd https://github.com/containerd/containerd/pull/12295 is already testing runc 1.4.0-rc.2 CRIO uses crun that already has the feature https://github.com/containers/crun/releases/tag/1.22 OCI spec is close to release the definitive 1.3.0 https://github.com/opencontainers/runtime-spec/issues/1295

aojea avatar Oct 14 '25 10:10 aojea

Thx for the update @aojea

mikebrow avatar Oct 14 '25 21:10 mikebrow

Containerd containerd/containerd#12295 is already testing runc 1.4.0-rc.2 CRIO uses crun that already has the feature https://github.com/containers/crun/releases/tag/1.22 OCI spec is close to release the definitive 1.3.0 opencontainers/runtime-spec#1295

@aojea @mikebrow But I think we'd still need a tagged version of opencontainers/runtime-spec we could point ourselves at.

klihub avatar Oct 15 '25 14:10 klihub

@klihub yeah, I was using the comment as a breadcrumb , so next time I check I can track teh history, otherwise I forget

I was not trying to push for merging before that, sorry for the confusion

aojea avatar Oct 16 '25 09:10 aojea

@klihub yeah, I was using the comment as a breadcrumb , so next time I check I can track teh history, otherwise I forget

I was not trying to push for merging before that, sorry for the confusion

@aojea No prob / no offense taken. I just asked to make sure I understand correctly where we are and whether we can try to move this forward yet.

klihub avatar Oct 16 '25 10:10 klihub

informational update , ETA for 1.3.0 spec is 2025/11/04 https://github.com/opencontainers/runc/issues/4875#issuecomment-3454561451 , runc will follow

aojea avatar Oct 28 '25 07:10 aojea

@aojea @mikebrow With opencontainers/runtime-spec v1.3.0 now tagged and released, unfortunately opencontainer/runtime-tools is now badly out of sync with runtime-spec.

[email protected] brings a flips the linux PID limit setting the from int64 to *int64. While this allows to set an 'unlimited' limit using nil, it is a backward incompatible change and it essentially makes it impossible to import [email protected] and runtimes-tools@current HEAD to the same golang code base.

I filed a PR for fixing this but I'm not sure what are the chances of getting that merged soon

klihub avatar Nov 04 '25 12:11 klihub

it seems the dependency problem got unblocked

aojea avatar Nov 17 '25 10:11 aojea