-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Redefine the Container Runtime Interface #54
Comments
Great, thanks for listing all these here! |
@dchen1107 @yujuhong can you clarify the actual status of the feature? |
Here is the current status and agreement sig-node team agreed upon for 1.5 release:
|
Can we add a link to the documentation for this issue? |
ping @dchen1107 @yujuhong |
@dchen1107 @yujuhong can you confirm that this item targets beta in 1.6? |
@yujuhong any update on this feature? Docs and release notes are required (please, provide them to the features spreadsheet. |
Pingity ping. Is there a Docs PR for this? Is one needed? |
I updated the spreadsheet with the release note and "no" for the doc a week ago. @idvoretskyi changed my answer to yes. Is there a reason for doing that? |
@yujuhong can you submit a documentation PR with a few words about the changes since the feature has been graduated to beta? |
It doesn't need a "doc" since it's not really a feature (it's really a refactoring). It does need more detailed release note. sig-node will update the release note to include the relevant bits. I also plan to send out an email to explain the impact of this change. |
I updated the tracking sheet for 1.6. Thanks. |
When do we close this? |
We made the docker-cri integration the default in 1.6. Closing this issue. |
does this imply CRI is now considered stable/GA? |
No. CRI as an internal API for kubelet and the built-in dockershim implementation is stable/GA; CRI as an external API for other runtimes is still Alpha. |
@yujuhong ok, so the docker integration is being promoted from beta in v1.6 to stable as part of v1.7? just thinking about this in the context of release notes |
@spiffxp ah I see. This issue is marked v1.6, so I didn't know you were asking about the release note for v1.7. |
d'oh, somehow I didn't notice this was v1.6 only, but yeah, probably worth pointing out for v1.7, thanks |
Remove James Russell (coverprice) from OWNERS
Description
We should separate image management, resource / cgroup management, logging management from today's runtime interface. After this effort, we effectively make today's runtime interface as pod / container lifecycle management interface. cc @yujuhong @dchen1107
Proposal here: kubernetes/kubernetes#25899
Progress Tracker
/pkg/apis/...
)FEATURE_STATUS is used for feature tracking and to be updated by @kubernetes/feature-reviewers.
FEATURE_STATUS: IN_DEVELOPMENT
More advice:
Design
Coding
and sometimes http://github.com/kubernetes/contrib, or other repos.
check that the code matches the proposed feature and design, and that everything is done, and that there is adequate
testing. They won't do detailed code review: that already happened when your PRs were reviewed.
When that is done, you can check this box and the reviewer will apply the "code-complete" label.
Docs
The text was updated successfully, but these errors were encountered: