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

kata-deploy: Add drop-in conf support for containerd #9499

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

zvonkok
Copy link
Contributor

@zvonkok zvonkok commented Apr 17, 2024

Upcoming containerd 2.0 will have drop-in conf support. First patch to get this functionality into our stack.

Copy link
Member

@c3d c3d left a comment

Choose a reason for hiding this comment

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

Except for a minor nit, looks good to me.

@sprt
Copy link
Contributor

sprt commented Apr 17, 2024

Do you have a reference for this? Would like to learn more about this containerd feature.

@zvonkok
Copy link
Contributor Author

zvonkok commented Apr 18, 2024

@sprt There is some work being done on the proper merge behavior: containerd/containerd#9982

Copy link
Member

@BbolroC BbolroC left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @zvonkok !

Upcoming containerd 2.0 will have drop-in conf support.
First patch to get this functionality into our stack.

Signed-off-by: Zvonko Kaiser <zkaiser@nvidia.com>
@fidencio
Copy link
Member

fidencio commented May 6, 2024

Upcoming containerd 2.0 will have drop-in conf support. First patch to get this functionality into our stack.

One question here, you want to let the user decide what to do?
Can we check for whether user is running 1.7 or higher (as this was actually introduced in 1.7), and then just manually add the imports (in case those are not there), and drop-in the files regardless?

Another thing that's crossing my mind is how we differentiate v1 / v2 / v3 when there's no configuration set, and drop-ins are used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test size/tiny Smallest and simplest task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants