QPS changes (#58)

Increase default kube-api-qps to 20 and also bump kube-api-burst to 40
platform9 · Sep 7, 2018 · be6d2d7 · be6d2d7
1 parent 76edce5
commit be6d2d7
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/constants/constants.go b/constants/constants.go
@@ -52,13 +52,15 @@ var CNIPluginsFilename = fmt.Sprintf("cni-plugins-amd64-%s.tgz", CNIVersion)
 
 const (
 	// TODO(dlipovetsky) Move fields to configuration
-	KubeletFailSwapOn = false
-	KubeletMaxPods    = 500
+	KubeletFailSwapOn   = false
+	KubeletMaxPods      = 500
+	KubeletKubeAPIQPS   = 20
+	KubeletKubeAPIBurst = 40
 
 	NodeadmKubeletSystemdDropinFilename = "20-nodeadm.conf"
 	NodeadmKubeletSystemdDropinTemplate = `[Service]
 Environment="KUBELET_DNS_ARGS=--cluster-dns={{ .ClusterDNS }} --cluster-domain={{ .ClusterDomain }}"
-Environment="KUBELET_EXTRA_ARGS=--max-pods={{ .MaxPods }} --fail-swap-on={{ .FailSwapOn }} --hostname-override={{ .HostnameOverride }}"
+Environment="KUBELET_EXTRA_ARGS=--max-pods={{ .MaxPods }} --fail-swap-on={{ .FailSwapOn }} --hostname-override={{ .HostnameOverride }} --kube-api-qps={{ .KubeAPIQPS }} --kube-api-burst={{ .KubeAPIBurst }}"
 `
 )
 

diff --git a/utils/install.go b/utils/install.go
@@ -80,12 +80,16 @@ func placeAndModifyNodeadmKubeletSystemdDropin(netConfig apis.Networking) {
 		ClusterDNS       string
 		ClusterDomain    string
 		HostnameOverride string
+		KubeAPIQPS       int
+		KubeAPIBurst     int
 	}{
 		FailSwapOn:       constants.KubeletFailSwapOn,
 		MaxPods:          constants.KubeletMaxPods,
 		ClusterDNS:       dnsIP.String(),
 		ClusterDomain:    netConfig.DNSDomain,
 		HostnameOverride: hostnameOverride,
+		KubeAPIQPS:       constants.KubeletKubeAPIQPS,
+		KubeAPIBurst:     constants.KubeletKubeAPIBurst,
 	}
 
 	writeTemplateIntoFile(constants.NodeadmKubeletSystemdDropinTemplate, "nodeadm-kubelet-systemd-dropin", confFile, data)