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

Merged
merged 8 commits into from
Aug 19, 2024

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.

@ggrossetie
Copy link
Member

@felixvanoost sent you an invite to join the @yuzutech organization, you should be able to transfer the repository

@felixvanoost
Copy link
Contributor Author

felixvanoost commented Jul 24, 2024

@ggrossetie I tried transferring ownership of the repo but I get an error message saying that I need permissions to create public repositories in @yuzutech first.

Untitled

@ggrossetie
Copy link
Member

@felixvanoost I've allowed repository creation, could you please try again? Ref: https://docs.github.com/en/organizations/managing-organization-settings/setting-permissions-for-deleting-or-transferring-repositories

@felixvanoost
Copy link
Contributor Author

@felixvanoost I've allowed repository creation, could you please try again? Ref: https://docs.github.com/en/organizations/managing-organization-settings/setting-permissions-for-deleting-or-transferring-repositories

I successfully transferred it last night. 🚀

@ggrossetie
Copy link
Member

@felixvanoost
Copy link
Contributor Author

@ggrossetie Good to go!

@ggrossetie
Copy link
Member

Many thanks for your work 🙇🏻

@ggrossetie ggrossetie merged commit d96fd25 into yuzutech:main Aug 19, 2024
1 check passed
@felixvanoost felixvanoost deleted the use-erd-native-image branch August 19, 2024 12:21
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