Skip to content

test/assets: remove rekor v2 dsse/0.0.2 bundle-verify fixtures#374

Merged
jku merged 1 commit into
sigstore:mainfrom
codysoyland:remove-rekor2-dsse-fixtures
May 25, 2026
Merged

test/assets: remove rekor v2 dsse/0.0.2 bundle-verify fixtures#374
jku merged 1 commit into
sigstore:mainfrom
codysoyland:remove-rekor2-dsse-fixtures

Conversation

@codysoyland

@codysoyland codysoyland commented May 19, 2026

Copy link
Copy Markdown
Member

Summary

Removes the four rekor2-dsse-* bundle-verify fixtures, whose Rekor TLog entries use kind=dsse/0.0.2.

Upcoming client releases drop verifier support for the dsse Rekor entry type in favor of representing DSSE envelopes as hashedrekord (see architecture-docs PR #63, rekor-v2-spec §6.1.4). Those client PRs are currently blocked by these fixtures — e.g. sigstore-python#1776 fails with Unsupported DSSE log entry type: dsse/0.0.2 on test_verify[*-rekor2-dsse-happy-path].

Removing these fixtures here lets a new conformance release ship ahead of the client changes so client CI stays green. Replacement hashedrekord-DSSE fixtures will land in #371.

The four pre-existing kind=dsse/0.0.1 fixtures (dsse-*_fail, happy-path-intoto-in-dsse-v3) are intentionally left in place; they can be retired separately if/when clients drop 0.0.1 support too.

Removed

  • test/assets/bundle-verify/rekor2-dsse-happy-path
  • test/assets/bundle-verify/rekor2-dsse-invalid-sig_fail
  • test/assets/bundle-verify/rekor2-dsse-mismatch-envelope_fail
  • test/assets/bundle-verify/rekor2-dsse-mismatch-sig_fail

The rekor2-dsse-* fixtures embed a Rekor TLog entry with
kind=dsse/0.0.2. Upcoming sigstore-go and sigstore-python releases
drop verifier support for the dsse entry type in favor of representing
DSSE envelopes as hashedrekord (see architecture-docs PR sigstore#63,
rekor-v2-spec section 6.1.4). Removing these fixtures here lets a new
conformance release ship ahead of the client changes so client CI
stays green. Replacement hashedrekord-DSSE fixtures will land in a
follow-up PR.

Removed:
  - test/assets/bundle-verify/rekor2-dsse-happy-path
  - test/assets/bundle-verify/rekor2-dsse-invalid-sig_fail
  - test/assets/bundle-verify/rekor2-dsse-mismatch-envelope_fail
  - test/assets/bundle-verify/rekor2-dsse-mismatch-sig_fail
Signed-off-by: Cody Soyland <cody.soyland@chainguard.dev>
@codysoyland codysoyland marked this pull request as ready for review May 19, 2026 14:48
@Hayden-IO Hayden-IO requested a review from loosebazooka May 19, 2026 15:04

@loosebazooka loosebazooka left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yeah I think this is a fine transition, but it may not be too much to ask clients to just add xfails and figure it out. Either way, lets get this moving. I'm fine with this approach.

@loosebazooka

Copy link
Copy Markdown
Member

Oh do you need one of us to merge?

@loosebazooka

Copy link
Copy Markdown
Member

@jku no concerns? I'll merge it

@jku jku left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

sounds good

@jku jku merged commit e2cc8e5 into sigstore:main May 25, 2026
5 checks passed
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.

3 participants