-
Notifications
You must be signed in to change notification settings - Fork 159
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
Build Docker containers on GitHub actions #979
Conversation
@AaronDewes Thank you for the PR. It will be really useful for the project, but it can affect RTL's integration for multiple providers like BTCPayServer, Umbrel, etc. We are keeping the PR open until we are not done with in-depth impact analysis. |
There should be no big issues here, the only required change is a one-time change of one line, and existing users won't have any issues, because it'll only affect future versions. But please let me know if you see any other impact this could have. I'm developing an umbrel alternative @runcitadel, and our app system is docker-compose based, so I have some experience with this and there should be no big effort to migrate for external developers by doing this. (Literally just replacing |
For BTCPay, it has a slightly more complicated build system, but it's not that hard to implement either: AaronDewes/btcpayserver-docker@a787f31 (This commit won't work right now, because the new Docker config is not merged) |
Hi @AaronDewes thanks for the PR. Ideally we should generate the images both via Github actions and Circle CI and give the node solution providers time to transition to Github actions eventually. So basically we should be deprecating Circle CI/Docker Hub images and eventually phase it out. |
I don't really understand why the node providers should need transition time, it's a one-time one-line change needed and the old images will still exists, so it's only one step when upgrading. However, it's also possible to push to Docker hub from GitHub actions, so the new, simpler Dockerfile would be used and only one ci provider, and the old images would still get updated. |
@AaronDewes I am unable to deploy the build on I have created a branch named |
Weird, because the builds seem to work sucesfully. I'll try in my fork again |
Closing it in favour of PR #1421. |
This builds Docker containers from one Dockerfile, on GitHub actions, to the GitHub container registry.
It uses buildx and the built image will automatically run on arm64, amd64 and armv7 (3 images get pushed and then commpiled into one manifest for all architectures, so every arch can download the right one).
This shouldn't have any impact on anyone using the images, except instead of putting
image: shahanafarooqui/rtl
into docker-compose.yml, it's nowimage: ghcr.io/ride-the-lightning/rtl
.