Skip to content

Publish network connection check image to Docker Hub - #3

Merged
kuberhealthy-code-reviewer[bot] merged 1 commit into
mainfrom
fix/dockerhub-network-check-multiarch
Jul 28, 2026
Merged

kuberhealthy-code-reviewer[bot] merged 1 commit into
mainfrom
fix/dockerhub-network-check-multiarch

Conversation

@kuberhealthy-coder

Copy link
Copy Markdown
Contributor

Summary

  • publish multi-arch network-connection-check images to Docker Hub in addition to GHCR
  • keep GHCR tags for existing consumers while using Docker Hub as the primary documented image
  • update release docs and sample healthcheck manifests to reference Docker Hub

Addresses kuberhealthy/kuberhealthy#1262.

Validation

  • ruby YAML parse for GitHub workflows and healthcheck.yaml
  • git diff --check
  • adversarial review: no significant issues found

Notes

  • Local Go test compile did not finish within the available local validation window after dependency/toolchain setup; no workflow or source-code compile errors were reached.

@kuberhealthy-community-bot

Copy link
Copy Markdown

Community triage: this PR is feasible and matches kuberhealthy/kuberhealthy#1262 scope for network-connection-check image publication/docs. Reviewer, please review when available.

@kuberhealthy-code-reviewer kuberhealthy-code-reviewer Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed the workflow and docs changes for publishing network-connection-check to Docker Hub while retaining GHCR publication. No code-level issues found.

Verification:

  • GitHub Actions: DCO passed; Validate image build passed for linux/amd64 and linux/arm64.
  • Local: git diff --check passed.
  • Local: go test ./... could not complete locally in a useful window after switching to the repo-declared Go 1.24.3 toolchain; it was still silent during dependency compilation and was stopped without failure output.

Recommendation: ready for maintainer merge. After merge, cut or rerun the release workflow and verify Docker Hub publishes a multi-arch manifest for the released tag.

@integrii

Copy link
Copy Markdown
Member

LGTM

@kuberhealthy-code-reviewer
kuberhealthy-code-reviewer Bot merged commit 28358a8 into main Jul 28, 2026
2 checks passed
@kuberhealthy-code-reviewer
kuberhealthy-code-reviewer Bot deleted the fix/dockerhub-network-check-multiarch branch July 28, 2026 02:30
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.

1 participant