Mention minimum etcd versions #10208

erictune · 2018-09-06T15:36:25Z

Source: https://groups.google.com/d/msg/kubernetes-dev/jMPA4JzKiY4/HIx2ugvLBAAJ

erictune · 2018-09-06T15:37:13Z

/assign @jpbetz

k8sio-netlify-preview-bot · 2018-09-06T15:40:20Z

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

Built with commit 79e6df1

https://deploy-preview-10208--kubernetes-io-master-staging.netlify.com

k8sio-netlify-preview-bot · 2018-09-06T15:40:55Z

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

Built with commit 7c2bf92

https://deploy-preview-10208--kubernetes-io-master-staging.netlify.com

jpbetz

/lgtm

Thanks for finding a place in the docs for this!

wojtek-t · 2018-09-06T18:03:40Z

content/en/docs/tasks/administer-cluster/configure-upgrade-etcd.md

+* The minimum recommended versions of etcd to run in production are:
+  * `3.1.11+`
+  * `3.2.10+`
+  * `3.3.0+`


Should we also mention k8s releases in which those versions are recommended?

E.g. 3.2.10+ from k8s releases 1.10+ (IIRC)

@jpbetz - thoughts?

Yes, maybe something like:

The minimum recommended versions of etcd to run in production are: * etcd `3.1.11+` for Kubernetes 1.8-1.9 * etcd `3.2.10+` for Kubernetes 1.10+

?

erictune · 2018-09-06T18:24:11Z

Docs in master would refer to 1.10+. We would typically not document previous version requirements in master, AIUI. I can remove the mention of 3.1.11+ if you like.

…

On Thu, Sep 6, 2018 at 11:19 AM Joe Betz ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In content/en/docs/tasks/administer-cluster/configure-upgrade-etcd.md <#10208 (comment)>: > @@ -34,6 +34,12 @@ content_template: templates/task * Keeping stable etcd clusters is critical to the stability of Kubernetes clusters. Therefore, run etcd clusters on dedicated machines or isolated environments for [guaranteed resource requirements](https://github.com/coreos/etcd/blob/master/Documentation/op-guide/hardware.md#hardware-recommendations). +* The minimum recommended versions of etcd to run in production are: + * `3.1.11+` + * `3.2.10+` + * `3.3.0+` Yes, maybe something like: The minimum recommended versions of etcd to run in production are: * etcd `3.1.11+` for Kubernetes 1.8-1.9 * etcd `3.2.10+` for Kubernetes 1.10+ ? — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#10208 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHuudvEz1elVMPeGKgT3pMqc9ylR49Vxks5uYWckgaJpZM4WdMvI> .

jpbetz · 2018-09-06T18:27:08Z

Either that or something like this would be fine:

The minimum recommended versions of etcd to run in production are:
  * `3.1.11+`
  * `3.2.10+` (preferred)

wojtek-t · 2018-09-06T18:28:32Z

+1 for the above proposal by @jpbetz

MikeSpreitzer · 2018-09-06T19:58:25Z

Thanks for documenting this.

erictune · 2018-09-07T17:38:50Z

@jpbetz PTAL, not sure I captured what you asked for exactly right. This is documenting for 1.11.

jpbetz · 2018-09-07T18:16:59Z

For 1.11 our main test pairing was k8s 1.11/etcd 3.2, so I'd like to shy away from any claims of 3.3 support. We officially claim backward compatibility for all 3.x etcd, and even 2.x (with a deprecation deadline coming up this year). Maybe we should just include the 3.2 version here since this is about what we're recommending (as opposed to supporting) and 3.2 is the only version we have a strong direct test signal for for 1.11.

jpbetz · 2018-09-07T18:18:02Z

content/en/docs/tasks/administer-cluster/configure-upgrade-etcd.md

