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

Bump v1.77.0 #102

Merged
merged 18 commits into from
Sep 12, 2023
Merged

Bump v1.77.0 #102

merged 18 commits into from
Sep 12, 2023

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Sep 6, 2023

This PR bumps the version from v1.76.1 to v1.77.0.
Please review the changes and merge this PR if everything looks good.

New immich-cli version

No new immich-cli version available.

Upstream release notes

Monitored upstream files

diff --git a/machine-learning/Dockerfile b/machine-learning/Dockerfile
index 103c8ba6..bd65975f 100644
--- a/machine-learning/Dockerfile
+++ b/machine-learning/Dockerfile
@@ -1,4 +1,4 @@
-FROM python:3.11.4-bullseye@sha256:5b401676aff858495a5c9c726c60b8b73fe52833e9e16eccdb59e93d52741727 as builder
+FROM python:3.11-bookworm as builder
 
 ENV PYTHONDONTWRITEBYTECODE=1 \
   PYTHONUNBUFFERED=1 \
@@ -14,9 +14,9 @@ COPY poetry.lock pyproject.toml requirements.txt ./
 RUN poetry install --sync --no-interaction --no-ansi --no-root --only main
 RUN pip install --no-deps -r requirements.txt
 
-FROM python:3.11.4-slim-bullseye@sha256:91d194f58f50594cda71dcd2e8fdefd90e7ecc57d07823813b67c8521e565dcd
+FROM python:3.11-slim-bookworm
 
-RUN apt-get update && apt-get install -y --no-install-recommends tini && rm -rf /var/lib/apt/lists/*
+RUN apt-get update && apt-get install -y --no-install-recommends tini libmimalloc2.0 && rm -rf /var/lib/apt/lists/*
 
 WORKDIR /usr/src/app
 ENV NODE_ENV=production \
@@ -27,6 +27,7 @@ ENV NODE_ENV=production \
   PYTHONPATH=/usr/src
 
 COPY --from=builder /opt/venv /opt/venv
+COPY start.sh log_conf.json ./
 COPY app .
 ENTRYPOINT ["tini", "--"]
-CMD ["python", "-m", "app.main"]
+CMD ["./start.sh"]

/home/runner/work/immich-distribution/immich-distribution
/tmp/tmp.Rb4Gmqxbv4 removed

Checklist

  • Review the changes above
  • Possible write a news entry (and push it to this PR)
  • Wait for the CI to finish
  • Merge the PR

ref #101

I set 7 to save one worker thread for health checks, with this I hope
that I will queue requests inside HAProxy instead of throwing them down
to a overloaded backend.

I'm not sure that's a good value for microservices so I picked 7 there
as well :)
The current symlink is created on first run, so I run the cli just to
trigger a creation of symlinks for the current user.
This is a much better solution, to actually wait for the processing to
finish before we carry on.
@nsg nsg merged commit dae125f into master Sep 12, 2023
8 checks passed
@nsg nsg deleted the bump/v1.77.0 branch September 12, 2023 14:57
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

Successfully merging this pull request may close these issues.

1 participant