-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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/volumes.md to v1beta3 #8547
Conversation
@@ -3,17 +3,17 @@ This document describes the current state of Volumes in kubernetes. Familiarity | |||
|
|||
A Volume is a directory, possibly with some data in it, which is accessible to a Container. Kubernetes Volumes are similar to but not the same as [Docker Volumes](https://docs.docker.com/userguide/dockervolumes/). | |||
|
|||
A Pod specifies which Volumes its containers need in its [ContainerManifest](https://developers.google.com/compute/docs/containers/container_vms#container_manifest) property. | |||
A Pod specifies which Volumes its containers need in its [spec.volumes](http://kubernetes.io/third_party/swagger-ui/#!/v1beta3/createPod) property. |
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.
It's not spec.volumes but spec.containers[*].volumes
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.
No, it is spec.volumes. The doc also mentions the container's VolumeMounts property in the next paragraph.
Btw, is there any better place we can point people to other than the swagger-ui? swagger-ui currently takes very long time to load.
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.
It will be somewhat faster once we delete v1beta1 and v1beta2.
also update some details in the doc
Thanks @thockin. I've addressed your comments. Please take a look. We can add a section for persistent volumes in the future. |
LGTM |
Merging to shrink the backlog. |
update docs/volumes.md to v1beta3
update docs/volumes.md to v1beta3; also update some details in the doc
@erictune @thockin @nikhiljindal
ref #8087