Skip to content

Drop Docker Hub image job; keep GHCR - #12

Merged
antonengelhardt merged 1 commit into
mainfrom
ci/drop-dockerhub
Sep 5, 2026
Merged

antonengelhardt merged 1 commit into
mainfrom
ci/drop-dockerhub

Conversation

@antonengelhardt

Copy link
Copy Markdown
Owner

Summary

  • Removes the docker-image job from Test / Build / Release. It logged into Docker Hub with empty secrets and failed every Dependabot PR (Username and password required).
  • GHCR push stays. That is what ae-k8s-04 pulls.

Test plan

  • Test workflow no longer has docker-image
  • ghcr-image still runs on PRs and on main

Made with Cursor

The docker-image job failed every PR: DOCKERHUB_USERNAME/PASSWORD are unset.

Co-authored-by: Cursor <cursoragent@cursor.com>
Copilot AI lite review requested due to automatic review settings September 5, 2026 15:09

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The change cleanly removes a consistently failing Docker Hub publish job without impacting the existing GHCR build/push path.

Pull request overview

This PR removes the Docker Hub image publishing job from the CI workflows to stop failures caused by missing Docker Hub credentials, while keeping the GHCR image publishing path used by downstream consumers.

Changes:

  • Removed the docker-image job from test.yml, build.yml, and release.yml.
  • Kept the existing ghcr-image jobs intact for PRs / main / tagged releases.
File summaries
File Description
.github/workflows/test.yml Removes Docker Hub publish job that was failing PR runs; GHCR publish remains for internal PRs.
.github/workflows/build.yml Drops Docker Hub publish on main builds; preserves GHCR latest image push.
.github/workflows/release.yml Removes Docker Hub publish on tag releases; preserves GHCR version + latest image push.
Review details
  • Files reviewed: 3/3 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@antonengelhardt
antonengelhardt merged commit 1e1a577 into main Sep 5, 2026
14 checks passed
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