-
Notifications
You must be signed in to change notification settings - Fork 14k
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 docs to promote PodReadyToStartContainersCondition into beta #43533
Conversation
👷 Deploy Preview for kubernetes-io-vnext-staging processing.
|
/sig node |
48e526f
to
d9bdf56
Compare
Hi @charles-chenzz , Please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday 14th November 2023. |
d9bdf56
to
c71a216
Compare
Thanks Please split the blog article into a separate PR that targets the main branch. /hold |
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.
You'll also need to change https://kubernetes.io/docs/concepts/workloads/pods/pod-lifecycle/ I think
I think it's better to change it when this feature promote to GA |
5a69e26
to
c71a216
Compare
/hold let’s address @sftim comments before we merge this. |
@charles-chenzz do you have some time Thursday or Friday to work on this? I think docs are due this week so it’s be nice to get this PR mergable |
I got time to work on this PR later today(16:00 in my timezone now, got some time in 20:00-22:00) and Friday. @kannon92 |
@@ -242,19 +242,21 @@ When a Pod's containers are Ready but at least one custom condition is missing o | |||
|
|||
### Pod network readiness {#pod-has-network} |
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.
@sftim wdyt about changing the name of this section? This was the name of the old condition and we renamed it to PodReadyToStartContainers.
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 “Pod sandbox readiness”, then?
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.
We can make that change after this PR lands.
Looking much better @charles-chenzz! Just one nit and then it LGTM. |
Hi, @charles-chenzz ! Kubernetes v1.29 Docs shadow here. Just a reminder that the deadline to have this PR reviewed and merged is Tuesday, 28 November. Let me know if you need any help! CCing KEP owners: @kannon92 |
Hi, @charles-chenzz! The deadline to have this PR merged is tomorrow. What can I do to help get this across the line as soon as possible? @sftim Do you feel that the changes you requested here have been addressed? |
/hold cancel (when adding a hold, it's a good idea to make it clear what conditions need to be met in order to unhold) |
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
LGTM label has been added. Git tree hash: 41d8a8b8140f3916f9c2da8d180ef8d477ab80d3
|
kubernetes/enhancements#4138