Persistent Volume Claims of /downloads and /movies should be located on same volume with one pvc about billimek-charts

[somerandow]

Took a stab at this, as I actually just ran into the same issue, albeit on plain Docker. While bazarr doesn't necessarily have the same issue with linking, figured all the 'arr apps should all probably be laid out the same.
If the commits here look good/cohesive, I can go ahead and do the same for sonarr, etc. and toss a PR together for each.

[billimek]

Hi @abel408, correct! We have the extraExistingClaimMounts configuration option to help address this (also for radarr)

An example of the implementation is here:

    persistence:
      config:
        enabled: true
        storageClass: "rook-ceph-block"
        size: 2Gi
      downloads:
        enabled: false
      tv:
        enabled: false
      extraExistingClaimMounts:
        - name: media
          existingClaim: nfs-media-pvc
          mountPath: /media
          readOnly: false

It seems like we should still 'allow' someone to choose to have the different mounts for downloads and tv/movies. What do you think?

[abel408]

Ahhh, thank you. I do agree with allowing different mount points, however, the chart by default will create a mount point for each and radarr and sonarr, by default, will use hardlinks so I think there is a bit of a discrepancy there. I would imagine the only reason someone would want a mount point for each would be if they are organizing things manually? I just learned the hard way that radarr and sonarr have been duplicating all my files and am now in the process of reorganizing :/. I have a feeling I'm not the only one.

[billimek]

@abel408 this is a good point, we should consider changing the default behavior. Something different than the extraExistingClaimMounts would be necessary for the two charts because we won't have an existing claim to reference by default.

@onedr0p what do you think? I think we would need to change the chart to collapse /downloads and /tv or /movies into a single PVC and allow folks to leverage extraExistingClaimMounts if they really want them separate. We could bump the chart major version to indicate a breaking change.

[onedr0p]

@billimek do you think a single media volume is good instead of downloads and tv or movies ?

[billimek]

@onedr0p yeah, exactly! Can you think of any downsides taking that approach?

[onedr0p]

@billimek maybe, my current deployments use media in extraExistingClaimMounts. Do you think that is a conflict?

    persistence:
      downloads:
        enabled: false
      tv:
        enabled: false
      extraExistingClaimMounts:
        - name: media
          existingClaim: nfs-media-pvc
          mountPath: /media
          readOnly: false

[billimek]

With the change we're discussing, yeah. I see it as being something like this:

    persistence:
      config:
        enabled: true
      media:
        enabled: true
        existingClaim: nfs-media-pvc
        subPath: /media
      extraExistingClaimMounts: <not needed now>

... which would be a 'breaking' change

[onedr0p]

Sounds good! I think this would be a great change.

[onedr0p]

@WojoInc looks great, feel free to open PRs :)

[somerandow]

Sg! Submitted PR for Bazarr. I plan on doing the same for Sonarr, Radarr, and Lidarr after work today. Any other charts that may need the same thing done?

[billimek]

Should be fixed now by #224, #225, #226, & #227

[windowsrefund]

As someone who is starting out with these charts, I'm not worried about backwards compatibility or breaking changes and just want to get things up and running as cleanly as possible. Is there some documentation or walk-through published anywhere that would show how to create a sane values.yaml (to be passed into the install via -f) which results in sonarr using the same volume for radarr and plex? In other words, all 3 would end up using the same PVC for media reads and writes?

[onedr0p]

@windowsrefund what you want is NFS storage but radarr and Sonarr don't support it, it's better to use block storage

[windowsrefund]

Hi, I'm actually using openebs block storage (cStor) but think I can get RWX if I plug nfs-server-provisioner into the mix. Baically, this

I should just be able to create a PVC on my own for the purpose of using from sonarr, radarr, and the rest.