immich-app / immich-charts

Helm chart implementation of Immich
https://immich.app
GNU Affero General Public License v3.0
117 stars 49 forks source link

Support for separate workers #105

Closed 0x0013 closed 3 months ago

0x0013 commented 3 months ago

This PR enables running microservices in a separate container again.

With defaults, this should render the same as in previous version v0.7.0.

When .Values.microservices.enable is set to true:

fixes #94

bo0tzz commented 3 months ago

Thanks for the PR! The plan is to make the workers configuration more granular, so there's separate entries for tasks like transcoding etc etc. My goal with #94 is to have the option to configure any amount of containers with arbitrary workers enabled. This PR essentially ends up just reverting #99, which is not the intent. As such I'll unfortunately need to close it, but thank you for the effort!

0x0013 commented 3 months ago

@bo0tzz thanks for the clarification. It makes sense and I did consider if this could be made more granular - the IMMICH_WORKERS_EXCLUDE variable to the api server could be dynamically constructed to exclude any workers that are specifically enabled, which is why I opted for that variable instead of just setting IMMICH_WORKERS_INCLUDE to api - to ensure all (future) possible workers are covered. However, since immich currently only supports api and microservices, I didn't see the point of a more complex implementation at this point in time.

Looking forward to a more generic implementation for this. I will continue using my fork in the meantime, since I need to use some elevated permissions to the microservices container to be able to utilize HW encoding, and I would not want to do that on the more exposed web access container.

bo0tzz commented 3 months ago

Thanks for the effort, and sorry to close this!