openmoji icon indicating copy to clipboard operation
openmoji copied to clipboard

Migrate to `hfg-gmuend` controlled font-builder docker image

Open JeppeKlitgaard opened this issue 1 year ago • 14 comments

At some point after #422 is merged it would be ideal to change over to use a docker image distributed through ghcr.io using this organisation rather than through @JeppeKlitgaard for security and maintainability purposes. There is no rush on this however.

The relevant dockerfile can be found in helpers/docker and a workflow is already set up in .github/workflows.

TODO:

  • [ ] Provide a README.md next to Dockerfile
  • [ ] Container building GitHub workflow
  • [ ] Multi-arch docker builds to support Mac M1

JeppeKlitgaard avatar Jan 06 '23 21:01 JeppeKlitgaard

Yes, let's do this once PR #422 is merged. Might need a few instructions how to set things up.

b-g avatar Jan 08 '23 09:01 b-g

Running the font-builder docker image on my M1 machine yields:

Status: Downloaded newer image for ghcr.io/jeppeklitgaard/font_builder:latest
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/arm64/v8) and no specific platform was requested

Not super familiar with the docker build pipeline ... but it would be nice to speed up the image on M1 computers. Building locally with docker build -t <image-name> --platform linux/x86_64apparently does the trick.

Would be it be possible to publish the image via the github action in addition as well for --platform linux/x86_64?

b-g avatar Jan 08 '23 10:01 b-g

Seems to be possible: Multi-platform images https://docs.docker.com/build/building/multi-platform/

b-g avatar Jan 08 '23 10:01 b-g

There is already a GitHub workflow in #422 to build the images. Adding multi-platform support to this should be possible.

I believe we simply need to set something up in the hfg-gmuend organisation to support this.

Surely it wouldn't be x86_64 for M1 Mac's but something ARM-related? Could be wrong on this.

JeppeKlitgaard avatar Jan 08 '23 20:01 JeppeKlitgaard

Great. Should I create a repo e.g. hfg-gmuend/openmoji-font-builder or hfg-gmuend/openmoji-font-buildsystem

b-g avatar Jan 10 '23 08:01 b-g

I think it would be easiest if the dockerfile is hosted in this repo (currently in helpers/docker/), but a separate repo is definitely also an option

JeppeKlitgaard avatar Jan 10 '23 12:01 JeppeKlitgaard

Ok, either way is fine. Let's decide once the new font buildsystem was merged.

b-g avatar Jan 10 '23 14:01 b-g

@JeppeKlitgaard PR #422 is finally merged! This is mega! Let us tackle the remaining related open todos ... what should I do? :)

b-g avatar Jan 13 '23 10:01 b-g

Hi @JeppeKlitgaard, hope you're doing well! Any chance to finish off this issue: move everything to the hfg-gmuend org and add x86_64 support? Ping :)

b-g avatar Oct 04 '23 14:10 b-g

Apologies for missing the ping!

I will try to take a look at this soon. I just started a new job and am still living out of a suitcase with family, so things are a bit hectic on my end at the moment. As I recall we need to do something to enable a container registry on the hfg-gmuend org, which I don't think I am able to do by myself. I will poke around a bit and we can coordinate

JeppeKlitgaard avatar Oct 04 '23 17:10 JeppeKlitgaard

@JeppeKlitgaard No worries, not super urgent ... just a gentle ping for news :) + All the best for settling in the new place and job!

As I recall we need to do something to enable a container registry on the hfg-gmuend org, which I don't think I am able to do by myself. I will poke around a bit and we can coordinate

Of course. Happy to setup a new repo, give you the necessary permissions ... just let me know.

b-g avatar Oct 05 '23 07:10 b-g

Just a heads up that this is still on my todo list, but the past few weeks have been quite hectic. Hopefully things will calm down soon!

JeppeKlitgaard avatar Oct 16 '23 20:10 JeppeKlitgaard

Many thanks for for letting us know, that is very nice of you. We are aiming to publish OpenMoji v15 in the next weeks. All emojis have been designed to complete with Unicode 15. I just have to find time to add them to library and polish a few small other things ... But as the font-builder docker image is not a blocker, no crazy rush on our end :) All the best

b-g avatar Oct 17 '23 07:10 b-g

Hi @JeppeKlitgaard, ping :) Any news? Tried it today ... bug gave up after 1h. Just not familiar enough with the nitty gritty docker details. I think I could build it via cd helpers/dockers and docker buildx build -f font_builder.Dockerfile --platform linux/amd64 --tag openmoji:latest --load . but then of course helpers/generate-fonts.shwas breaking ... any help would be great :) + Merry 🎄

b-g avatar Dec 20 '23 17:12 b-g