KEMBAR78
workflows/docker: add Docker Hub push fallback by p-linnane · Pull Request #20838 · Homebrew/brew · GitHub
Skip to content

Conversation

p-linnane
Copy link
Member

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes? Here's an example.
  • Have you successfully run brew style with your changes locally?
  • Have you successfully run brew typecheck with your changes locally?
  • Have you successfully run brew tests with your changes locally?

We often get 400 errors even with retries when using docker buildx imagetools to push to Docker Hub. Let's try to fall back to a manual push in case that happens.

Signed-off-by: Patrick Linnane <patrick@linnane.io>
@p-linnane p-linnane requested a review from Copilot October 8, 2025 18:03
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a fallback mechanism for Docker Hub push operations when the primary docker buildx imagetools create command fails. Instead of exiting with an error after 3 attempts, the workflow now falls back to manually pulling, tagging, and pushing images to Docker Hub.

  • Replaces hard failure after 3 attempts with a manual push fallback
  • Implements a loop to pull source images and tag them with target tags
  • Uses standard docker push commands as an alternative to buildx imagetools

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@p-linnane p-linnane requested a review from Copilot October 8, 2025 18:07
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@p-linnane p-linnane added this pull request to the merge queue Oct 9, 2025
@p-linnane p-linnane removed this pull request from the merge queue due to a manual request Oct 9, 2025
github-merge-queue bot pushed a commit that referenced this pull request Oct 9, 2025
workflows/docker: add Docker Hub push fallback
@p-linnane p-linnane added this pull request to the merge queue Oct 9, 2025
Merged via the queue into main with commit 6bc3220 Oct 9, 2025
38 checks passed
@p-linnane p-linnane deleted the docker-hub-fallback branch October 9, 2025 16:51
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