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

Add docker buildx section and use the official install guide for buildah #30

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hydai
Copy link
Member

@hydai hydai commented May 30, 2023

No description provided.

@hydai hydai requested a review from juntao May 30, 2023 04:55
Copy link
Member

juntao commented May 30, 2023

Hello, I am a code review bot on flows.network. Here are my reviews of code commits in this PR.


Commit 77676810f3d1cd9608342924a1c57d64342454e6

The pull request adds instructions on how to use Docker buildx and Buildah to create and publish a container image. It also includes a note regarding the use of self-defined annotation, which is still a new feature in Buildah. The main potential problem is that the instructions assume the reader is running Ubuntu zesty or xenial. It would be better to provide instructions that work for a wider range of Linux distributions, or at least provide alternative instructions for other distros. Another issue is that the authentication details for push command are hardcoded, so the user would need to manually replace them. It would be better to provide instructions on how to retrieve the authentication information or how to configure it programmatically.

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.

2 participants