Merge pull request #21 from goci-io/psp

fix: do not set defaults when max value is set to 0
goci-io · Aug 1, 2020 · 74f951a · 74f951a
2 parents 640cd18 + ee4075b
commit 74f951a
Showing 1 changed file with 9 additions and 11 deletions.
diff --git a/limits.tf b/limits.tf
@@ -1,3 +1,8 @@
+locals {
+  container_cpu_default    = var.max_cpu == 0 ? {} : { cpu = "50m" }
+  container_memory_default = var.max_memory == 0 ? {} : { memory = "64Mi" }
+  storage_amount_default   = var.max_storage == 0 ? {} : { storage = "2Gi" }
+}
 
 resource "kubernetes_limit_range" "limits" {
   metadata {
@@ -7,24 +12,17 @@ resource "kubernetes_limit_range" "limits" {
 
   spec {
     limit {
-      type = "PersistentVolumeClaim"
-
-      default = {
-        storage = "2Gi"
-      }
+      type    = "PersistentVolumeClaim"
+      default = local.storage_amount_default
 
       max = {
         storage = var.max_storage
       }
     }
 
     limit {
-      type = "Container"
-
-      default = {
-        cpu    = "50m"
-        memory = "64Mi"
-      }
+      type    = "Container"
+      default = merge(local.container_cpu_default, local.container_memory_default)
 
       max = {
         cpu    = var.max_cpu