Skip to content

chore: push image to latest on tags#398

Closed
sazap10 wants to merge 4 commits intomainfrom
also_push_latest_image_ghcr
Closed

chore: push image to latest on tags#398
sazap10 wants to merge 4 commits intomainfrom
also_push_latest_image_ghcr

Conversation

@sazap10
Copy link
Copy Markdown
Contributor

@sazap10 sazap10 commented Mar 26, 2026

Goal

Push github image to latest tag as part of publish step, also allow configuring builder image for local builds when using zscaler

Design

Allow using in development with latest image

Changeset

  • Update publish github workflow to push latest tag
  • Update builder image to be overridden

Testing

No functional change

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 26, 2026

🎯 Coverage Target Met!

📈 Coverage Metrics

Metric Coverage Target Status
Lines 93.5% 80%
Functions 85.9% 80%
Branches 86.7% 80%
Statements 93.5% 80%

📊 Test Statistics

  • Total Lines: 20,688 / 22,119
  • Total Functions: 318 / 370
  • Total Branches: 937 / 1081
  • Total Statements: 20688 / 22119

🔍 Files Needing Coverage

File Coverage
src/reflect/prompt/sap-test.ts 9.5%
src/qmetry/client/testsuite.ts 10.3%
src/common/prompts.ts 16.7%
src/qmetry/client/issues.ts 20.1%
src/swagger/client/api.ts 30.0%

📝 Report generated on Node.js v24.13.1 • View workflow • Coverage by Vitest + v8

Copy link
Copy Markdown
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 container publishing workflow so that, after publishing a versioned multi-arch image to GHCR, it also publishes a :latest multi-arch manifest pointing at the same architecture-specific images.

Changes:

  • Add creation of a GHCR :latest manifest list using the existing ${IMAGE_TAG}-amd64 / ${IMAGE_TAG}-arm64 images
  • Push the :latest manifest to GHCR as part of the create-manifest job

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

Comment on lines +173 to +178
# Create and push 'latest' tag manifest list
docker manifest create "${GHCR_REGISTRY}:latest" \
--amend "${GHCR_REGISTRY}:${IMAGE_TAG}-amd64" \
--amend "${GHCR_REGISTRY}:${IMAGE_TAG}-arm64"
docker manifest push "${GHCR_REGISTRY}:latest"
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

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

Pushing the :latest manifest is currently unconditional within create-manifest. Since this workflow also runs on workflow_dispatch and on any release publish (including pre-releases), this can unintentionally overwrite ghcr.io/<repo>:latest with a manual/dev build or a pre-release image. Consider gating this step (or adding a separate step) so latest is only pushed for non-prerelease releases (e.g., if: github.event_name == 'release' && github.event.release.prerelease == false) and/or behind an explicit workflow_dispatch input.

Copilot uses AI. Check for mistakes.
@sazap10 sazap10 changed the title chore: also push image to latest on tags chore: push image to latest on tags Mar 26, 2026
@sazap10
Copy link
Copy Markdown
Contributor Author

sazap10 commented Mar 30, 2026

bundled into #330

@sazap10 sazap10 closed this Mar 30, 2026
@sazap10 sazap10 deleted the also_push_latest_image_ghcr branch March 30, 2026 10:06
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