Skip to content
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

aws+gce+cinder storageclass, change zone texts if not specified #3693

Merged
merged 2 commits into from
May 10, 2017

Conversation

zetaab
Copy link
Member

@zetaab zetaab commented May 9, 2017

update aws+gce availability texts, the algorithm is not random

Doc change for kubernetes/kubernetes#44798

AWS + gce texts changed also because it is not really "random zone"

https://github.com/kubernetes/kubernetes/blob/master/pkg/volume/util.go#L295 "Volumes are generally round-robin-ed across all active zones"


This change is Reviewable

update aws+gce availability texts, the algorithm is not random
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 9, 2017
@zetaab zetaab changed the title cinder storageclass, zone algorithm if not specified aws+gce+cinder storageclass, change zone texts if not specified May 9, 2017
@chenopis chenopis requested a review from jsafrane May 9, 2017 18:00
@chenopis chenopis self-assigned this May 9, 2017
@jsafrane
Copy link
Member

lgtm from tech side

@pospispa
Copy link

lgtm

@chenopis chenopis merged commit c1b8664 into kubernetes:master May 10, 2017
pospispa pushed a commit to pospispa/openshift-docs that referenced this pull request May 12, 2017
…n Storage Class is not Configured

Backport of Kubernetes PR openshift#3693 (kubernetes/website#3693).

Corresponding source code OpenShift origin PR openshift#14159 (openshift/origin#14159).

Documents the Cinder volume plugin behaviour in case the availability parameter in a Storage Class is not configured.

Improves description of aws-ebs and gce-pd volume plugins behavior in case zone parameter in a Storage Class is not configured.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants