kube-plex icon indicating copy to clipboard operation
kube-plex copied to clipboard

Extra Data mounts are not also mounted in transcoder pod, transcoding fails

Open runecalico opened this issue 5 years ago • 4 comments

Having the persistence.extraData mount options in the helm chart is super awesome as I don't have all of my media under a single share .. Unfortunately it looks like when the transcoder pod is created, it doesn't pass any of the extraData mounts to the transcoder, so attempts to transcode any file contained in one of those extraData mounts fails. The transcoder pod deployment yaml doesn't show those extraData mounts ..

    volumeMounts:
    - mountPath: /data
      name: data
      readOnly: true
    - mountPath: /config
      name: config
      readOnly: true
    - mountPath: /transcode
      name: transcode
    - mountPath: /var/run/secrets/kubernetes.io/serviceaccount
      name: default-token-9shvd
      readOnly: true
    workingDir: /transcode/Transcode/Sessions/plex-transcode-d48pgie0z4f612e7g3ih5qhs-d18c2e9c-f4dc-49a7-9362-ecad21e9dc04

Unfortunately I'm a total noob with Go, so while I think the relevant code is in kube-plex\main.go, I don't really know how to ADD support for those extraData mounts (especially since there could be 0 or more of them).

Please Advise.

runecalico avatar Apr 14 '20 01:04 runecalico

From the amount of reports I think we can gather kube-plex is broken. We're out of luck until a maintainer steps in or @munnerz comes back. Use the flag --set kubePlex.enabled=false until it's fixed

onedr0p avatar Apr 14 '20 18:04 onedr0p

I will take a look at it to see what the issue is with the transcoder pods and see if I can push a fix out to get this working again.

simplyzee avatar Apr 15 '20 11:04 simplyzee

Understood. I was submitting the report under the concept of "I might get lucky and someone knows how to do this and is willing to do it too" :)

runecalico avatar Apr 15 '20 21:04 runecalico

We might be able to try using a PodPreset to expose certain ENV and Volumes to the transcoder instead of having to modify the go .. however that would still require the transcoder pods be created with a label or something that could be used to target them. https://kubernetes.io/docs/tasks/inject-data-application/podpreset/

runecalico avatar Apr 19 '20 18:04 runecalico