fix CI latest Docker image tagging #413
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 Summary
Fix CI overwriting the
latest
Docker image with both portable and non-portable images (as seen here in the latest release CI action: https://github.com/flashbots/mev-boost/actions/runs/3428613593/jobs/5713128560)Currently, the
latest
image on Docker hub is the portable image.This PR updates the Docker release CI step to do that explicitly.
Also removes pushing images for
:1
and:1.4
when releasingv1.4.0
(these image tags seem unnecessary and I'd opt for removing them from Docker hub, otherwise we should also add them for portable).See also #412
✅ I have run these commands
make lint
make test-race
go mod tidy