From 00745a04b2e3caffd7b3e06388c4dfe0df96006f Mon Sep 17 00:00:00 2001 From: Lukas Hankeln Date: Thu, 9 Jul 2026 07:20:03 +0200 Subject: [PATCH] feat(goldilocks): adding extra env values and update vpa to support extraenv Signed-off-by: Lukas Hankeln --- stable/goldilocks/Chart.yaml | 4 ++-- stable/goldilocks/README.md | 2 ++ stable/goldilocks/templates/controller-deployment.yaml | 4 ++++ stable/goldilocks/templates/dashboard-deployment.yaml | 4 ++++ stable/goldilocks/values.yaml | 4 ++++ 5 files changed, 16 insertions(+), 2 deletions(-) diff --git a/stable/goldilocks/Chart.yaml b/stable/goldilocks/Chart.yaml index 15c4bdffd..475f8f7ff 100644 --- a/stable/goldilocks/Chart.yaml +++ b/stable/goldilocks/Chart.yaml @@ -1,6 +1,6 @@ apiVersion: v2 appVersion: "v4.14.1" -version: 10.4.1 +version: 10.5.0 kubeVersion: ">= 1.22.0-0" description: | A Helm chart for running Fairwinds Goldilocks. See https://github.com/FairwindsOps/goldilocks @@ -16,7 +16,7 @@ keywords: - kubernetes dependencies: - name: vpa - version: 4.12.* + version: 4.13.* repository: https://charts.fairwinds.com/stable condition: vpa.enabled - name: metrics-server diff --git a/stable/goldilocks/README.md b/stable/goldilocks/README.md index 5ed3794ea..fb583fe3c 100644 --- a/stable/goldilocks/README.md +++ b/stable/goldilocks/README.md @@ -82,6 +82,7 @@ This will completely remove the VPA and then re-install it using the new method. | controller.rbac.extraClusterRoleBindings | list | `[]` | A list of ClusterRoles for which ClusterRoleBindings will be created for the ServiceAccount, if enabled | | controller.serviceAccount.create | bool | `true` | If true, a service account will be created for the controller. If set to false, you must set `controller.serviceAccount.name` | | controller.serviceAccount.name | string | `nil` | The name of an existing service account to use for the controller. Combined with `controller.serviceAccount.create` | +| controller.extraEnv | list | `[]` | Extra environment variables to add to the controller container | | controller.flags | object | `{}` | A map of additional flags to pass to the controller. For monitoring all namespaces out of the box, add the following flag "on-by-default: true" | | controller.logVerbosity | string | `"2"` | Controller log verbosity. Can be set from 1-10 with 10 being extremely verbose | | controller.nodeSelector | object | `{}` | Node selector for the controller pod | @@ -103,6 +104,7 @@ This will completely remove the VPA and then re-install it using the new method. | dashboard.service.type | string | `"ClusterIP"` | The type of the dashboard service | | dashboard.service.port | int | `80` | The port to run the dashboard service on | | dashboard.service.annotations | object | `{}` | Extra annotations for the dashboard service | +| dashboard.extraEnv | list | `[]` | Extra environment variables to add to the dashboard container | | dashboard.flags | object | `{}` | A map of additional flags to pass to the dashboard. For monitoring all namespaces out of the box, add the following flag "on-by-default: true". | | dashboard.logVerbosity | string | `"2"` | Dashboard log verbosity. Can be set from 1-10 with 10 being extremely verbose | | dashboard.excludeContainers | string | `"linkerd-proxy,istio-proxy"` | Container names to exclude from displaying in the Goldilocks dashboard | diff --git a/stable/goldilocks/templates/controller-deployment.yaml b/stable/goldilocks/templates/controller-deployment.yaml index c0a1fddf6..c47b0896f 100644 --- a/stable/goldilocks/templates/controller-deployment.yaml +++ b/stable/goldilocks/templates/controller-deployment.yaml @@ -61,6 +61,10 @@ spec: securityContext: {{- toYaml .Values.controller.securityContext | nindent 12 }} {{- end }} + {{- with .Values.controller.extraEnv }} + env: + {{- toYaml . | nindent 12 }} + {{- end }} resources: {{- toYaml .Values.controller.resources | nindent 12 }} {{- if .Values.controller.deployment.extraVolumeMounts }} diff --git a/stable/goldilocks/templates/dashboard-deployment.yaml b/stable/goldilocks/templates/dashboard-deployment.yaml index 43e050c02..91ea10901 100644 --- a/stable/goldilocks/templates/dashboard-deployment.yaml +++ b/stable/goldilocks/templates/dashboard-deployment.yaml @@ -66,6 +66,10 @@ spec: securityContext: {{- toYaml .Values.dashboard.securityContext | nindent 12 }} {{- end }} + {{- with .Values.dashboard.extraEnv }} + env: + {{- toYaml . | nindent 12 }} + {{- end }} ports: - name: http containerPort: 8080 diff --git a/stable/goldilocks/values.yaml b/stable/goldilocks/values.yaml index fb850f6fe..e154ab177 100644 --- a/stable/goldilocks/values.yaml +++ b/stable/goldilocks/values.yaml @@ -46,6 +46,8 @@ controller: create: true # controller.serviceAccount.name -- The name of an existing service account to use for the controller. Combined with `controller.serviceAccount.create` name: + # controller.extraEnv -- Extra environment variables to add to the controller container + extraEnv: [] # controller.flags -- A map of additional flags to pass to the controller. For monitoring all namespaces out of the box, add the following flag "on-by-default: true" flags: {} # controller.logVerbosity -- Controller log verbosity. Can be set from 1-10 with 10 being extremely verbose @@ -107,6 +109,8 @@ dashboard: port: 80 # dashboard.service.annotations -- Extra annotations for the dashboard service annotations: {} + # dashboard.extraEnv -- Extra environment variables to add to the dashboard container + extraEnv: [] # dashboard.flags -- A map of additional flags to pass to the dashboard. For monitoring all namespaces out of the box, add the following flag "on-by-default: true". flags: {} # dashboard.logVerbosity -- Dashboard log verbosity. Can be set from 1-10 with 10 being extremely verbose