From c646437d6e5cc1f7b18d8660106bc3b49c436d2b Mon Sep 17 00:00:00 2001
From: StevenMcElligott <89483932+StevenMcElligott@users.noreply.github.com>
Date: Thu, 17 Nov 2022 17:40:20 -0500
Subject: [PATCH] fix(SyncTube): Fix port + cleanup (#4526)

---
 charts/incubator/synctube/Chart.yaml     | 20 ++++++-------
 charts/incubator/synctube/questions.yaml | 36 ++++++++++++------------
 charts/incubator/synctube/values.yaml    |  3 +-
 3 files changed, 29 insertions(+), 30 deletions(-)

diff --git a/charts/incubator/synctube/Chart.yaml b/charts/incubator/synctube/Chart.yaml
index 357c42782c0..083e2af70e2 100644
--- a/charts/incubator/synctube/Chart.yaml
+++ b/charts/incubator/synctube/Chart.yaml
@@ -1,12 +1,3 @@
-annotations:
-  truecharts.org/SCALE-support: "true"
-  truecharts.org/catagories: |
-    - Cloud
-    - Productivity
-    - Tools-Utilities
-    - MediaApp-Video
-    - MediaServer-Video
-    - Network-Web
 apiVersion: v2
 appVersion: "latest"
 dependencies:
@@ -36,4 +27,13 @@ sources:
   - https://github.com/RblSb/SyncTube
   - https://hub.docker.com/r/littlestache/synctube
 type: application
-version: 2.0.1
+version: 2.1.0
+annotations:
+  truecharts.org/SCALE-support: "true"
+  truecharts.org/catagories: |
+    - Cloud
+    - Productivity
+    - Tools-Utilities
+    - MediaApp-Video
+    - MediaServer-Video
+    - Network-Web
diff --git a/charts/incubator/synctube/questions.yaml b/charts/incubator/synctube/questions.yaml
index 54848617e04..31a9544bbe0 100644
--- a/charts/incubator/synctube/questions.yaml
+++ b/charts/incubator/synctube/questions.yaml
@@ -11,8 +11,8 @@ questions:
 # Include{containerConfig}
 # 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
@@ -20,14 +20,14 @@ 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: 4200
@@ -38,8 +38,8 @@ questions:
 # Include{serviceList}
 # Include{persistenceRoot}
         - variable: storageconfig
-          label: "storageconfig Storage"
-          description: "This is the location for the config file."
+          label: Config Storage
+          description: This is the location for the config file.
           schema:
             additional_attrs: true
             type: dict
@@ -48,7 +48,7 @@ questions:
 # Include{persistenceList}
 # Include{ingressRoot}
         - variable: main
-          label: "Main Ingress"
+          label: Main Ingress
           schema:
             additional_attrs: true
             type: dict
@@ -60,41 +60,41 @@ questions:
 # Include{security}
 # Include{securityContextAdvancedRoot}
               - variable: privileged
-                label: "Privileged mode"
+                label: Privileged mode
                 schema:
                   type: boolean
                   default: false
               - 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: false
               - variable: runAsNonRoot
-                label: "runAsNonRoot"
+                label: runAsNonRoot
                 schema:
                   type: boolean
                   default: false
 # 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/incubator/synctube/values.yaml b/charts/incubator/synctube/values.yaml
index 1e49be57109..5a393fcf3e4 100644
--- a/charts/incubator/synctube/values.yaml
+++ b/charts/incubator/synctube/values.yaml
@@ -1,4 +1,3 @@
-env: {}
 image:
   pullPolicy: IfNotPresent
   repository: tccr.io/truecharts/synctube
@@ -18,7 +17,7 @@ service:
     ports:
       main:
         port: 4200
-        protocol: TCP
+        protocol: HTTP
         targetPort: 4200
 
 portal: