Revise page about multiple zones #23498

sftim · 2020-08-27T22:11:19Z

Revise https://kubernetes.io/docs/setup/best-practices/multiple-zones/ [preview]:

Drop provider-specific details, in line with current content guide. We're a long way past Kubernetes 1.2 as well, this page was rather stale.
General rewording.

Fixes #14964
Fixes #23982

netlify · 2020-08-27T22:25:41Z

Deploy preview for kubernetes-io-master-staging ready!

Built with commit 7196db6

https://deploy-preview-23498--kubernetes-io-master-staging.netlify.app

sftim · 2020-08-28T11:38:31Z

If this merges before PR #23466 then that other PR should ideally be revised before merging.

content/en/docs/setup/best-practices/multiple-zones.md

kbhawkey · 2020-09-18T19:19:38Z

@justinsb Would you have time to review?
@tengqm , Since you recently updated this page (zh) would you have time to review?
In general, things look good. Thanks!

content/en/docs/setup/best-practices/multiple-zones.md

sftim

Add suggestions based on feedback

content/en/docs/setup/best-practices/multiple-zones.md

sftim · 2020-09-22T11:38:32Z

/label tide/merge-method-squash

sftim · 2020-09-22T11:41:17Z

/remove-label tide/merge-method-squash

kbhawkey · 2020-09-22T13:29:15Z

content/en/docs/setup/best-practices/multiple-zones.md

+When you set up your cluster, you might also need to consider whether and how
+your setup can restore service if all of the failure zones in a region go
+off-line at the same time. For example, do you rely on there being at least
+Make sure that any cluster-critical repair work does not rely


@sftim. I read through the page again. There are a few places to edit.
Have a look at:
For example, do you rely on there being at least Make sure that any cluster-critical repair work does not rely

kbhawkey · 2020-09-22T13:31:18Z

content/en/docs/setup/best-practices/multiple-zones.md

+When persistent volumes are created, the `PersistentVolumeLabel`
+[admission controller](/docs/reference/access-authn-authz/admission-controllers/)
+automatically adds zone labels to any PersistentVolumes that are linked to a specific
+zone. The {< glossary_tooltip text="scheduler" term_id="kube-scheduler" >}} then ensures,


☝️ This tooltip is not rendering

kbhawkey · 2020-09-22T13:32:05Z

content/en/docs/setup/best-practices/multiple-zones.md

+## Networking
+
+Kubernetes itself does not provide zone-aware networking. You can use a
+[network plugin](docs/concepts/extend-kubernetes/compute-storage-net/network-plugins/)


nit: Kubernetes does not provide ...

kbhawkey · 2020-09-22T13:34:11Z

content/en/docs/setup/best-practices/multiple-zones.md

-The pods should be spread across all 3 zones:
+When nodes start up, the kubelet on each node automatically adds
+{{< glossary_tooltip text="labels" term_id="label" >}} to the the Node object
+that represents that specific kubelet in the Kubernetes API.


cleanup: the the
possible cleanup: s/Node object/Node/

kbhawkey · 2020-09-22T13:48:52Z

content/en/docs/setup/best-practices/multiple-zones.md

+are only placed into the same zone as that volume.
+
+You can specify a {{< glossary_tooltip text="StorageClass" term_id="storage-class" >}}
+for PersistentVolumeClaims that specifies the failure domains (zones) that the


nit wording/break up sentence. Would this work?

To learn about configuring a StorageClass with failure domains or zones, see Allowed topologies.

kbhawkey · 2020-09-22T13:54:16Z

content/en/docs/setup/best-practices/multiple-zones.md

-find kubernetes/examples/guestbook-go/ -name '*.json' | xargs -I {} kubectl apply -f {}
-```
+Kubernetes automatically spreads the Pods for a
+workload resources (such as {{< glossary_tooltip text="Deployment" term_id="deployment" >}}


cleanup: remove a in for a workload resources

kbhawkey · 2020-09-22T13:58:16Z

content/en/docs/setup/best-practices/multiple-zones.md

-EOF
-```
+When you deploy a cluster control plane you can place a replica of each
+component in each failure zone that you have used. For example, if you


nit: simplify:
When you deploy a cluster control plane, place a replica of each
control plane component in each failure zone.

kbhawkey · 2020-09-22T13:59:51Z

content/en/docs/setup/best-practices/multiple-zones.md

+in conjunction with
+[Pod topology spread constraints](/docs/concepts/workloads/pods/pod-topology-spread-constraints/)
+to control how Pods are spread across your cluster among fault domains.
+such as regions, zones, and even specific nodes.


Check the sentences (rebase mixup).
how Pods are spread across your cluster among fault domains. such as regions, zones, and even specific nodes.

kbhawkey · 2020-09-22T14:01:22Z

content/en/docs/setup/best-practices/multiple-zones.md

