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

Multi-platform docker build fix #320

Merged
merged 6 commits into from
Jan 25, 2023
Merged

Multi-platform docker build fix #320

merged 6 commits into from
Jan 25, 2023

Conversation

chayim
Copy link
Contributor

@chayim chayim commented Jan 23, 2023

The docker builds fail, due to docker notiing that an image is a manifest. The docker API notes, when pulling edge dockers that edge-x86_64 is a manifest - though it's clearly an image. When running an inspect on an image, or pulling via tag, or sha - docker reports that Error: No such image: redisfab/redis-stack-server:edge-x86_64, yet in other cases, indicates that this is a manifest.

@chayim chayim changed the title disabling arm docker test Redisabling arm docker tests Jan 24, 2023
@chayim chayim changed the title Redisabling arm docker tests Multi-platform docker build fix Jan 25, 2023
@chayim
Copy link
Contributor Author

chayim commented Jan 25, 2023

Found the source of thie issue, in docker buildx

A bug in setup-buildx-action helped highlight the fix, a downgrade to buildx.

@chayim
Copy link
Contributor Author

chayim commented Jan 25, 2023

In CI: ensuring just these tests pass: https://github.com/redis-stack/redis-stack/actions/runs/3998114095

Validation of the separate edge build run (not using buildx): https://github.com/redis-stack/redis-stack/actions/runs/3995339550.

Note - as part of this we re-disabled the arm docker checks. Those were enabled in a bad merge, hence putting these two together.

@chayim chayim merged commit ee0ee0c into master Jan 25, 2023
@chayim chayim deleted the dockarm branch January 25, 2023 08:34
This was referenced Jan 25, 2023
@chayim chayim mentioned this pull request Aug 15, 2023
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.

None yet

2 participants