From 78998644092f89f70df55d37858368f21bb39776 Mon Sep 17 00:00:00 2001 From: Justin Santa Barbara Date: Mon, 23 Jan 2017 09:22:33 -0500 Subject: [PATCH] kube-dns autoscaler: set min replicas to 2 Issue https://github.com/kubernetes/kubernetes/issues/40063 Having a single pod would be a single point of failure. Multiple pods should be spread across AZs & nodes by k8s automatically. --- .../{v1.5.0.yaml.template => v1.5.1.yaml.template} | 2 +- upup/pkg/fi/cloudup/bootstrapchannelbuilder.go | 2 +- .../tests/bootstrapchannelbuilder/kopeio-vxlan/manifest.yaml | 4 ++-- .../tests/bootstrapchannelbuilder/simple/manifest.yaml | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) rename upup/models/cloudup/resources/addons/kube-dns.addons.k8s.io/{v1.5.0.yaml.template => v1.5.1.yaml.template} (99%) diff --git a/upup/models/cloudup/resources/addons/kube-dns.addons.k8s.io/v1.5.0.yaml.template b/upup/models/cloudup/resources/addons/kube-dns.addons.k8s.io/v1.5.1.yaml.template similarity index 99% rename from upup/models/cloudup/resources/addons/kube-dns.addons.k8s.io/v1.5.0.yaml.template rename to upup/models/cloudup/resources/addons/kube-dns.addons.k8s.io/v1.5.1.yaml.template index f95fe40947..09097ae754 100644 --- a/upup/models/cloudup/resources/addons/kube-dns.addons.k8s.io/v1.5.0.yaml.template +++ b/upup/models/cloudup/resources/addons/kube-dns.addons.k8s.io/v1.5.1.yaml.template @@ -46,7 +46,7 @@ spec: - --target=Deployment/kube-dns # When cluster is using large nodes(with more cores), "coresPerReplica" should dominate. # If using small nodes, "nodesPerReplica" should dominate. - - --default-params={"linear":{"coresPerReplica":256,"nodesPerReplica":16,"min":1}} + - --default-params={"linear":{"coresPerReplica":256,"nodesPerReplica":16,"min":2}} - --logtostderr=true - --v=2 diff --git a/upup/pkg/fi/cloudup/bootstrapchannelbuilder.go b/upup/pkg/fi/cloudup/bootstrapchannelbuilder.go index e99e63d85c..6a249cc973 100644 --- a/upup/pkg/fi/cloudup/bootstrapchannelbuilder.go +++ b/upup/pkg/fi/cloudup/bootstrapchannelbuilder.go @@ -85,7 +85,7 @@ func (b *BootstrapChannelBuilder) buildManifest() (*channelsapi.Addons, map[stri { key := "kube-dns.addons.k8s.io" - version := "1.5.0" + version := "1.5.1" location := key + "/v" + version + ".yaml" diff --git a/upup/pkg/fi/cloudup/tests/bootstrapchannelbuilder/kopeio-vxlan/manifest.yaml b/upup/pkg/fi/cloudup/tests/bootstrapchannelbuilder/kopeio-vxlan/manifest.yaml index 1d70617c53..e7233e0eb3 100644 --- a/upup/pkg/fi/cloudup/tests/bootstrapchannelbuilder/kopeio-vxlan/manifest.yaml +++ b/upup/pkg/fi/cloudup/tests/bootstrapchannelbuilder/kopeio-vxlan/manifest.yaml @@ -9,11 +9,11 @@ spec: selector: k8s-addon: core.addons.k8s.io version: 1.4.0 - - manifest: kube-dns.addons.k8s.io/v1.5.0.yaml + - manifest: kube-dns.addons.k8s.io/v1.5.1.yaml name: kube-dns.addons.k8s.io selector: k8s-addon: kube-dns.addons.k8s.io - version: 1.5.0 + version: 1.5.1 - manifest: limit-range.addons.k8s.io/v1.5.0.yaml name: limit-range.addons.k8s.io selector: diff --git a/upup/pkg/fi/cloudup/tests/bootstrapchannelbuilder/simple/manifest.yaml b/upup/pkg/fi/cloudup/tests/bootstrapchannelbuilder/simple/manifest.yaml index 4a6acff9f9..232c9db25f 100644 --- a/upup/pkg/fi/cloudup/tests/bootstrapchannelbuilder/simple/manifest.yaml +++ b/upup/pkg/fi/cloudup/tests/bootstrapchannelbuilder/simple/manifest.yaml @@ -9,11 +9,11 @@ spec: selector: k8s-addon: core.addons.k8s.io version: 1.4.0 - - manifest: kube-dns.addons.k8s.io/v1.5.0.yaml + - manifest: kube-dns.addons.k8s.io/v1.5.1.yaml name: kube-dns.addons.k8s.io selector: k8s-addon: kube-dns.addons.k8s.io - version: 1.5.0 + version: 1.5.1 - manifest: limit-range.addons.k8s.io/v1.5.0.yaml name: limit-range.addons.k8s.io selector: