From e2f7672e1d789ca3fab66d3a0471418f24ff1338 Mon Sep 17 00:00:00 2001
From: jessebot <jessebot@linux.com>
Date: Mon, 29 Jul 2024 13:58:48 +0200
Subject: [PATCH] Don't require nodePort if none is specified

fixes #44

Signed-off-by: jessebot <jessebot@linux.com>
---
 charts/nextcloud/Chart.yaml             | 2 +-
 charts/nextcloud/templates/service.yaml | 6 +++---
 charts/nextcloud/values.yaml            | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/charts/nextcloud/Chart.yaml b/charts/nextcloud/Chart.yaml
index a40cd088..03ac25b8 100644
--- a/charts/nextcloud/Chart.yaml
+++ b/charts/nextcloud/Chart.yaml
@@ -1,6 +1,6 @@
 apiVersion: v2
 name: nextcloud
-version: 5.5.0
+version: 5.5.1
 appVersion: 29.0.4
 description: A file sharing server that puts the control and security of your own data back into your hands.
 keywords:
diff --git a/charts/nextcloud/templates/service.yaml b/charts/nextcloud/templates/service.yaml
index 2223c945..866eb7c2 100644
--- a/charts/nextcloud/templates/service.yaml
+++ b/charts/nextcloud/templates/service.yaml
@@ -32,10 +32,10 @@ spec:
       targetPort: {{ .Values.nextcloud.containerPort }}
       protocol: TCP
       name: http
-      {{- if (eq .Values.service.type "NodePort") }}
-      nodePort: {{ .Values.service.nodePort | default "" }}
+      {{- with .Values.service.nodePort }}
+      nodePort: {{ . }}
       {{- end }}
   selector:
     app.kubernetes.io/name: {{ include "nextcloud.name" . }}
     app.kubernetes.io/instance: {{ .Release.Name }}
-    app.kubernetes.io/component: app
\ No newline at end of file
+    app.kubernetes.io/component: app
diff --git a/charts/nextcloud/values.yaml b/charts/nextcloud/values.yaml
index 301ef9ea..d924c6ff 100644
--- a/charts/nextcloud/values.yaml
+++ b/charts/nextcloud/values.yaml
@@ -495,7 +495,7 @@ service:
   type: ClusterIP
   port: 8080
   loadBalancerIP: ""
-  nodePort: nil
+  nodePort:
   annotations: {}
     ## Insert your annotations such as below
     # test/test: pumuckel