Sitelet https://github.com/redis-stack/redis-stack/pull/320
Skip to content

Multi-platform docker build fix - #320

Merged
chayim merged 6 commits into
masterfrom
dockarm
Jan 25, 2023
Merged

chayim merged 6 commits into
masterfrom
dockarm

Conversation

@chayim

@chayim chayim commented Jan 23, 2023 •

Copy link
Copy Markdown
Contributor

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

chayim commented Jan 25, 2023

Copy link
Copy Markdown
Contributor Author

Found the source of thie issue, in docker buildx

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

@chayim

chayim commented Jan 25, 2023

Copy link
Copy Markdown
Contributor Author

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 requested a review from ofekshenawa January 25, 2023 07:06
@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.

2 participants