Skip to content

Comments

ci: unify PR workflows and fix multiarch images#63

Merged
tembleking merged 6 commits intomainfrom
ci/push-pr-test-image
Feb 10, 2026
Merged

ci: unify PR workflows and fix multiarch images#63
tembleking merged 6 commits intomainfrom
ci/push-pr-test-image

Conversation

@tembleking
Copy link
Member

@tembleking tembleking commented Feb 9, 2026

Summary

  • Merge test.yaml and test_image.yaml into a single pull-request-ci.yaml with sequential jobs to avoid Nix cache rate limits
  • Push multi-arch test images to GHCR tagged pr-<PR_NUMBER> on pull requests (no latest)
  • Fix digest file concatenation bug (missing trailing newline) in both pull-request-ci and publish.yaml
  • Add max-parallel: 1 to publish.yaml build matrix

@tembleking tembleking requested a review from a team as a code owner February 9, 2026 16:12
Copilot AI review requested due to automatic review settings February 9, 2026 16:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the CI test image workflow to publish PR-scoped, multi-arch images to GHCR using per-arch digest uploads and a manifest list.

Changes:

  • Adds a Nix-based default shell and GHCR package write permissions to the test build job.
  • Pushes per-arch images by digest and uploads those digests as artifacts.
  • Introduces a follow-up job to assemble/push a multi-arch manifest tagged pr-<PR_NUMBER>.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Merge test.yaml and test_image.yaml into pull-request-ci.yaml with
sequential jobs to avoid Nix cache rate limits. Push test images
tagged pr-<id> on pull requests. Fix digest file concatenation bug
in both pull-request-ci and publish workflows.
@tembleking tembleking changed the title ci: push test image tagged pr-<id> on pull requests ci: unify PR workflows and push test image tagged pr-<id> Feb 10, 2026
Copilot AI review requested due to automatic review settings February 10, 2026 09:29
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Replace skopeo copy (which defaults to :latest) with
docker/build-push-action using push-by-digest=true in both
pull-request-ci and publish workflows.
Copilot AI review requested due to automatic review settings February 10, 2026 10:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tembleking tembleking changed the title ci: unify PR workflows and push test image tagged pr-<id> ci: unify PR workflows Feb 10, 2026
@tembleking tembleking changed the title ci: unify PR workflows ci: unify PR workflows and fix multiarch images Feb 10, 2026
@tembleking tembleking enabled auto-merge (squash) February 10, 2026 11:03
@tembleking tembleking merged commit 71ee2c5 into main Feb 10, 2026
6 checks passed
@tembleking tembleking deleted the ci/push-pr-test-image branch February 10, 2026 11:07
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