Skip to content

Commit

Permalink
[stable/goldilocks] Turn off resource limits by default (#1329)
Browse files Browse the repository at this point in the history
* [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
  • Loading branch information
asenyaev authored Sep 27, 2023
1 parent efbb784 commit bd00f90
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 9 deletions.
2 changes: 1 addition & 1 deletion stable/goldilocks/Chart.yaml
Original file line number Diff line number Diff line change
@@ -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
Expand Down
4 changes: 2 additions & 2 deletions stable/goldilocks/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -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 |
Expand Down Expand Up @@ -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 | `{}` | |
Expand Down
8 changes: 2 additions & 6 deletions stable/goldilocks/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand Down

0 comments on commit bd00f90

Please sign in to comment.