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

Use pre-built Docker image for erd #1754

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

felixvanoost
Copy link
Contributor

Replaces the kroki-builder-erd Docker image used to build the erd binary with a pre-built image published upstream. This resolves the issue with using erd on ARM64 machines described in #1239 and #1084, and should slightly reduce the build time of the Kroki Docker images.

The maintainer of https://github.com/BurntSushi/erd merged my PR to publish a linux/arm64 Docker image in October 2023 but didn't complete setting up the required ARM64 runner. There's been no further progress since then, so I've created a fork of the project and published the images myself.

@ggrossetie I'm happy to transfer ownership of the fork to you if you'd like to maintain control over it.

Closes #1239
Closes #1084

@felixvanoost felixvanoost changed the title Use pre-built erd Docker image Use pre-built Docker image for erd May 31, 2024
@ggrossetie
Copy link
Member

Hey! thanks for moving this issue forward.

I'm not familiar with the GitHub container repository. Do I need to do something special to activate it?
I guess I will need to update the GitHub Action to use my own ARM64 runner right? Currently, the build is using buildjet-4vcpu-ubuntu-2204-arm. I guess I won't have access to this runner once the project has been transferred to the yuzutech org?

Also, I've noticed that you have a unknown/unknown arch:
image

@felixvanoost
Copy link
Contributor Author

The GitHub container registry should be enabled by default; you don't have to do anything special. All you should need to do is change to your local ARM runner as you described, then update the URL of the image in the Dockerfile.

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

Successfully merging this pull request may close these issues.

Error generating erd with Docker on ARM Docker containers for more platforms
2 participants