Skip to content

Stop publishing release images to GHCR#71

Merged
gontzess merged 1 commit into
mainfrom
gontzess/remove-ghcr-release-publishing
May 6, 2026
Merged

Stop publishing release images to GHCR#71
gontzess merged 1 commit into
mainfrom
gontzess/remove-ghcr-release-publishing

Conversation

@gontzess
Copy link
Copy Markdown
Contributor

@gontzess gontzess commented May 5, 2026

Why

Connector release images no longer need to be published to GHCR. Keeping the GHCR path means extra registry auth, extra image metadata, and extra attestation verification for an output that is no longer consumed.

What this changes

  • Removes the GHCR image from the Docker GoReleaser template.
  • Removes GHCR login from Docker publishing and release verification.
  • Records only ECR Public image metadata for public Docker images.
  • Removes GHCR image attestation verification from the release validation script.
  • Updates release workflow docs, protobuf comments, and the workflow diagram.

Validation

  • go test ./...
  • git diff --check
  • YAML parse for .github/workflows/release.yaml
  • Rendered .goreleaser-docker-oci-template.yaml.tmpl with sample values and parsed it as YAML
  • make protogen
  • make docs
  • Live release v0.1.122-test.8: Docker enabled, no GHCR login step, ECR Public publish, manifest publication, registry API recording, and release verification passed
  • Rebased onto main after Remove legacy release recording lambdas #69 merged; reran go test ./..., git diff --check, and YAML parse

@gontzess gontzess requested review from ennyjfrick, ggreer and kans May 5, 2026 20:52
Base automatically changed from gontzess/axiomatic-release-workflow-cleanup to main May 6, 2026 17:37
@gontzess gontzess force-pushed the gontzess/remove-ghcr-release-publishing branch from a986a71 to 10c2342 Compare May 6, 2026 17:45
@gontzess gontzess merged commit 840c50f into main May 6, 2026
1 check passed
@gontzess gontzess deleted the gontzess/remove-ghcr-release-publishing branch May 6, 2026 17:53
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