-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Publish docker #166
feat: Publish docker #166
Conversation
Ok I think it's done now lol. Pending new reviews |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excited to see this work when we release v0.13.0!
Looks like the package was marked private lol, but now that's fixed and I've confirmed that the EDIT: Well something went wrong idk what lol |
We might as well publish our own Docker image. Would make deployment much easier if a host could simply write
ghcr.io/byu-cs-discord/csbot
in acompose.yml
file, without having to clone and build directly.New Docker deploy flow:
main
x.x.x
,x.x
,x
, andlatest
latest
is equivalent to the newest SemVer tag, whilemain
is equivalent to the main branchghcr.io/byu-cs-discord/csbot:latest
Startup time is a bit slow during command deployment, but we can speed that up later with a smarter deployment solution (one that edits existing commands rather than strict redeploying, which I think would fix the issue with command permissions resetting each time).
Note also that we run database migrations on startup, too.