diff --git a/charts/jellyfin/templates/deployment.yaml b/charts/jellyfin/templates/deployment.yaml index 6f4b6ae..41faa6b 100644 --- a/charts/jellyfin/templates/deployment.yaml +++ b/charts/jellyfin/templates/deployment.yaml @@ -37,6 +37,7 @@ spec: {{- toYaml . | nindent 8 }} {{- end }} serviceAccountName: {{ include "jellyfin.serviceAccountName" . }} + automountServiceAccountToken: {{ .Values.serviceAccount.automount }} securityContext: {{- toYaml .Values.podSecurityContext | nindent 8 }} {{- if or .Values.jellyfin.enableDLNA .Values.podPrivileges.hostNetwork }} diff --git a/charts/jellyfin/tests/integration_common_configs_test.yaml b/charts/jellyfin/tests/integration_common_configs_test.yaml index aa2d3ab..c6f9063 100644 --- a/charts/jellyfin/tests/integration_common_configs_test.yaml +++ b/charts/jellyfin/tests/integration_common_configs_test.yaml @@ -241,7 +241,7 @@ tests: replicaCount: 3 persistence.config.accessMode: ReadWriteMany persistence.media.accessMode: ReadWriteMany - persistence.media.type: nfs + persistence.media.type: hostPath persistence.media.hostPath: /nfs/jellyfin/media affinity: podAntiAffinity: diff --git a/charts/jellyfin/tests/service_test.yaml b/charts/jellyfin/tests/service_test.yaml index b40dbc6..d2e7aae 100644 --- a/charts/jellyfin/tests/service_test.yaml +++ b/charts/jellyfin/tests/service_test.yaml @@ -81,18 +81,6 @@ tests: path: spec.type value: LoadBalancer - - it: should support ExternalName type - set: - service.type: ExternalName - service.externalName: jellyfin.example.com - asserts: - - equal: - path: spec.type - value: ExternalName - - equal: - path: spec.externalName - value: jellyfin.example.com - # ============================================================================= # PORT CONFIGURATION # ============================================================================= @@ -388,19 +376,6 @@ tests: - isNull: path: spec.ports[0].nodePort - # ExternalName requires externalName field - - it: should set externalName for ExternalName type - set: - service.type: ExternalName - service.externalName: external.service.com - asserts: - - equal: - path: spec.type - value: ExternalName - - equal: - path: spec.externalName - value: external.service.com - # Multiple loadBalancerSourceRanges - it: should support multiple loadBalancerSourceRanges set: diff --git a/charts/jellyfin/tests/validation_mutual_exclusions_test.yaml b/charts/jellyfin/tests/validation_mutual_exclusions_test.yaml index 2b43671..2439b52 100644 --- a/charts/jellyfin/tests/validation_mutual_exclusions_test.yaml +++ b/charts/jellyfin/tests/validation_mutual_exclusions_test.yaml @@ -305,16 +305,6 @@ tests: path: spec.type value: LoadBalancer - - it: should create service with ExternalName type - template: service.yaml - set: - service.type: ExternalName - service.externalName: jellyfin.example.com - asserts: - - equal: - path: spec.type - value: ExternalName - # ============================================================================= # PORT RANGE VALIDATIONS # ============================================================================= diff --git a/charts/jellyfin/values.schema.json b/charts/jellyfin/values.schema.json index ba63d42..53b249c 100644 --- a/charts/jellyfin/values.schema.json +++ b/charts/jellyfin/values.schema.json @@ -38,7 +38,7 @@ "repository": { "type": "string", "default": "docker.io/jellyfin/jellyfin", - "pattern": "^[a-z0-9.-]+(/[a-z0-9._-]+)*$", + "pattern": "^[a-z0-9.-]+(:[0-9]+)?(/[a-z0-9._-]+)*$", "description": "Container image repository for Jellyfin" }, "tag": { @@ -151,7 +151,7 @@ "properties": { "type": { "type": "string", - "enum": ["ClusterIP", "NodePort", "LoadBalancer"], + "enum": ["ClusterIP", "NodePort", "LoadBalancer", "ExternalName"], "default": "ClusterIP", "description": "Kubernetes service type" },