Skip to content

Add Dockerfile and container CI#15

Merged
the-snesler merged 7 commits intomainfrom
dockerfile
Feb 25, 2026
Merged

Add Dockerfile and container CI#15
the-snesler merged 7 commits intomainfrom
dockerfile

Conversation

@the-snesler
Copy link
Copy Markdown
Member

@the-snesler the-snesler commented Feb 18, 2026

This updates the build CI task to create a Docker container and push it to GitHub Container Registry. In the future, we could then pull it down on a server, set the appropriate .env variables, and deploy. Because it uses GHCR's caching, it may be able to speed up build times too.

Most of the code is reused from a combination of https://github.com/the-snesler/buckys-buzzer-beater and https://github.com/docker/metadata-action. This PR already pushed a container that in theory you could download and run right now https://github.com/DSSD-Madison/ST-Madison-Backend/pkgs/container/st-madison-backend

@the-snesler the-snesler marked this pull request as ready for review February 18, 2026 17:09
@the-snesler
Copy link
Copy Markdown
Member Author

the-snesler commented Feb 18, 2026

Attempted to deploy and getting this error:

called `Result::unwrap()` on an `Err` value: DuckDBFailure(Error { code: Unknown, extended_code: 1 }, Some("IO Error: Extension \"/root/.duckdb/extensions/v1.4.3/linux_amd64_musl/httpfs.duckdb_extension\" could not be loaded: Dynamic loading not supported"))

Back to the drawing board

@the-snesler the-snesler marked this pull request as draft February 18, 2026 22:36
@the-snesler the-snesler marked this pull request as ready for review February 18, 2026 23:20
@the-snesler
Copy link
Copy Markdown
Member Author

It is working. I switched to prebuilt binaries, and now it is deployed at: http://192.210.203.38:2160/health
I'll give it a domain some other time, but at least it works end-to-end. Does mean I needed to change the README instructions- I am sure we could figure out which folder it looks for DuckDB in, but for now it's easier to just tell it to download every time.

Copy link
Copy Markdown
Collaborator

@adrian-brady adrian-brady left a comment

Choose a reason for hiding this comment

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

LGTM

@the-snesler the-snesler merged commit 7483537 into main Feb 25, 2026
2 of 4 checks passed
@the-snesler the-snesler deleted the dockerfile branch February 25, 2026 00:14
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