Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tests: Remove sigstore-rs client test #1447

Merged
merged 1 commit into from
Feb 6, 2025
Merged

Conversation

jku
Copy link
Member

@jku jku commented Feb 6, 2025

  • sigstore-rs does not support signing in GitHub Actions so we need to test verify only with a bundle made by another client
  • There is a signature bundle available for this purpose. It's produced by sigstore-python so is bundle v0.3: sigstore-rs only supports <=0.2

We've tested manually and the test commands succeed when the bundle is v0.2 but we're not keen to make the tests more complicated: Remove sigstore-rs tests for now.

* sigstore-rs does not support signing in GitHub Actions so we need to
  test verify with a bundle made by another client
* There is a signature bundle available for this purpose. It's produced
  by sigstore-python so is bundle v0.3: sigstore-rs only support <=0.2

We've tested manually and the test commands succeed when the bundle is
v0.2 but we're not keen to make the tests more complicated: Remove
sigstore-rs tests for now.

Signed-off-by: Jussi Kukkonen <[email protected]>
@jku jku requested a review from kommendorkapten February 6, 2025 09:27
@jku jku merged commit d7ee322 into sigstore:main Feb 6, 2025
3 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.

2 participants