From bd00f90564ba5f16e885a211a09e1a41602ff7c0 Mon Sep 17 00:00:00 2001
From: Andrey Senyaev <76472231+asenyaev@users.noreply.github.com>
Date: Wed, 27 Sep 2023 17:28:34 +0300
Subject: [PATCH] [stable/goldilocks] Turn off resource limits by default
 (#1329)

* [stable/goldilocks] Turn off resources by default

* [stable/goldilocks] Move back requests in resources by default

* [stable/goldilocks] Bump a version to 7.3.0
---
 stable/goldilocks/Chart.yaml  | 2 +-
 stable/goldilocks/README.md   | 4 ++--
 stable/goldilocks/values.yaml | 8 ++------
 3 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/stable/goldilocks/Chart.yaml b/stable/goldilocks/Chart.yaml
index 93630d4f7..8a2ab1a73 100644
--- a/stable/goldilocks/Chart.yaml
+++ b/stable/goldilocks/Chart.yaml
@@ -1,6 +1,6 @@
 apiVersion: v2
 appVersion: "v4.10.0"
-version: 7.2.0
+version: 7.3.0
 kubeVersion: ">= 1.22.0-0"
 description: |
   A Helm chart for running Fairwinds Goldilocks. See https://github.com/FairwindsOps/goldilocks
diff --git a/stable/goldilocks/README.md b/stable/goldilocks/README.md
index 37d916cf1..7d45b8f6a 100644
--- a/stable/goldilocks/README.md
+++ b/stable/goldilocks/README.md
@@ -84,7 +84,7 @@ This will completely remove the VPA and then re-install it using the new method.
 | controller.tolerations | list | `[]` | Tolerations for the controller pod |
 | controller.affinity | object | `{}` | Affinity for the controller pods |
 | controller.topologySpreadConstraints | list | `[]` | Topology spread constraints for the controller pods |
-| controller.resources | object | `{"limits":{"cpu":"25m","memory":"256Mi"},"requests":{"cpu":"25m","memory":"256Mi"}}` | The resources block for the controller pods |
+| controller.resources | object | `{"limits":{},"requests":{"cpu":"25m","memory":"256Mi"}}` | The resources block for the controller pods |
 | controller.podSecurityContext | object | `{}` | Defines the podSecurityContext for the controller pod |
 | controller.securityContext | object | `{"allowPrivilegeEscalation":false,"capabilities":{"drop":["ALL"]},"readOnlyRootFilesystem":true,"runAsNonRoot":true,"runAsUser":10324}` | The container securityContext for the controller container |
 | controller.deployment.extraVolumeMounts | list | `[]` | Extra volume mounts for the controller container |
@@ -118,7 +118,7 @@ This will completely remove the VPA and then re-install it using the new method.
 | dashboard.ingress.hosts[0].paths[0].path | string | `"/"` |  |
 | dashboard.ingress.hosts[0].paths[0].type | string | `"ImplementationSpecific"` |  |
 | dashboard.ingress.tls | list | `[]` |  |
-| dashboard.resources | object | `{"limits":{"cpu":"25m","memory":"256Mi"},"requests":{"cpu":"25m","memory":"256Mi"}}` | A resources block for the dashboard. |
+| dashboard.resources | object | `{"limits":{},"requests":{"cpu":"25m","memory":"256Mi"}}` | A resources block for the dashboard. |
 | dashboard.podSecurityContext | object | `{}` | Defines the podSecurityContext for the dashboard pod |
 | dashboard.securityContext | object | `{"allowPrivilegeEscalation":false,"capabilities":{"drop":["ALL"]},"readOnlyRootFilesystem":true,"runAsNonRoot":true,"runAsUser":10324}` | The container securityContext for the dashboard container |
 | dashboard.nodeSelector | object | `{}` |  |
diff --git a/stable/goldilocks/values.yaml b/stable/goldilocks/values.yaml
index 28879fc47..f50e929cf 100644
--- a/stable/goldilocks/values.yaml
+++ b/stable/goldilocks/values.yaml
@@ -61,9 +61,7 @@ controller:
   topologySpreadConstraints: []
   # controller.resources -- The resources block for the controller pods
   resources:
-    limits:
-      cpu: 25m
-      memory: 256Mi
+    limits: {}
     requests:
       cpu: 25m
       memory: 256Mi
@@ -160,9 +158,7 @@ dashboard:
 
   # dashboard.resources -- A resources block for the dashboard.
   resources:
-    limits:
-      cpu: 25m
-      memory: 256Mi
+    limits: {}
     requests:
       cpu: 25m
       memory: 256Mi