@@ -388,6 +392,8 @@ TARGET_VERSION=2.2.1
 STORAGE_MEDIA_TYPE=application/json
 ```

+** Note: this upgrade procedure is for getting from 2.x to 3.x.  Version `3.0.17` is not recommended for running in production (see [prerequisites](#prereqisites) for minimum recommended etcd versions.


Looks great!

Why single out 3.0.17? The guidance I had been seeing previously said just get it above 3.0.10. https://groups.google.com/forum/#!topic/etcd-dev/nZQl17RjxHQ dumps on all of 3.0 --- and more emphatically than I am seeing here. Somewhere there should be maintained an evergreen doc for general etcd use like https://groups.google.com/forum/#!topic/etcd-dev/nZQl17RjxHQ

Just saying 3.0 here sounds fine.

@MikeSpreitzer For the evergreen doc, that would be for the etcd docs? We should probably move discussion to github.com/etcd-io/etcd.

Yes, @jpbetz , I think the etcd docs should include a maintained statement like what you sent out in email.

I don't want to change the tested upgrade procedure.
I updated the advice to not single out 3.0.17, and just suggest further upgrading to the recommended version of etcd.

Source: https://groups.google.com/d/msg/kubernetes-dev/jMPA4JzKiY4/HIx2ugvLBAAJ

erictune · 2018-09-07T18:21:50Z

Updated to recommend only 3.2.10+.

…

On Fri, Sep 7, 2018 at 11:18 AM Joe Betz ***@***.***> wrote: ***@***.**** commented on this pull request. ------------------------------ In content/en/docs/tasks/administer-cluster/configure-upgrade-etcd.md <#10208 (comment)>: > @@ -388,6 +392,8 @@ TARGET_VERSION=2.2.1 STORAGE_MEDIA_TYPE=application/json ``` +** Note: this upgrade procedure is for getting from 2.x to 3.x. Version `3.0.17` is not recommended for running in production (see [prerequisites](#prereqisites) for minimum recommended etcd versions. Looks great! — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#10208 (review)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHuudtxrWC5AeWA_-xXATfZsUZXCKTrzks5uYrhdgaJpZM4WdMvI> .

jpbetz · 2018-09-07T18:54:04Z

/lgtm

wojtek-t · 2018-09-10T08:29:43Z

lgtm

tengqm · 2018-09-10T09:16:15Z

/approve

k8s-ci-robot · 2018-09-10T09:16:25Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: tengqm

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:

~~OWNERS~~ [tengqm]

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

* fix invalid href of cloud controller manager (#10240) * fix invalid yaml format (#10238) * update storage-limits doc with Azure disk part (#10224) update storage-limits doc with Azure disk part fix comments * Update kubelet-config-file.md (#10222) Update link to KubeletConfiguration struct. * fix a trivial misspelling (#10244) * Fix cassandra-statefulset.yaml indent level (#10243) * Mention minimum etcd versions (#10208) Source: https://groups.google.com/d/msg/kubernetes-dev/jMPA4JzKiY4/HIx2ugvLBAAJ * fix 404 error (#10250) * Small verb tweak (#10190) Present participle, ftw. * Add AnchorJS logic for header links (#10155) * Add AnchorJS JavaScript * Remove existing inpage_heading logic * Remove underline from anchor tags * Use single icon and add touch visibility * Use paragraph link icon for AnchorJS * Update Sass to use code formatting in docsContent headers * Update header size coverage to H3-H6 * fix broken link in kubefed.md (#10254) * Update the version numbers for the X-Remote-Extra- and Impersonate-Extra- key fixes (#9827) The fix was cherry picked into 1.11.3, 1.10.7, and 1.9.11: kubernetes/kubernetes#67162 kubernetes/kubernetes#67163 kubernetes/kubernetes#67164 * fix typo (#10168) * fix typo * addressing comments. * Update setup-ha-etcd-with-kubeadm.md * fix typos (#10252) * fix description of contribute guide (#10253) * describe truncate feature about advanced audit (#10236) * describe truncate feature about advanced audit * Update audit.md

zacharysarah · 2018-09-12T20:34:35Z

content/en/docs/setup/scratch.md

 We recommend that you use the etcd version which is provided in the Kubernetes binary distribution.   The Kubernetes binaries in the release
 were tested extensively with this version of etcd and not with any other version.
 The recommended version number can also be found as the value of `TAG` in `kubernetes/cluster/images/etcd/Makefile`.

+For the miniumum recommended version of etcd, please refer to


@erictune 👋 Thanks for making these updates! For future reference, omit "please" per the style guide.

/cc @tengqm

* Update docs for fields allowed at root of CRD schema (#9973) * add plugin docs and examples (#10053) * docs update to promote TaintNodesByCondition to beta (#9626) * HPA Specificity Improvements (#8757) Updated the HPA docs to reference the `autoscaling/v2beta2` API version, and added documentation about the new fields. * adjust docs for pod ready++ (#10049) * Remove --cadvisor-port - has been deprecated since v1.10 (#10023) Change-Id: Id2a685473a243aef492a98ff450759f39e362557 * Add Documentation for Snapshot Feature (#9948) * Add documentation for snapshot feature * Update volume-snapshots.md * Add dry-run to api-concepts (#10033) * kubeadm-init: Update the offline support section (#10062) The update includes the following things (in mind with Kubernetes 1.12): - Remove the 1.8 image versions - Add the 1.10 image versions that were missing until now - Include a comment for the missing arch suffixes in 1.12 Signed-off-by: Rostislav M. Georgiev <rostislavg@vmware.com> * Say bye to `DynamicProvisioningScheduling` (#10157) The mentioned feature gate is now collapsed into `VolumeScheduling`. xref: kubernetes/kubernetes#67432 * Update ResourceQuota per PriorityClass state for 1.12 (#10229) * TokenRequest and TokenRequestProjection now beta (#10161) xref: kubernetes/kubernetes#67349 * Change feature state for kms provider to beta. (#10230) KMS Provider will be graduating to beta in v1.12, reflecting this change on the website. * coredns default (#10200) * Promote ShareProcessNamespace to beta in docs (#9996) * Add CoreDNS details to DNS Debug docs (#10201) * add coredns details * address nits, add query logging section * Update docs with topology aware dynamic provisioning (#9939) * Document topology aware volume binding feature * update for readability * Update storage-classes.md * comma splice * don't abbreviate * HPA Algorithm Information Improvements (#9780) * Update HPA docs with more algorithm details The HPA docs pointed to an out-of-date document for information on the algorithm details, which users were finding confusing. This sticks a section on the algorithm in the HPA docs instead, documenting both general behavior and corner cases. * Add glossary info, HPA docs on quantities People often ask about the quantity notation when working with the metrics APIs, so this adds a glossary entry on quantities (since they're used elsewhere in the system), and a short explantation in the HPA walkthough. * Information about HPA readiness and stabilization This adds information about the new changes to HPA readiness and stabilization from kubernetes/enhancements#591, and other minor changes that landed in Kubernetes 1.12. * Update horizontal-pod-autoscale.md * Audit 1.12 doc (#9953) * audit 1.12 document * remove legacy audit feature kubernetes/kubernetes#65862 * update feature gate doc * MountPropagation is now GA (#10090) * RuntimeClass documentation (#10102) * RuntimeClass documentation * Update runtime-class.md * Add documentation for Scheduler performance tuning (#10048) * Add documentation for Scheduler performance tuning * Update scheduler-perf-tuning.md * TTL controller for cleaning up finished resources (#10064) * TTL controller for cleaning up finished resources * Address comments * Update ttlafterfinished.md * Bump quota configuration api version (#10217) * Incremental update from master (#10278) * fix invalid href of cloud controller manager (#10240) * fix invalid yaml format (#10238) * update storage-limits doc with Azure disk part (#10224) update storage-limits doc with Azure disk part fix comments * Update kubelet-config-file.md (#10222) Update link to KubeletConfiguration struct. * fix a trivial misspelling (#10244) * Fix cassandra-statefulset.yaml indent level (#10243) * Mention minimum etcd versions (#10208) Source: https://groups.google.com/d/msg/kubernetes-dev/jMPA4JzKiY4/HIx2ugvLBAAJ * fix 404 error (#10250) * Small verb tweak (#10190) Present participle, ftw. * Add AnchorJS logic for header links (#10155) * Add AnchorJS JavaScript * Remove existing inpage_heading logic * Remove underline from anchor tags * Use single icon and add touch visibility * Use paragraph link icon for AnchorJS * Update Sass to use code formatting in docsContent headers * Update header size coverage to H3-H6 * fix broken link in kubefed.md (#10254) * Update the version numbers for the X-Remote-Extra- and Impersonate-Extra- key fixes (#9827) The fix was cherry picked into 1.11.3, 1.10.7, and 1.9.11: kubernetes/kubernetes#67162 kubernetes/kubernetes#67163 kubernetes/kubernetes#67164 * fix typo (#10168) * fix typo * addressing comments. * Update setup-ha-etcd-with-kubeadm.md * fix typos (#10252) * fix description of contribute guide (#10253) * describe truncate feature about advanced audit (#10236) * describe truncate feature about advanced audit * Update audit.md * docs update to promote ScheduleDaemonSetPods to beta (#9923) * Dynamic volume limit updates for 1.12 (#10211) * add a placeholder commit * Update docs for csi volume limits * Update storage-limits.md * Add "MayRunAs" value among other GroupStrategies (#9888) * Add CoreDNS details to the customize DNS doc (#10228) * Add CoreDNS details to the customize DNS doc Rewrite the document to include more details about CoreDNS, since it's now the default from v1.12 * Address comments * Improve doc wording * Fix link * Update dns-custom-nameservers.md * Update dns-custom-nameservers.md * Fix secrets docs in 1.12 branch (#10056) * Fix secrets docs * Update secret.md * Revert CoreDNS Docs (#10319) * Revert "Add CoreDNS details to DNS Debug docs (#10201)" This reverts commit 462817a. * Revert "Add CoreDNS details to the customize DNS doc (#10228)" This reverts commit e7319ee. * Revert "coredns default (#10200)" This reverts commit 698e93b. * Add CRI installation instructions page Added cri-installation page with CRI installation instructions Referenced it from kubeadm-init and install-kubeadm pages. * kubeadm: update API types documentation for 1.12 (#10283) v1alpha2 -> v1alpha3 MasterConfiguration -> [new-api-types] * TokenRequest feature documentation (#10295) * AdvancedAuditing is now GA (#10156) xref: kubernetes/kubernetes#65862 `AdvancedAuditing` feature is GA in 1.12. This PR adjusts the related docs. * update runtime-class.md (#10332) * update runtime-class.md * Update runtime-class.md * Document cross-authorizer permissions for creating RBAC roles (#10015) * Document cross-authorizer permissions for creating RBAC roles * Update rbac.md * kubeadm: update authored content for 1.12 (reference docs and cluster creation) (#10348) * kubeadm: update authored content in reference docs for 1.12 * kubeadm: add time frame in create-cluster-kubeadm for 1.12 * add AllowedProcMountTypes and ProcMountType to docs (#9911) Signed-off-by: Jess Frazelle <acidburn@microsoft.com> * kubeadm: add new command line reference (#10306) Add: - placeholder files - include place holder files - include "renew" sub command - add missing tabs for "alpha phase kubelet" * Documenting SCTP support in Kubernetes (#10279) * Documenting SCTP support in Kubernetes Service, Endpoint, NetworkPolicy and Pod * Updates based on comments on the PR * kubectl expose update with SCTP support * Updated according to comments in the PR * Revert "kubectl expose update with SCTP support" This reverts commit 0d5a1e6. * TLS Bootstrap and Server Cert Rotation feature documentation (#10232) * TokenRequest feature documentation * line wrapping to make review not insane * update content for GA without major refactor * Update kubelet-tls-bootstrapping.md * Add clarifications for volume snapshots (#10296) * Update kubadm ha installation for 1.12 (#10264) * Update kubadm ha installation for 1.12 Signed-off-by: Chuck Ha <ha.chuck@gmail.com> * update stable version Signed-off-by: Chuck Ha <ha.chuck@gmail.com> * Update stacked control plane for v1.12 (#2) * use v1alpha3 Signed-off-by: Chuck Ha <ha.chuck@gmail.com> * more v1alpha3 (#4) * updates Signed-off-by: Chuck Ha <ha.chuck@gmail.com> * Document how to run in-tree cloud providers with kubeadm (#10357) Change-Id: Iab6b996a830503d74a6eb0c507c5f8ca7a39235b * kubeadm reference doc for release 1.12 (#10359) * Revert "Revert "Add CoreDNS details to DNS Debug docs (#10201)"" This reverts commit bb30f4d. * Revert "Revert "Add CoreDNS details to the customize DNS doc (#10228)"" This reverts commit bc23d45. * Revert "Revert "coredns default (#10200)"" This reverts commit 7f4350d. * add missing instruction for ha guide (#10374) Signed-off-by: Chuck Ha <ha.chuck@gmail.com> * kubeadm - Ha upgrade updates (#10340) * Update HA upgrade docs * Adds external etcd HA upgrade guide Signed-off-by: Chuck Ha <ha.chuck@gmail.com> * copyedit * more edits * add runasgroup in psp (#10076) * update KubeletPluginsWatcher feature gate (#10205) * generated 1.12 docs * Building Multi-arch images with Manifests (#10379) In 1.12, a variety of images used in a typical kubernetes installation have started to using manifests to better support environments with arm or ppc64le architectures. For example all images used with kubeadm by default have manifests, another would be all the tests in the conformance test suite. Here we capture the best practices for everyone to start using manifests in their own workflows. Change-Id: I5ba4c5fe55ffc9486a8251760f3352be4f2e1494 * Upgrade docs for v1.12 (#10344) * generated assets and docs * remove 1.7 * update 1.12 * update plugin documentation under docs>tasks>extend-kubectl (#10259) * update plugin documentation under docs>tasks>extend-kubectl * Update kubectl-plugins.md

k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Sep 6, 2018

k8s-ci-robot requested review from mml and wojtek-t September 6, 2018 15:36

k8s-ci-robot assigned jpbetz Sep 6, 2018

jpbetz approved these changes Sep 6, 2018

View reviewed changes

wojtek-t reviewed Sep 6, 2018

View reviewed changes

erictune force-pushed the patch-1 branch from 79e6df1 to 68d102b Compare September 7, 2018 17:36

erictune force-pushed the patch-1 branch from 68d102b to 9eb2b87 Compare September 7, 2018 18:00

k8s-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Sep 7, 2018

erictune mentioned this pull request Sep 7, 2018

Mention minimum etcd versions #10207

Closed

jpbetz reviewed Sep 7, 2018

View reviewed changes

Mention minimum etcd versions

7c2bf92

Source: https://groups.google.com/d/msg/kubernetes-dev/jMPA4JzKiY4/HIx2ugvLBAAJ

erictune force-pushed the patch-1 branch from 9eb2b87 to 7c2bf92 Compare September 7, 2018 18:20

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

k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 10, 2018

k8s-ci-robot merged commit 726ce50 into kubernetes:master Sep 10, 2018

zacharysarah reviewed Sep 12, 2018

View reviewed changes

k8s-ci-robot requested a review from tengqm September 12, 2018 20:34

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Mention minimum etcd versions #10208

Mention minimum etcd versions #10208

erictune commented Sep 6, 2018 •

edited

Loading

erictune commented Sep 6, 2018

k8sio-netlify-preview-bot commented Sep 6, 2018

k8sio-netlify-preview-bot commented Sep 6, 2018 •

edited

Loading

jpbetz left a comment •

edited

Loading

wojtek-t Sep 6, 2018

jpbetz Sep 6, 2018

erictune commented Sep 6, 2018 via email

jpbetz commented Sep 6, 2018

wojtek-t commented Sep 6, 2018

MikeSpreitzer commented Sep 6, 2018

erictune commented Sep 7, 2018

jpbetz commented Sep 7, 2018

jpbetz Sep 7, 2018

MikeSpreitzer Sep 7, 2018

jpbetz Sep 7, 2018

MikeSpreitzer Sep 7, 2018

erictune Sep 7, 2018

erictune commented Sep 7, 2018 via email

jpbetz commented Sep 7, 2018

wojtek-t commented Sep 10, 2018

tengqm commented Sep 10, 2018

k8s-ci-robot commented Sep 10, 2018

zacharysarah Sep 12, 2018

Mention minimum etcd versions #10208

Mention minimum etcd versions #10208

Conversation

erictune commented Sep 6, 2018 • edited Loading

erictune commented Sep 6, 2018

k8sio-netlify-preview-bot commented Sep 6, 2018

k8sio-netlify-preview-bot commented Sep 6, 2018 • edited Loading

jpbetz left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

erictune commented Sep 6, 2018 via email

jpbetz commented Sep 6, 2018

wojtek-t commented Sep 6, 2018

MikeSpreitzer commented Sep 6, 2018

erictune commented Sep 7, 2018

jpbetz commented Sep 7, 2018

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

erictune commented Sep 7, 2018 via email

jpbetz commented Sep 7, 2018

wojtek-t commented Sep 10, 2018

tengqm commented Sep 10, 2018

k8s-ci-robot commented Sep 10, 2018

Choose a reason for hiding this comment

erictune commented Sep 6, 2018 •

edited

Loading

k8sio-netlify-preview-bot commented Sep 6, 2018 •

edited

Loading

jpbetz left a comment •

edited

Loading