Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
runecalico opened this issue Apr 14, 2020 · 4 comments
Assignees

Comments

@runecalico
Copy link

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.

@onedr0p
Copy link
Contributor

onedr0p commented Apr 14, 2020

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

@simplyzee
Copy link
Collaborator

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.

@runecalico
Copy link
Author

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
Copy link
Author

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/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants