Skip to content

Adding a docker option#25

Closed
gbit-is wants to merge 9 commits into
OpenMaxIO:openMaxIO-mainfrom
gbit-is:openMaxIO-main
Closed

Adding a docker option#25
gbit-is wants to merge 9 commits into
OpenMaxIO:openMaxIO-mainfrom
gbit-is:openMaxIO-main

Conversation

@gbit-is

@gbit-is gbit-is commented Jul 11, 2025

Copy link
Copy Markdown

With all the tools required for building OpenMaxUI I decided to do this inside a docker container, I decided to use ubuntu as a base image since I am familiar with it, it could probably be built as a lighter image inside alpine or something

This pull request includes the Dockerfile required for building the image and a docker compose file with an nginx reverse proxy to provide https access on port 443

Note: the initial build time is significant and the final docker image ends up being close to 5GB in size

@nreisingercres

Copy link
Copy Markdown

5GB is huge for this docker image. It'll be better to use multi stages, have the first stage build the artifacts (both frontend with node and back end with golang) then copy them to the second stage(the release image) then the final container image will not contain everything used to build it.
https://docs.docker.com/build/building/multi-stage/

@gbit-is

gbit-is commented Jul 11, 2025

Copy link
Copy Markdown
Author

Totally agree, that is not something I have ever done before but I might have time next week to figure it out

@gbit-is

gbit-is commented Jul 11, 2025

Copy link
Copy Markdown
Author

lol, it was muuuuch simpler then I thought
I will be submitting the much smaller and better image, along with a fix to the nginx config

@gbit-is gbit-is closed this Jul 11, 2025
@gbit-is

gbit-is commented Jul 11, 2025

Copy link
Copy Markdown
Author

Closing this pull and will open a new one

@gbit-is

gbit-is commented Jul 11, 2025

Copy link
Copy Markdown
Author

Link to the new pull request:

#26

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