fix(docker-compose): remove /mnt and /root (#4115)

Signed-off-by: Stavros Kois <47820033+stavros-k@users.noreply.github.com>

charts/stable/docker-compose/Chart.yaml

@@ -21,7 +21,7 @@ sources:
   - https://github.com/truecharts/charts/tree/master/charts/stable/docker-compose
   - https://hub.docker.com/_/docker
 type: application
-version: 2.0.52
+version: 2.1.0
 annotations:
   truecharts.org/catagories: |
     - docker

charts/stable/docker-compose/questions.yaml

@@ -11,17 +11,18 @@ questions:
 # Include{controllerExpert}
 # Include{controllerExpertExtraArgs}
   - variable: env
-    group: "Container Configuration"
+    group: Container Configuration
-    label: "Image Environment"
+    label: Image Environment
     schema:
       additional_attrs: true
       type: dict
       attrs:
         - variable: COMPOSE_FILE
-          label: "COMPOSE_FILE"
+          label: Docker Compose File
-          description: "Path on the host of the composefile you want to load. Supported are paths inside /mnt, /root and /cluster"
+          description: Path on the host of the compose file you want to load. You MUST mount the storage yourself
           schema:
             type: string
+            default: ""
 # Include{containerConfig}
 # Include{serviceExpertRoot}
             default: true
@@ -32,42 +33,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"
+          label: runAsUser
-          description: "The UserID of the user running the application"
+          description: The UserID of the user running the application
           schema:
             type: int
             default: 0
         - variable: runAsGroup
-          label: "runAsGroup"
+          label: runAsGroup
-          description: "The groupID this App of the user running the application"
+          description: The groupID this App of the user running the application
           schema:
             type: int
             default: 0
         - variable: fsGroup
-          label: "fsGroup"
+          label: fsGroup
-          description: "The group that should own ALL storage."
+          description: The group that should own ALL storage.
           schema:
             type: int
             default: 0

charts/stable/docker-compose/values.yaml

@@ -46,26 +46,26 @@ service:
         type: HTTPS
 
 env:
-  DOCKER_TLS_CERTDIR: "/certs"
+  DOCKER_TLS_CERTDIR: /certs
   COMPOSE_FILE: ""
 
 lifecycle:
   postStart:
     exec:
       command:
-        - "/bin/bash"
+        - /bin/bash
-        - "-c"
+        - -c
         - |
           if [ "${COMPOSE_FILE}" ]; then
-          counter=0;
+            counter=0;
-          while ! docker info > /dev/null 2>&1 && [[ ${counter} -lt 10 ]]; do
+            while ! docker info > /dev/null 2>&1 && [[ ${counter} -lt 10 ]]; do
-          echo "docker not running yet. Waiting..." >> /proc/1/fd/1;
+              echo "docker not running yet. Waiting..." >> /proc/1/fd/1;
-          sleep 5;
+              sleep 5;
-          counter=$((counter+1));
+              counter=$((counter+1));
-          done
+            done
-          docker-compose -f "${COMPOSE_FILE}" up -d && echo "Loading COMPOSE_FILE at location ${COMPOSE_FILE}" >> /proc/1/fd/1;
+            docker-compose -f "${COMPOSE_FILE}" up -d && echo "Loading COMPOSE_FILE at location ${COMPOSE_FILE}" >> /proc/1/fd/1;
           else
-          echo "COMPOSE_FILE not set, not auto-loading compose-file" >> /proc/1/fd/1;
+            echo "COMPOSE_FILE not set, not auto-loading compose-file" >> /proc/1/fd/1;
           fi
 probes:
   liveness:
@@ -81,24 +81,17 @@ probes:
 persistence:
   varrun:
     enabled: false
-  root:
-    enabled: true
-    type: hostPath
-    hostPath: /root
-    mountPath: /root
-    hostPathType: ""
-    readOnly: false
   docker-certs-ca:
     enabled: true
-    mountPath: "/config"
+    mountPath: /config
 
 volumeClaimTemplates:
   docker-certs-client:
     enabled: true
-    mountPath: "/certs/client"
+    mountPath: /certs/client
   docker:
     enabled: true
-    mountPath: "/var/lib/docker"
+    mountPath: /var/lib/docker
 
 portal:
   enabled: false