-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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 pods user guide to reflect StatefulSet changes #1718
Update pods user guide to reflect StatefulSet changes #1718
Conversation
@@ -135,9 +135,9 @@ simplified management. | |||
|
|||
## Durability of pods (or lack thereof) | |||
|
|||
Pods aren't intended to be treated as durable [pets](https://blog.engineyard.com/2014/pets-vs-cattle). They won't survive scheduling failures, node failures, or other evictions, such as due to lack of resources, or in the case of node maintenance. | |||
Pods aren't intended to be treated as durable [instances](https://blog.engineyard.com/2014/pets-vs-cattle). They won't survive scheduling failures, node failures, or other evictions, such as due to lack of resources, or in the case of node maintenance. |
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.
The blog link was there to explain the "pet" metaphor. Since we're moving away from using that metaphor, does it still make sense to link the blog?
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.
I agree, the blog link is no longer as relevant, and maybe "entities" rather than "instances" due the the latter being an overloaded term?
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.
Agree. Removed the link and changed to "entities"
LGTM |
This is fine for now; keep in mind that Pod's guide is becoming the Pod concept documentation in the very near future. I'll be sure to track these changes in Pod concept. |
Marking as "Open Issues" for now for the Todo links, once we get the targets merged. Otherwise LGTM. |
Updated by removing the obsolete TODO links. |
cc @erictune @foxish @kow3ns @enisoc @kubernetes/sig-apps
This change is