Skip to content

.github/actions: login with cosign to sign helm OCI charts#43782

Merged
joestringer merged 1 commit intomainfrom
pr/fix-cosign-sign
Jan 15, 2026
Merged

.github/actions: login with cosign to sign helm OCI charts#43782
joestringer merged 1 commit intomainfrom
pr/fix-cosign-sign

Conversation

@aanm
Copy link
Copy Markdown
Member

@aanm aanm commented Jan 15, 2026

As helm uses a different file to login into the registries, we also need to use cosign login so the oci helm charts can be properly signed by cosign.

Fixes: 7494f7c ("workflows/release: release helm charts as OCI packages")

As helm uses a different file to login into the registries, we also need
to use cosign login so the oci helm charts can be properly signed by
cosign.

Fixes: 7494f7c ("workflows/release: release helm charts as OCI packages")
Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm requested review from a team as code owners January 15, 2026 09:33
@aanm aanm added the needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch label Jan 15, 2026
@aanm aanm requested a review from nbusseneau January 15, 2026 09:33
@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Jan 15, 2026
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jan 15, 2026

/test

@aanm aanm enabled auto-merge January 15, 2026 09:34
@aanm aanm added the release-note/misc This PR makes changes that have no direct user impact. label Jan 15, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jan 15, 2026
Copy link
Copy Markdown
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

LGTM thanks!

@aanm aanm added this pull request to the merge queue Jan 15, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 15, 2026
@joestringer joestringer added this pull request to the merge queue Jan 15, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 15, 2026
@joestringer
Copy link
Copy Markdown
Member

This has been kicked out of the merge queue a couple of times. Most likely cause is ongoing GitHub outage. Let's retry later. https://www.githubstatus.com

@joestringer joestringer added this pull request to the merge queue Jan 15, 2026
Merged via the queue into main with commit 561161f Jan 15, 2026
386 of 392 checks passed
@joestringer joestringer deleted the pr/fix-cosign-sign branch January 15, 2026 19:59
@giorio94 giorio94 mentioned this pull request Jan 19, 2026
8 tasks
@giorio94 giorio94 added backport-pending/1.19 The backport for Cilium 1.19.x for this PR is in progress. and removed needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch labels Jan 19, 2026
@github-actions github-actions bot added backport-done/1.19 The backport for Cilium 1.19.x for this PR is done. and removed backport-pending/1.19 The backport for Cilium 1.19.x for this PR is in progress. labels Jan 19, 2026
@cilium-release-bot cilium-release-bot bot moved this to Released in cilium v1.19.0 Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-done/1.19 The backport for Cilium 1.19.x for this PR is done. release-note/misc This PR makes changes that have no direct user impact.

Projects

No open projects
Status: Released

Development

Successfully merging this pull request may close these issues.

4 participants