Skip to content

[ACR] Add Docker image build and provenance attestations to ACR#156

Draft
chriscpaul wants to merge 12 commits into
github:mainfrom
chriscpaul:patch-1
Draft

[ACR] Add Docker image build and provenance attestations to ACR#156
chriscpaul wants to merge 12 commits into
github:mainfrom
chriscpaul:patch-1

Conversation

@chriscpaul

@chriscpaul chriscpaul commented May 27, 2026

Copy link
Copy Markdown

As part of the work to move sidecar images to ACR I have identified this repo as publishing an image that is also needed in ACR (Azure Container Registry). This work is to publish to ACR in it's own workflow as a test. Once this has been confirmed I will then move this into the release workflow.

This CI Check should not block merging if it fails

The is purely an addition to additional existing workflow logic/steps

Copilot AI review requested due to automatic review settings May 27, 2026 00:25
@chriscpaul chriscpaul requested review from a team and jkbschmid as code owners May 27, 2026 00:25
@chriscpaul chriscpaul marked this pull request as draft May 27, 2026 14:05
@chriscpaul

Copy link
Copy Markdown
Author

Need to test publishing to ACR via invoking the
.github/workflows/release_acr.yaml

will delete this after confirming it works

@chriscpaul chriscpaul marked this pull request as ready for review June 1, 2026 16:14
@chriscpaul

chriscpaul commented Jun 2, 2026

Copy link
Copy Markdown
Author

a fork does not have access to the original repo secrets so this will need to be merged in so I can perform this backfill and then bring these publish steps into the normal release.yml

@chriscpaul chriscpaul marked this pull request as draft June 3, 2026 20:19
- Remove separate release_acr.yaml test workflow
- Add ACR login, build+push, and attestation steps to release.yaml
- Both ghcr.io and ACR images are built on tag push in a single workflow
- Update checkout action to v6.0.3

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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