From 1b842a8ab27bcee00de69a5ce31425ad49cb6036 Mon Sep 17 00:00:00 2001
From: Mat Schaffer <115565899+matschaffer-roblox@users.noreply.github.com>
Date: Thu, 26 Oct 2023 09:25:11 -0700
Subject: [PATCH] Add envFrom to operator deployment (#1785)

* Add envFrom to operator deployment

Useful to when env vars are used for auth when downloading `spark.archives` from S3.

* Fix over-indenting
---
 charts/spark-operator-chart/templates/deployment.yaml | 2 ++
 charts/spark-operator-chart/values.yaml               | 3 +++
 2 files changed, 5 insertions(+)

diff --git a/charts/spark-operator-chart/templates/deployment.yaml b/charts/spark-operator-chart/templates/deployment.yaml
index 96c37ec52..9deacae43 100644
--- a/charts/spark-operator-chart/templates/deployment.yaml
+++ b/charts/spark-operator-chart/templates/deployment.yaml
@@ -47,6 +47,8 @@ spec:
       - name: {{ .Chart.Name }}
         image: {{ .Values.image.repository }}:{{ default .Chart.AppVersion .Values.image.tag }}
         imagePullPolicy: {{ .Values.image.pullPolicy }}
+        envFrom:
+          {{- toYaml .Values.envFrom | nindent 10 }}
         securityContext:
           {{- toYaml .Values.securityContext | nindent 10 }}
         {{- if .Values.metrics.enable }}
diff --git a/charts/spark-operator-chart/values.yaml b/charts/spark-operator-chart/values.yaml
index e850d7fda..c7d672f60 100644
--- a/charts/spark-operator-chart/values.yaml
+++ b/charts/spark-operator-chart/values.yaml
@@ -71,6 +71,9 @@ ingressUrlFormat: ""
 # -- Set higher levels for more verbose logging
 logLevel: 2
 
+# -- Pod environment variable sources
+envFrom: []
+
 # podSecurityContext -- Pod security context
 podSecurityContext: {}