feat: Add missing namespace field to Helm templates

[zimmertr]

Should fix this:

When deploying to K8s with Argo CD like this:

namespace: plex-media-server-system

resources:
  - resources/namespace.yml
  - resources/persistentVolume.yml
  - resources/persistentVolumeClaim.yml

commonLabels:
  app.kubernetes.io/name: plex-media-server

helmCharts:
  - includeCRDs: true
    name: plex-media-server
    namespace: plex-media-server-system
    releaseName: plex-media-server
    repo: https://raw.githubusercontent.com/plexinc/pms-docker/gh-pages
    valuesFile: values.yml
    version: 1.4.0

# The Helm Chart doesn't support disabling the dynamic config PersistentVolume.
# I do want one, but I want to statically reference a Mountpoint instead.
patches:
  - patch: |-
      apiVersion: apps/v1
      kind: StatefulSet
      metadata:
        name: plex-media-server
      spec:
        template:
          spec:
            containers:
            - name: plex-media-server-pms
              volumeMounts:
              - name: pms-config
                mountPath: /config
        volumeClaimTemplates:
    target:
      kind: StatefulSet
      name: plex-media-server

[cilindrox]

LGTM - good catch @zimmertr - we rely on the Application's spec.destination.namespace, so these flew under the radar :/

[zimmertr]

Thank you! Do you know when this will make it into the next release?

[cilindrox]

@zimmertr this should ship as part of v1.5.0 when #168 is merged

[cilindrox]

v1.5.0 of the chart is out: https://github.com/plexinc/pms-docker/releases/tag/helm-chart-1.5.0

Thanks @zimmertr

[zimmertr]

Thank you! That worked, the container deploys successfully with my kustomization now.

[cilindrox]

@zimmertr feel free to open a PR for the pms-config mount if you'd like - seems like a useful addition :)