Skip to content

Conversation

@chaptersix
Copy link
Contributor

@chaptersix chaptersix commented Nov 20, 2025

What was changed

  • added GHA to build and publish docker images
  • Updated go releaser to v2
  • Minor tweaks to other GHA

Why?

Make is easier to publish docker images on release.

@CLAassistant
Copy link

CLAassistant commented Nov 20, 2025

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@chaptersix chaptersix marked this pull request as ready for review November 21, 2025 12:47
@chaptersix chaptersix requested review from a team as code owners November 21, 2025 12:47
@chaptersix chaptersix marked this pull request as draft November 21, 2025 13:00
Major improvements:
- Add full parameterization (registry, namespace, image name)
- Auto-detect registry: temporalio → docker.io, others → ghcr.io
- Separate workflow for managing 'latest' tag on release events
- Dynamic Docker labels using GITHUB_REPOSITORY variable
- Add packages:write permission for GHCR
- Remove artifact uploading (no longer needed)

Benefits:
- Works out-of-box for both upstream and forks
- Flexible registry support (Docker Hub, GHCR, any registry)
- Clean separation of release vs latest-tag concerns
- Proper package association in GitHub

Configuration requirements:
- DOCKER_USERNAME and DOCKER_PASSWORD secrets needed for Docker Hub
- GITHUB_TOKEN automatically provides GHCR access
@chaptersix chaptersix marked this pull request as ready for review November 21, 2025 17:49
chaptersix added a commit to chaptersix/temporal-cli that referenced this pull request Nov 22, 2025
Address PR temporalio#877 Feedback: Simplify Dockerfile and Workflows
@semgrep-managed-scans
Copy link

Semgrep found 1 missing-explicit-permissions finding:

No explicit GITHUB_TOKEN permissions found at the workflow or job level. Add a permissions: block at the workflow root (applies to all jobs) or per job with least privilege (e.g., contents: read and only specific writes like pull-requests: write if needed).

@chaptersix chaptersix requested a review from bergundy November 26, 2025 19:24
@chaptersix chaptersix marked this pull request as draft November 26, 2025 22:02
@chaptersix chaptersix marked this pull request as ready for review December 1, 2025 16:59
Copy link
Member

@bergundy bergundy left a comment

Choose a reason for hiding this comment

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

Overall LGTM. Didn't have anything major.

@chaptersix chaptersix requested a review from bergundy December 2, 2025 22:54
@chaptersix
Copy link
Contributor Author

just realized that this repo uses release candidates and those are not currently published to docker hub.

@chaptersix chaptersix marked this pull request as draft December 3, 2025 00:23
@chaptersix
Copy link
Contributor Author

moving the image build to the manually triggered workflow.

@chaptersix chaptersix marked this pull request as ready for review December 3, 2025 15:11
@chaptersix chaptersix merged commit da6d754 into main Dec 4, 2025
8 checks passed
@chaptersix chaptersix deleted the alex/docker-image-re branch December 4, 2025 17:45
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.

4 participants