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

feat(server): run microservices in worker thread #9426

Merged
merged 1 commit into from
May 14, 2024

Conversation

zackpollard
Copy link
Contributor

POC for bringing microservices into the server instance by using Node.JS worker threads.

Copy link

cloudflare-pages bot commented May 13, 2024

Deploying immich with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5f2cf91
Status: ✅  Deploy successful!
Preview URL: https://6ca1b140.immich.pages.dev
Branch Preview URL: https://feat-worker-threads-microser.immich.pages.dev

View logs

@zackpollard zackpollard force-pushed the feat/worker-threads-microservices branch from 7c00a48 to 94d13e4 Compare May 14, 2024 12:28
@zackpollard zackpollard marked this pull request as ready for review May 14, 2024 12:29
@zackpollard zackpollard force-pushed the feat/worker-threads-microservices branch from 94d13e4 to 1f84c9a Compare May 14, 2024 12:34
Copy link
Member

@bo0tzz bo0tzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

function bootstrap() {
switch (immichApp) {
case 'immich': {
process.title = 'immich_server';
if (process.env.INTERNAL_MICROSERVICES === 'true') {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we eventually want to have an env var for each worker, or rather something like IMMICH_ENABLED_WORKERS=api,web,microservices?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can figure this out later, but a pattern like that might make sense

@zackpollard zackpollard force-pushed the feat/worker-threads-microservices branch from 1f84c9a to 13b65e6 Compare May 14, 2024 13:25
@zackpollard zackpollard changed the title feat: initial microservices in worker thread POC feat(server): run microservices in worker thread May 14, 2024
@zackpollard zackpollard force-pushed the feat/worker-threads-microservices branch from 13b65e6 to 5f2cf91 Compare May 14, 2024 13:29
@zackpollard zackpollard merged commit 1ea55d6 into main May 14, 2024
23 checks passed
@zackpollard zackpollard deleted the feat/worker-threads-microservices branch May 14, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants