Skip to content

In servarr cleanuparr ignores jellyfin.persistence.media.existingClaim #72

Description

@amedranogil

Template location: charts/servarr/templates/cleanuparr/deployment.yaml

Current behaviour:

- name: media
  {{- if and .Values.jellyfin.enabled .Values.jellyfin.persistence.media.enabled }}
  persistentVolumeClaim:
    claimName: {{ .Release.Name }}-jellyfin-media   # ← always uses generated name

Unlike the lidarr/radarr/sonarr templates (which correctly check
.Values.jellyfin.persistence.media.existingClaim), the cleanuparr template
hardcodes {{ .Release.Name }}-jellyfin-media. When an existingClaim is set, no PVC
with that generated name is created (correctly skipped by jellyfin/pvc.yaml), so
cleanuparr stays Pending with:

0/1 nodes are available: persistentvolumeclaim "servarr-jellyfin-media" not found.

Suggested fix (consistent with how other deployments handle it):

- name: media
  {{- if and .Values.jellyfin.enabled .Values.jellyfin.persistence.media.enabled }}
  persistentVolumeClaim:
    claimName: {{ if .Values.jellyfin.persistence.media.existingClaim }}{{ .Values.jellyfin.persistence.media.existingClaim }}{{- else }}{{ .Release.Name }}-jellyfin-media{{- end }}

Chart version: 1.5.2
Kubernetes: k3s
Reproduction: Use a pre-existing NFS PVC as jellyfin.persistence.media.existingClaim with cleanuparr.enabled: true.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions