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

buildx to create container image and load in local registry. #1938

Merged
merged 1 commit into from
Jan 11, 2024

Conversation

pjuarezd
Copy link
Member

@pjuarezd pjuarezd commented Jan 11, 2024

--platform linux/amd64,linux/arm64 --load does not work because local docker registry can't host different architectures

$(GOOS)/$(GOARCH) would also not work because causes an error pulling ubi image from redhat

ERROR: failed to solve: registry.access.redhat.com/ubi9/ubi-micro:latest: no match for platform in manifest: not found

`--platform linux/amd64,linux/arm64 --load` does not work because local docker registry can host different architectures

`$(GOOS)/$(GOARCH)` would also not work because causes an error pulling ubi image from redhat

```
ERROR: failed to solve: registry.access.redhat.com/ubi9/ubi-micro:latest: no match for platform in manifest: not found
```

Signed-off-by: pjuarezd <pjuarezd@users.noreply.github.com>
@pjuarezd pjuarezd requested review from shtripat and jiuker January 11, 2024 09:36
Copy link
Contributor

@shtripat shtripat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pretty needed for dev testing as well

Copy link
Contributor

@cesnietor cesnietor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I had to manually update it, thanks for the change.

@pjuarezd pjuarezd merged commit 345f3cb into minio:master Jan 11, 2024
25 checks passed
@pjuarezd pjuarezd deleted the makefile-build-image branch January 11, 2024 22:02
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.

3 participants