Merge pull request #39 from wandera/kube_api_limits

wandera · Jun 11, 2020 · a740469 · a740469
2 parents 149cd8a + 5979c68
commit a740469
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 0 deletions.
diff --git a/aws/cluster/cluster-spec-template.tf b/aws/cluster/cluster-spec-template.tf
@@ -11,6 +11,9 @@ data "template_file" "cluster-spec" {
     kube-dns-provider  = "${var.kube-dns-provider}"
     kops-state-bucket  = "${var.kops-state-bucket}"
 
+    controller-manager-kube-api-qps   = "${var.controller-manager-kube-api-qps}"
+    controller-manager-kube-api-burst = "${var.controller-manager-kube-api-burst}"
+
     master-lb-visibility     = "${var.master-lb-visibility == "Private" ? "Internal" : "Public"}"
     master-lb-dns-visibility = "${var.master-lb-visibility}"
     master-count             = "${length(var.master-availability-zones)}"

diff --git a/aws/cluster/templates/cluster-spec.yaml b/aws/cluster/templates/cluster-spec.yaml
@@ -62,6 +62,8 @@ ${featuregates-config}
     useServiceAccountCredentials: true
     horizontalPodAutoscalerSyncPeriod: ${hpa-sync-period}
     horizontalPodAutoscalerDownscaleStabilization: ${hpa-scale-downscale-stabilization}
+    kubeAPIQPS: ${controller-manager-kube-api-qps}
+    kubeAPIBurst: ${controller-manager-kube-api-burst}
     featureGates:
 ${featuregates-config}
   kubeDNS:

diff --git a/aws/cluster/variables.tf b/aws/cluster/variables.tf
@@ -286,6 +286,16 @@ variable "cloud-labels" {
   default = {}
 }
 
+variable "controller-manager-kube-api-qps" {
+  default     = 20
+  description = "Limit of queries per second to use when Kube Controller Manager is talking to Kubernetes API server."
+}
+
+variable "controller-manager-kube-api-burst" {
+  default     = 30
+  description = "Allowed burst in queries per second to use when Kube Controller Manager is talking to Kubernetes API server."
+}
+
 # Resource reservation on our nodes for Kubernetes daemons & the OS
 variable "kubelet-eviction-flag" {
   type        = "string"