-
Notifications
You must be signed in to change notification settings - Fork 38
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
Update README.md #155
Update README.md #155
Conversation
lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks.
@mikebrow I think you may need to sign-off the commit to satisfy the DCO requirement. |
Signed-off-by: Mike Brown <[email protected]>
a3bc0ce
to
75158b4
Compare
@@ -75,7 +75,7 @@ command to find them out: | |||
$ crio config |& grep -B1 -A5 cdi_spec_dirs | |||
``` | |||
|
|||
### Containerd configuration | |||
### containerd configuration | |||
|
|||
To enable and configure CDI support in the [containerd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how about adding? i ended up looking for this information to try out, which would be useful for other uses?
To enable and configure CDI support in the [containerd | |
[cotnainerd v1.7.0](https://github.com/containerd/containerd/releases/tag/v1.7.0) or later required. | |
To enable and configure CDI support in the [containerd |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe.. there is a chance we are going to backport support for CDI hook injection to the 1.6LTS branch, depending on alpha->beta promotion status in the upcoming k8s v1.29 release
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
merged bypassing the DCO check that had not run. Confirmed that the commit was signed-off. |
just a quick nit fix to lower case containerd