Skip to content
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

Closed
wants to merge 5 commits into from

Conversation

AaronDewes
Copy link
Contributor

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 now image: ghcr.io/ride-the-lightning/rtl.

@ShahanaFarooqui
Copy link
Collaborator

@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.

@AaronDewes
Copy link
Contributor Author

it can affect RTL's integration for multiple providers like BTCPayServer, Umbrel, etc.

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 image: shahanafarooqui/rtl with image: ghcr.io/ride-the-lightning/rtl once the new version is out)

@AaronDewes
Copy link
Contributor Author

AaronDewes commented Apr 22, 2022

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)

@saubyk
Copy link
Collaborator

saubyk commented Apr 22, 2022

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.

@AaronDewes
Copy link
Contributor Author

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.

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.

@ShahanaFarooqui
Copy link
Collaborator

ShahanaFarooqui commented Oct 11, 2022

@AaronDewes I am unable to deploy the build on ghcr.io/ride-the-lightning/rtl.

I have created a branch named ci-test and updated the push condition on it as well. But I am still unable to successfully deploy it on Docker.

@AaronDewes
Copy link
Contributor Author

Weird, because the builds seem to work sucesfully. I'll try in my fork again

@ShahanaFarooqui ShahanaFarooqui modified the milestones: Release 0.14.1-beta, Release 0.14.2-beta Oct 4, 2023
@ShahanaFarooqui ShahanaFarooqui modified the milestones: Release 0.14.2-beta, Release 0.15.0-beta Oct 13, 2023
@saubyk saubyk removed this from the 0.15.1 milestone Jun 10, 2024
@ShahanaFarooqui
Copy link
Collaborator

Closing it in favour of PR #1421.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants