Skip to content

Conversation

@klihub
Copy link
Member

@klihub klihub commented Apr 8, 2025

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 klihub force-pushed the devel/linux-net-devices branch from df6df04 to 02f0b68 Compare April 8, 2025 11:28
@aojea
Copy link
Contributor

aojea commented Apr 8, 2025

❤️

@klihub klihub force-pushed the devel/linux-net-devices branch from 02f0b68 to d886c51 Compare April 9, 2025 08:19
@klihub klihub requested review from fuweid and mikebrow May 2, 2025 08:24
@klihub klihub force-pushed the devel/linux-net-devices branch 2 times, most recently from df21d77 to 20533ae Compare June 24, 2025 11:10
@klihub klihub force-pushed the devel/linux-net-devices branch from 20533ae to 6e14a15 Compare June 24, 2025 11:13
@klihub
Copy link
Member Author

klihub commented Jun 24, 2025

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

@aojea
Copy link
Contributor

aojea commented Jun 25, 2025

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

@klihub klihub force-pushed the devel/linux-net-devices branch from 6e14a15 to a03dcca Compare June 25, 2025 06:26
@klihub
Copy link
Member Author

klihub commented Jun 25, 2025

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 runtime-spec@v1.2.1 and I think we usually tend to stick to a tagged version in both even in main.

@aojea
Copy link
Contributor

aojea commented Jun 25, 2025

ok, that is fair

@aojea
Copy link
Contributor

aojea commented Jun 25, 2025

Fixes: #180

@aojea
Copy link
Contributor

aojea commented Oct 14, 2025

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

@mikebrow
Copy link
Member

Thx for the update @aojea

@klihub klihub force-pushed the devel/linux-net-devices branch from a999d9b to 8df6cec Compare October 15, 2025 14:31
Point opencontainers/runtime-spec to a (yet untagged) version
with support for linux net device injection.

Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
We still have support for the deprecated OCI Prestart hook.
Add ignore rule for the related linting error.

Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
Allow adding and removing container linux net devices.

Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
Signed-off-by: Krisztian Litkey <krisztian.litkey@intel.com>
@klihub klihub force-pushed the devel/linux-net-devices branch from 8df6cec to 0a96043 Compare October 15, 2025 14:33
@klihub
Copy link
Member Author

klihub commented Oct 15, 2025

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.

@aojea
Copy link
Contributor

aojea commented Oct 16, 2025

@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

@klihub
Copy link
Member Author

klihub commented Oct 16, 2025

@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.

@aojea
Copy link
Contributor

aojea commented Oct 28, 2025

informational update , ETA for 1.3.0 spec is 2025/11/04 opencontainers/runc#4875 (comment) , runc will follow

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