Skip to content

Improve Docker container definition.#79

Merged
plietar merged 1 commit into
mainfrom
improve-container
Dec 3, 2025
Merged

Improve Docker container definition.#79
plietar merged 1 commit into
mainfrom
improve-container

Conversation

@plietar

@plietar plietar commented Dec 2, 2025

Copy link
Copy Markdown
Member

This changes the container from blocking on start until it finds a root to creating it itself instead. This allows the service to be run out of the box with just a docker run outpack_server.

In the past we were going to use a Git clone as the outpack root so it made sense to have more control over the creation of it, but we've gone down a different route now.

Additionally it adds tini as an entrypoint wrapper to make sure we have a well behaved PID 1. This avoids the issue of the container taking a long time to shutdown.

@plietar plietar requested a review from M-Kusumgar December 2, 2025 13:21
This changes the container from blocking on start until it finds a root
to creating it itself instead. This allows the service to be run out of
the box with just a `docker run outpack_server`.

In the past we were going to use a Git clone as the outpack root so it
made sense to have more control over the creation of it, but we've gone
down a different route now.

Additionally it adds [tini][tini] as an entrypoint wrapper to make sure
we have a well behaved PID 1. This avoids the issue of the container
taking a long time to shutdown.

[tini]: https://github.com/krallin/tini
@plietar plietar merged commit 8939764 into main Dec 3, 2025
19 checks passed
@plietar plietar deleted the improve-container branch December 3, 2025 18:51
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.

2 participants