Add ContainersReady status for Pods#9197
Add ContainersReady status for Pods#9197k8s-ci-robot merged 1 commit intokubernetes:masterfrom lucperkins:lperkins/issue-9099-container-ready-status
Conversation
|
Deploy preview for kubernetes-io-master-staging ready! Built with commit 4d34ca3 https://deploy-preview-9197--kubernetes-io-master-staging.netlify.com |
|
Deploy preview for kubernetes-io-master-staging ready! Built with commit 1338ecb https://deploy-preview-9197--kubernetes-io-master-staging.netlify.com |
| due to an error in communicating with the host of the Pod. | ||
| Value | Description | ||
| :-----|:----------- | ||
| `ContainersReady` | All Containers in the Pod are currently ready |
There was a problem hiding this comment.
No. This is not a Phase, it is a Condition.
There was a problem hiding this comment.
Yep, you're right. I've fixed this and expanded the section on Pod conditions.
| 1. Get the Cassandra StatefulSet: | ||
|
|
||
| kubectl get statefulset cassandra | ||
| ```bash |
There was a problem hiding this comment.
This and the following indentations are breaking the numbered list. Please see preview result.
There was a problem hiding this comment.
The changes to this doc were included by accident. I've removed these changes from this PR.
|
/lgtm |
|
@lucperkins Thanks for taking a crack at this. Ultimately I'd like to see a more thorough pass for style guide consistency (for example, de-bullet and reorganize the Pod conditions and Example states sections), but this PR represents an iterative first step with formatting updates. 👍 /lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: zacharysarah The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This PR addresses issue #9099. This PR is currently a WIP while I wait on information regarding appropriate usage.