From 9444cc9256ac2f4d63a61694c90e1141771efd33 Mon Sep 17 00:00:00 2001
From: Xstar97 <Xstar97@users.noreply.github.com>
Date: Wed, 9 Nov 2022 02:42:56 -0500
Subject: [PATCH] fix(esphome) add env variables to GUI (#4354)

* add ESPHOME_DASHBOARD_USE_PING to GUI

* cleanup

Co-authored-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>
---
 charts/stable/esphome/questions.yaml | 57 +++++++++++++++++-----------
 charts/stable/esphome/values.yaml    |  9 ++---
 2 files changed, 39 insertions(+), 27 deletions(-)

diff --git a/charts/stable/esphome/questions.yaml b/charts/stable/esphome/questions.yaml
index 3e629bde0d9..8d3938285c4 100644
--- a/charts/stable/esphome/questions.yaml
+++ b/charts/stable/esphome/questions.yaml
@@ -13,10 +13,23 @@ questions:
 # Include{controllerExpert}
 # Include{controllerExpertExtraArgs}
 # Include{containerConfig}
+  - variable: env
+    group: Container Configuration
+    label: Image Environment
+    schema:
+      additional_attrs: true
+      type: dict
+      attrs:
+        - variable: ESPHOME_DASHBOARD_USE_PING
+          label: Esphome Dasboard Use Ping
+          description: Check the status of the device
+          schema:
+            type: boolean
+            default: true
 # Include{serviceRoot}
         - variable: main
-          label: "Main Service"
-          description: "The Primary service on which the healthcheck runs, often the webUI"
+          label: Main Service
+          description: The Primary service on which the healthcheck runs, often the webUI
           schema:
             additional_attrs: true
             type: dict
@@ -24,22 +37,22 @@ questions:
 # Include{serviceSelectorLoadBalancer}
 # Include{serviceSelectorExtras}
                     - variable: main
-                      label: "Main Service Port Configuration"
+                      label: Main Service Port Configuration
                       schema:
                         additional_attrs: true
                         type: dict
                         attrs:
                           - variable: port
-                            label: "Port"
-                            description: "This port exposes the container port on the service"
+                            label: Port
+                            description: This port exposes the container port on the service
                             schema:
                               type: int
                               default: 6052
                               required: true
 # Include{advancedPortHTTP}
                                 - variable: targetPort
-                                  label: "Target Port"
-                                  description: "The internal(!) port on the container the Application runs on"
+                                  label: Target Port
+                                  description: The internal(!) port on the container the Application runs on
                                   schema:
                                     type: int
                                     default: 6052
@@ -50,8 +63,8 @@ questions:
 # Include{serviceList}
 # Include{persistenceRoot}
         - variable: config
-          label: "App Config Storage"
-          description: "Stores the Application Configuration."
+          label: App Config Storage
+          description: Stores the Application Configuration.
           schema:
             additional_attrs: true
             type: dict
@@ -59,8 +72,8 @@ questions:
 # Include{persistenceBasic}
 # Include{persistenceAdvanced}
         - variable: platformio
-          label: "Platformio Storage"
-          description: "Stores the Application Configuration."
+          label: Platformio Storage
+          description: Stores the Application Configuration.
           schema:
             additional_attrs: true
             type: dict
@@ -70,7 +83,7 @@ questions:
 # Include{persistenceList}
 # Include{ingressRoot}
         - variable: main
-          label: "Main Ingress"
+          label: Main Ingress
           schema:
             additional_attrs: true
             type: dict
@@ -83,42 +96,42 @@ questions:
 # Include{security}
 # Include{securityContextAdvancedRoot}
               - variable: privileged
-                label: "Privileged mode"
+                label: Privileged mode
                 schema:
                   type: boolean
                   default: true
               - variable: readOnlyRootFilesystem
-                label: "ReadOnly Root Filesystem"
+                label: ReadOnly Root Filesystem
                 schema:
                   type: boolean
                   default: false
               - variable: allowPrivilegeEscalation
-                label: "Allow Privilege Escalation"
+                label: Allow Privilege Escalation
                 schema:
                   type: boolean
                   default: true
               - variable: runAsNonRoot
-                label: "runAsNonRoot"
+                label: runAsNonRoot
                 schema:
                   type: boolean
                   default: false
 # Include{securityContextAdvanced}
 # Include{podSecurityContextRoot}
         - variable: runAsUser
-          label: "runAsUser"
-          description: "The UserID of the user running the application"
+          label: runAsUser
+          description: The UserID of the user running the application
           schema:
             type: int
             default: 0
         - variable: runAsGroup
-          label: "runAsGroup"
-          description: "The groupID this App of the user running the application"
+          label: runAsGroup
+          description: The groupID this App of the user running the application
           schema:
             type: int
             default: 0
         - variable: fsGroup
-          label: "fsGroup"
-          description: "The group that should own ALL storage."
+          label: fsGroup
+          description: The group that should own ALL storage.
           schema:
             type: int
             default: 568
diff --git a/charts/stable/esphome/values.yaml b/charts/stable/esphome/values.yaml
index 7688a44a95f..2f6085eb63d 100644
--- a/charts/stable/esphome/values.yaml
+++ b/charts/stable/esphome/values.yaml
@@ -22,9 +22,8 @@ service:
         targetPort: 6052
 
 env:
-  {}
-  # ESPHOME_DASHBOARD_USE_PING: true
-  # ESPHOME_DASHBOARD_RELATIVE_URL: "/"
+  ESPHOME_DASHBOARD_USE_PING: false
+  ESPHOME_DASHBOARD_RELATIVE_URL: /
   # ESPHOME_QUICKWIZARD:
   # ESPHOME_IS_HASSIO:
   # DISABLE_HA_AUTHENTICATION:
@@ -34,10 +33,10 @@ env:
 persistence:
   config:
     enabled: true
-    mountPath: "/config"
+    mountPath: /config
   platformio:
     enabled: true
-    mountPath: "/.platformio"
+    mountPath: /.platformio
 
 portal:
   enabled: true