Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add process for SIG-Node collaboration #234

Merged

Conversation

samuelkarp
Copy link
Member

This adds public documentation of the process for containerd project collaboration with Kubernetes SIG-Node. It is mostly a formalization of what we have already been doing informally, but the formalization should help with visibility and work distribution.

@samuelkarp samuelkarp force-pushed the sig-node-collaboration-process branch from c01c2b2 to f5a26ab Compare March 25, 2025 22:48
@samuelkarp samuelkarp marked this pull request as ready for review March 25, 2025 22:50
@samuelkarp
Copy link
Member Author

@containerd/committers

Copy link

@SergeyKanzhelev SergeyKanzhelev 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 few minor suggestions

@samuelkarp samuelkarp force-pushed the sig-node-collaboration-process branch from f5a26ab to f955528 Compare March 26, 2025 00:07
content/keps.md Outdated
development and implementation of CRI.

Changes to Kubernetes (including CRI) are primarily driven through the
Kubernetes Enhancement Proposal (KEP) process. KEPs are proposed to individual
Copy link
Member

Choose a reason for hiding this comment

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

Maybe add a link to KEP

content/keps.md Outdated
[containerd/containerd](https://github.com/containerd/containerd) repository
for tracking KEPs that require CRI changes in the container
runtime or one of our other containerd subprojects. (This includes
relatively minor changes, such as a runc update or checking containerd config
Copy link
Member

Choose a reason for hiding this comment

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

"runc update" means updating the /usr/local/sbin/runc binary (for the CI) or the runc update command?

At least the former one rarely relates to a KEP especially for patch releases, and does not seem to need an issue ticket.

Copy link
Member Author

Choose a reason for hiding this comment

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

@mikebrow suggested this set of examples; I think it means updating the runc binary.

Copy link
Member

Choose a reason for hiding this comment

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

yes I meant even the minor issue of updating the runc-version file over in script/ in this case...

I have agreed with the point that it does not seem we need an issue ticket in every case.. Believe we had a thread where you and I agreed some changes are so easy and obvious that we just get it done in a quick PR and you can mark that as done over in the actual KEP issue in k/enhancements where it's being discussed.

After a long discussion with Sam I was swayed to agreement for the process where even for the easy stuff, if there is something to track in the KEP description/PR then we should open an issue in c/c. "just in case" For example someone may know of an issue with moving up to that version of runc that would delay merge.

@samuelkarp samuelkarp marked this pull request as draft March 26, 2025 17:01
This adds public documentation of the process for containerd project
collaboration with Kubernetes SIG-Node.  It is mostly a formalization of
what we have already been doing informally, but the formalization should
help with visibility and work distribution.

Relates to containerd/project#141

Signed-off-by: Samuel Karp <samuelkarp@google.com>
@samuelkarp samuelkarp force-pushed the sig-node-collaboration-process branch from f955528 to f0f25b6 Compare March 26, 2025 18:56
@samuelkarp samuelkarp marked this pull request as ready for review March 26, 2025 18:57
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

@AkihiroSuda AkihiroSuda merged commit eda165c into containerd:main Mar 27, 2025
2 checks passed
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.

5 participants