Skip to content

Improve repository and registry parameterization#5

Merged
chaptersix merged 2 commits intomainfrom
fix-ghcr-registry-prefix
Nov 21, 2025
Merged

Improve repository and registry parameterization#5
chaptersix merged 2 commits intomainfrom
fix-ghcr-registry-prefix

Conversation

@chaptersix
Copy link
Owner

This change makes Docker image publishing fully configurable while maintaining backward compatibility with both temporalio (Docker Hub) and forks (GHCR).

Changes:

  • Add workflow inputs: registry, registry_namespace, and image_name
  • Smart defaults: temporalio uses Docker Hub, forks use GHCR
  • Conditional authentication for multiple registries
  • Standardize image name to "temporal" everywhere
  • Proper handling of Docker Hub's no-prefix format

Workflow inputs (all optional with smart defaults):

  • registry: Container registry (docker.io, ghcr.io, etc.)
  • registry_namespace: Organization/user (defaults to repository_owner)
  • image_name: Image name (defaults to "temporal")

Default behavior:

  • temporalio/cli → docker.io/temporalio/temporal
  • forks → ghcr.io/{owner}/temporal

This design is PR-able to upstream while working for forks out of the box.

This change makes Docker image publishing fully configurable while maintaining
backward compatibility with both temporalio (Docker Hub) and forks (GHCR).

Changes:
- Add workflow inputs: registry, registry_namespace, and image_name
- Smart defaults: temporalio uses Docker Hub, forks use GHCR
- Conditional authentication for multiple registries
- Standardize image name to "temporal" everywhere
- Proper handling of Docker Hub's no-prefix format

Workflow inputs (all optional with smart defaults):
- registry: Container registry (docker.io, ghcr.io, etc.)
- registry_namespace: Organization/user (defaults to repository_owner)
- image_name: Image name (defaults to "temporal")

Default behavior:
- temporalio/cli → docker.io/temporalio/temporal
- forks → ghcr.io/{owner}/temporal

This design is PR-able to upstream while working for forks out of the box.
Keep the conditional expression approach (no local variables) to avoid HCL syntax errors.
@chaptersix chaptersix merged commit 09c343c into main Nov 21, 2025
5 of 6 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.

1 participant