Repository navigation
Feature/container systemd service alt - #154
Draft
Louwrensth wants to merge 19 commits into
Draft
Louwrensth wants to merge 19 commits into
Louwrensth wants to merge 19 commits into
Conversation
- [server].authentication_type is unused in code. - [authentication].type is the one. - add comment to section [authentication] for simdb-server behind F5
Louwrensth
force-pushed
the
feature/container-systemd-service-alt
branch
from
October 16, 2026 15:15
25debde to
032e045
Compare
…ging', 'feature/config-tweaks-for-server', 'bugfix/publish-pypy-is-privileged', 'bugfix/simdb-version' and 'feature/docker-postgres-backup-scripts' into tmp/union
- pin base image tags
- uv sync --no-build for dependencies
So that it fails quickly rather than try to (slowly) build a dependency when the built package proves unavailable. This was the case for flask-mail~=0.9.1 at python 3.12.
- docker-compose: only need to build image once
Add `image` spec so docker compose builds it only once instead four times for migrations, web, worker, and beat.
remove docker-compose-pyver.yml
- gunicorn: expose access and error logs to stdout
- add gunicorn.conf.py without mounting it
it is baked into the image and reads everything tunable from the GUNICORN_* environment.
- .gitignore: add common data folders
- .gitignore, .dockerignore: cleanup
- scripts/simdb{,_server}: remove redundant scripts
uv already installs the ones defined in pyproject.toml, under [project.scripts]
- simdb instance: move nginx.dev.conf to docker/
migrations service should wait for a healthy postgres. postgres should not need `locale` command, it doesn't exist on alpine.
- multi-stage build + server build: with all compile dependencies. server: with only execute capabilities and OCI labels. add validation stage. - non-root user Celery was complaining loudly. - uv sync --no-dev only, UV_NO_DEV environment makes uv run throw warning "has no effect". - setuptools tweak SETUPTOOLS_SCM_PRETEND_VERSION_FOR_IMAS_SIMDB
- add docker compose layer - add system unit service and env files - add Makefile for easy deployment - add to docs
- use docker/version.sh - simplify variables - simplify gha workflow to reduce artifact juggling - simple ci triggers for image building tagging and publishing - consistent image addresses - consistent git/image/package version tags - remove docker test target to verify code - documentation errata
Louwrensth
force-pushed
the
feature/container-systemd-service-alt
branch
from
October 16, 2026 15:39
032e045 to
7333acb
Compare
Add docker/entrypoint/boot-nonroot.sh instead of fixed USER directive in Dockerfile. It will setup the correct user matching APP_UID and APP_GID so that the data directory can be written to without needing to rebuild the image.
Louwrensth
force-pushed
the
feature/container-systemd-service-alt
branch
from
October 19, 2026 14:51
684877a to
25e7f57
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Same as
but rebased onto several smaller PRs to check it's identical.