Skip to content
This repository was archived by the owner on Jan 19, 2026. It is now read-only.

ci!: Update signing for sigstore v3#1401

Merged
flavio merged 1 commit intokubewarden:mainfrom
viccuad:ci/sigstore-v3
Oct 22, 2025
Merged

ci!: Update signing for sigstore v3#1401
flavio merged 1 commit intokubewarden:mainfrom
viccuad:ci/sigstore-v3

Conversation

@viccuad
Copy link
Copy Markdown
Member

@viccuad viccuad commented Oct 22, 2025

Description

With the move to sigstore v3, instead of saving the cert and sig on each signing operation, we save an X.bundle.sigstore.

Test

Testing on my fork:
https://github.com/viccuad/kwctl/actions/runs/18715036597
https://github.com/viccuad/kwctl/actions/runs/18716654959

Additional Information

Tradeoff

Potential improvement

@codecov
Copy link
Copy Markdown

codecov bot commented Oct 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.67%. Comparing base (3d6a52e) to head (7d7a040).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1401      +/-   ##
==========================================
- Coverage   86.74%   86.67%   -0.07%     
==========================================
  Files          34       34              
  Lines        4495     4495              
==========================================
- Hits         3899     3896       -3     
- Misses        596      599       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

With the move to sigstore v3, instead of saving the cert and sig on each
signing operation, we save an `X.bundle.sigstore`.

Signed-off-by: Víctor Cuadrado Juan <[email protected]>
@viccuad
Copy link
Copy Markdown
Member Author

viccuad commented Oct 22, 2025

Fixed a Windows error and repushed.

@flavio flavio merged commit 11f43b6 into kubewarden:main Oct 22, 2025
18 of 19 checks passed
@github-project-automation github-project-automation bot moved this from Pending review to Done in Kubewarden Admission Controller Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

Development

Successfully merging this pull request may close these issues.

2 participants