Skip to content

remove Docker Hub publish workflow#3

Merged
tnsardesai merged 1 commit into
mainfrom
hypeship/remove-dockerhub-publish
Jun 9, 2026
Merged

remove Docker Hub publish workflow#3
tnsardesai merged 1 commit into
mainfrom
hypeship/remove-dockerhub-publish

Conversation

@tnsardesai

@tnsardesai tnsardesai commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Summary

ECR is the only registry we deploy from now. This deletes .github/workflows/docker.yml which built and pushed the image to Docker Hub on every main push — nothing consumes that image anymore.

Follow-up (not in this PR)

  • delete repo secrets DOCKERHUB_USERNAME and DOCKERHUB_TOKEN
  • scale the Railway service to 0, then delete it after a hold window

Test plan

  • confirm build.yml (ECR) still runs and pushes on next main push
  • confirm deploy.yml cross-repo dispatch still fires the AWS deploy

Note

Low Risk
CI-only removal of an unused Docker Hub pipeline; no application or deploy logic changes in this diff.

Overview
Removes the Docker Hub CI workflow (.github/workflows/docker.yml) that previously built the nvidia-license-server-exporter image on PRs and pushed latest / SHA tags to Docker Hub on main pushes.

Deployment is now expected to rely solely on the existing ECR build path (build.yml) and AWS deploy dispatch (deploy.yml); this PR does not change those workflows.

Reviewed by Cursor Bugbot for commit b488867. Bugbot is set up for automated code reviews on this repo. Configure here.

ECR is the only registry we deploy from now. Deleting the Docker Hub
publish path stops producing images that nothing consumes.
@tnsardesai tnsardesai marked this pull request as ready for review June 9, 2026 18:34
@firetiger-agent

Copy link
Copy Markdown

Firetiger deploy monitoring skipped

This PR didn't match the auto-monitor filter configured on your GitHub connection:

PRs in the kernel, infra, hypeman, and hypeship repos. kernel is a ~mono repo with many logical services underneath, ensure to focus on the implicated service for the PR

Reason: PR repo not identified in provided information; cannot confirm it belongs to kernel, infra, hypeman, or hypeship.

To monitor this PR anyway, reply with @firetiger monitor this.

@tnsardesai tnsardesai requested review from Sayan- and rgarcia June 9, 2026 18:34
@tnsardesai tnsardesai merged commit 51bee4c into main Jun 9, 2026
3 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