johanhelsing / matchbox

Painless peer-to-peer WebRTC networking for rust wasm (and native!)
Apache License 2.0
918 stars 76 forks source link

Feature: matchbox_server docker image GitHub action #420

Open AgustinRamiroDiaz opened 9 months ago

AgustinRamiroDiaz commented 9 months ago

Part of https://github.com/johanhelsing/matchbox/issues/372

What

Adds this github action configured for this repo

Testing done

You can see the triggers of my tests

The pushed images are here

Configuration from your side

Requires creating an access token in Docker Hub, and configuring the secret DOCKERHUB_TOKEN and the variable DOCKERHUB_USER in this repo

image

image

simbleau commented 9 months ago

Wow, thanks for this!

johanhelsing commented 8 months ago

We are already building and pushing images to the github container registry and then manually pushing to dockerhub.

With this PR, we'd build the server image twice, though. Would be good if there was just one image built per version. Also not sure if continuing to push to ghcr.io is valuable or not.

Maybe we should just choose one container registry, and either drop docker or ghcr? Either way, getting rid of the manual steps would be great.

AgustinRamiroDiaz commented 8 months ago

@johanhelsing oh I haven't seen that before, it's under the code workflow.

I can do whatever you think it's best. We could merge the 2 workflows into the code one and keep building + pushing to both registries there, or if you plan on dropping support we can also move to the current implementation. Let me know so I update the PR :D

johanhelsing commented 8 months ago

Thanks, I think it's best to push to both registries, if not too complicated. I think the image should build on all pr runs, but only push on tags.

I think the push on tag path might be broken, perhaps the workflow is not triggering on tags anymore... at least I had to do it manually last release.

Thanks for working on this!

AgustinRamiroDiaz commented 8 months ago

I think the push on tag path might be broken, perhaps the workflow is not triggering on tags anymore... at least I had to do it manually last release.

@johanhelsing I think that was because tags were not taken into account in the workflow triggers. I'll fix that too in this PR

Thanks for working on this!

You're welcome :)