Skip to content

chore: Add a debian based docker file and docker compose file - #344

Merged
rholshausen merged 4 commits into
masterfrom
debian-base
Jul 21, 2026
Merged

chore: Add a debian based docker file and docker compose file#344
rholshausen merged 4 commits into
masterfrom
debian-base

Conversation

@rholshausen

Copy link
Copy Markdown
Contributor

Adds a debian version of the main docker file and a docker compose file to test it.

This is to support #333

@rholshausen
rholshausen requested review from JP-Ellis and YOU54F July 16, 2026 06:27
@JP-Ellis

Copy link
Copy Markdown
Member

I think it just needs to be incorporated into the release pipeline and/or CI, just making sure it can be built and reach a health check endpoint. Otherwise, I think it looks good

@rholshausen

Copy link
Copy Markdown
Contributor Author

@YOU54F can you check if this is ok, and also let us know how to integrate it with the release process

@YOU54F

YOU54F commented Jul 20, 2026

Copy link
Copy Markdown
Member

Yes will take a look, I wonder if we should pull arm rather than having to build 6 image variants, that will drop us down to 2 archs to support, and 2 base images (alpine + debian). Unfortunately we don't get arch specific stats for image pulls, it was initially introduced as part of #30

we will want to add

  1. smoke test ✅
  2. audit ✅
  3. update release flows to publish with -debian suffixed tags, they will need updating to pass in -f Dockerfile with .debian appended for the debian passes. ✅

@YOU54F YOU54F left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should be good to go now, although the publishing flow has not been tested, I normally do this on my own org with a fork

@rholshausen
rholshausen merged commit 51b1170 into master Jul 21, 2026
6 checks passed
@rholshausen
rholshausen deleted the debian-base branch July 21, 2026 05:36
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.

3 participants