chore: for redis, use service name as the default

immich-app / immich

High performance self-hosted photo and video management solution.

https://immich.app

GNU Affero General Public License v3.0

33.05k stars 1.59k forks source link

chore: for redis, use service name as the default #9109

Closed jrasm91 closed 2 weeks ago

jrasm91 commented 2 weeks ago

default to service name instead of container name

cloudflare-pages[bot] commented 2 weeks ago

Deploying immich with Cloudflare Pages

Latest commit:	`9713206`
Status:	✅ Deploy successful!
Preview URL:	https://227d532c.immich.pages.dev
Branch Preview URL:	https://chore-redis-hostname.immich.pages.dev

View logs

bo0tzz commented 2 weeks ago

This works without changing redis.container_name?

jrasm91 commented 2 weeks ago

This works without changing redis.container_name?

I don't need to change the container name and it still works because redis is now referring to the service name of the container. Under the services section of the compose file we have an entry for redis and this is referring to that now.

(not to mention the e2e uses jobs and everything is still working)