+{{< glossary_tooltip text="labels" term_id="label" >}} to the the Node object
+that represents that specific kubelet in the Kubernetes API.
+These labels can include
+[zone information](/docs/reference/kubernetes-api/labels-annotations-taints/#topologykubernetesiozone)


nit: Add period at end of sentence

Drop provider-specific details, in line with current content guide. Plus general rewording.

kbhawkey · 2020-10-07T18:53:04Z

The docs cleanup looks good.
/lgtm

k8s-ci-robot · 2020-10-07T18:53:20Z

LGTM label has been added.

Git tree hash: 8296e8400701a26d1f5e873ef39c4e8769f26f55

irvifa · 2020-10-08T06:11:39Z

/approve

k8s-ci-robot · 2020-10-08T06:12:01Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: irvifa

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

~~content/en/OWNERS~~ [irvifa]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 27, 2020

k8s-ci-robot requested review from justinsb and quinton-hoole August 27, 2020 22:12

k8s-ci-robot added language/en Issues or PRs related to English language sig/docs Categorizes an issue or PR as relevant to SIG Docs. labels Aug 27, 2020

sftim force-pushed the 20200827_revise_multiple_zones_setup branch from 465a245 to 4c2ebdc Compare September 6, 2020 15:39

kbhawkey reviewed Sep 18, 2020

View reviewed changes

content/en/docs/setup/best-practices/multiple-zones.md Outdated Show resolved Hide resolved

kbhawkey reviewed Sep 18, 2020

View reviewed changes

content/en/docs/setup/best-practices/multiple-zones.md Outdated Show resolved Hide resolved

kbhawkey reviewed Sep 18, 2020

View reviewed changes

content/en/docs/setup/best-practices/multiple-zones.md Outdated Show resolved Hide resolved

tengqm reviewed Sep 20, 2020

View reviewed changes

content/en/docs/setup/best-practices/multiple-zones.md Outdated Show resolved Hide resolved

sftim mentioned this pull request Sep 20, 2020

Running in multiple zones #23982

Closed

sftim commented Sep 22, 2020

View reviewed changes

k8s-ci-robot added the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Sep 22, 2020

sftim force-pushed the 20200827_revise_multiple_zones_setup branch from 7736683 to 0db4e5a Compare September 22, 2020 11:40

k8s-ci-robot removed the tide/merge-method-squash Denotes a PR that should be squashed by tide when it merges. label Sep 22, 2020

kbhawkey reviewed Sep 22, 2020

View reviewed changes

sftim mentioned this pull request Sep 28, 2020

new blog post: Custom K8s Scheduler for Highly Available Apps #24136

Merged

Revise page about multiple zones

7196db6

Drop provider-specific details, in line with current content guide. Plus general rewording.

sftim force-pushed the 20200827_revise_multiple_zones_setup branch from 0db4e5a to 7196db6 Compare October 7, 2020 11:19

k8s-ci-robot assigned kbhawkey Oct 7, 2020

k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 7, 2020

k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 8, 2020

k8s-ci-robot merged commit 56b0a2f into kubernetes:master Oct 8, 2020

sftim deleted the 20200827_revise_multiple_zones_setup branch September 12, 2021 10:50

s-kawamura-w664 mentioned this pull request Mar 10, 2023

[ja] Update page weights under setup and tutorials #39907

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Revise page about multiple zones #23498

Revise page about multiple zones #23498

sftim commented Aug 27, 2020 •

edited

Loading

netlify bot commented Aug 27, 2020 •

edited

Loading

sftim commented Aug 28, 2020

kbhawkey commented Sep 18, 2020

sftim left a comment

sftim commented Sep 22, 2020

sftim commented Sep 22, 2020

kbhawkey Sep 22, 2020

kbhawkey Sep 22, 2020

kbhawkey Sep 22, 2020

kbhawkey Sep 22, 2020

kbhawkey Sep 22, 2020

kbhawkey Sep 22, 2020

kbhawkey Sep 22, 2020

kbhawkey Sep 22, 2020

kbhawkey Sep 22, 2020

kbhawkey commented Oct 7, 2020

k8s-ci-robot commented Oct 7, 2020

irvifa commented Oct 8, 2020

k8s-ci-robot commented Oct 8, 2020

Revise page about multiple zones #23498

Revise page about multiple zones #23498

Conversation

sftim commented Aug 27, 2020 • edited Loading

netlify bot commented Aug 27, 2020 • edited Loading

sftim commented Aug 28, 2020

kbhawkey commented Sep 18, 2020

sftim left a comment

Choose a reason for hiding this comment

sftim commented Sep 22, 2020

sftim commented Sep 22, 2020

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

kbhawkey commented Oct 7, 2020

k8s-ci-robot commented Oct 7, 2020

irvifa commented Oct 8, 2020

k8s-ci-robot commented Oct 8, 2020

sftim commented Aug 27, 2020 •

edited

Loading

netlify bot commented Aug 27, 2020 •

edited

Loading