Skip to content

fix(ci): sign OCI artifacts with both OCI 1.1 and legacy formats#1195

Merged
poiana merged 1 commit intofalcosecurity:mainfrom
c2ndev:ci/cosign-legacy-v2-signatures
Feb 4, 2026
Merged

fix(ci): sign OCI artifacts with both OCI 1.1 and legacy formats#1195
poiana merged 1 commit intofalcosecurity:mainfrom
c2ndev:ci/cosign-legacy-v2-signatures

Conversation

@c2ndev
Copy link
Contributor

@c2ndev c2ndev commented Feb 4, 2026

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area plugins

/area registry

/area build

/area documentation

What this PR does / why we need it:

After PR #1033 upgraded cosign-installer from v3 to v4, cosign v3 is now used which defaults to OCI 1.1 referrers format for signatures. This PR adds an additional signing step with --registry-referrers-mode=legacy to also produce signatures in the legacy tag-based format (sha256-<digest>.sig).

This ensures backward compatibility for consumers using older cosign versions or tools that expect the legacy signature format

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

@poiana
Copy link
Contributor

poiana commented Feb 4, 2026

Welcome @c2ndev! It looks like this is your first PR to falcosecurity/plugins 🎉

@poiana poiana requested review from irozzo-1A and leogr February 4, 2026 15:24
@poiana poiana added the size/XS label Feb 4, 2026
Signed-off-by: c2ndev <cannarella.dev@gmail.com>
@c2ndev c2ndev force-pushed the ci/cosign-legacy-v2-signatures branch from fcda162 to bf4e6e2 Compare February 4, 2026 15:30
@poiana poiana added the lgtm label Feb 4, 2026
@poiana
Copy link
Contributor

poiana commented Feb 4, 2026

LGTM label has been added.

DetailsGit tree hash: 4cabce56abca3762c1da1534dc7dd169267a02bf

@poiana
Copy link
Contributor

poiana commented Feb 4, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: c2ndev, leogr

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@poiana poiana added the approved label Feb 4, 2026
@poiana poiana merged commit 0c5514a into falcosecurity:main Feb 4, 2026
11 checks passed
@leogr
Copy link
Member

leogr commented Feb 4, 2026

/approve
/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants