Merge pull request #75 from maxim-mityutko/feature-74-auth

[feature] Enable port 8971
blakeblackshear · Sep 22, 2024 · 442e906 · 442e906
2 parents f5e48b2 + 827fbcb
commit 442e906
Show file tree

Hide file tree

Showing 6 changed files with 16 additions and 7 deletions.
diff --git a/charts/frigate/Chart.yaml b/charts/frigate/Chart.yaml
@@ -1,8 +1,8 @@
 apiVersion: v2
-appVersion: "0.13.2"
+appVersion: "0.14.0"
 description: NVR With Realtime Object Detection for IP Cameras
 name: frigate
-version: 7.5.1
+version: 7.6.0
 keywords:
   - tensorflow
   - coral

diff --git a/charts/frigate/README.md b/charts/frigate/README.md
@@ -84,7 +84,7 @@ helm upgrade --install \
 | imagePullSecrets | list | `[]` | Docker image pull policy |
 | ingress.annotations | object | `{}` | annotations to configure your Ingress. See your Ingress Controller's Docs for more info. |
 | ingress.enabled | bool | `false` | Enables the use of an Ingress Controller to front the Service and can provide HTTPS |
-| ingress.hosts | list | `[{"host":"chart.example.local","paths":["/"]}]` | list of hosts and their paths that ingress controller should repsond to. |
+| ingress.hosts | list | `[{"host":"chart.example.local","paths":[{"path":"/", "portName":"http-auth"}]}]` | list of hosts and their paths and ports that ingress controller should repsond to. |
 | ingress.tls | list | `[]` | list of TLS configurations |
 | nameOverride | string | `""` | Overrides the name of resources |
 | nodeSelector | object | `{}` | Node Selector configuration |

diff --git a/charts/frigate/templates/deployment.yaml b/charts/frigate/templates/deployment.yaml
@@ -47,6 +47,9 @@ spec:
             - name: http
               containerPort: 5000
               protocol: TCP
+            - name: http-auth 
+              containerPort: 8971
+              protocol: TCP
             - name: rtmp
               containerPort: 1935
               protocol: TCP

diff --git a/charts/frigate/templates/ingress.yaml b/charts/frigate/templates/ingress.yaml
@@ -35,13 +35,13 @@ spec:
       http:
         paths:
           {{- range .paths }}
-          - path: {{ . }}
+          - path: {{ .path }}
             pathType: "ImplementationSpecific"
             backend:
               service:
                 name: {{ $fullName }}
                 port:
-                  name: http
+                  name: {{ .portName }}
           {{- end }}
   {{- end }}
 {{- end }}
diff --git a/charts/frigate/templates/service.yaml b/charts/frigate/templates/service.yaml
@@ -51,6 +51,10 @@ spec:
 {{ if (and (eq .Values.service.type "NodePort") (not (empty .Values.service.nodePort))) }}
       nodePort: {{.Values.service.nodePort}}
 {{ end }}
+    - name: http-auth
+      port: 8971
+      protocol: TCP
+      targetPort: http-auth
     - name: rtmp
       port: 1935
       protocol: TCP

diff --git a/charts/frigate/values.yaml b/charts/frigate/values.yaml
@@ -161,11 +161,13 @@ ingress:
     # nginx.org/proxy-send-timeout: "3600"
     # nginx.org/websocket-services: "<release_name>-frigate" # TODO: can this be automated?
 
-  # -- list of hosts and their paths that ingress controller should repsond to.
+  # -- list of hosts and their paths and ports that ingress controller should repsond to.
+  # -- alternatively use `http` if anonymous auth is allowed 
   hosts:
     - host: chart.example.local
       paths:
-        - '/'
+        - path: '/'
+          portName: http-auth
 
   # -- list of TLS configurations
   tls: []