Skip to content

Remove unused ephemeral signer - #4938

Merged
steiza merged 1 commit into
sigstore:mainfrom
aaronlew02:delete-ephemeral
Jun 12, 2026
Merged

Remove unused ephemeral signer#4938
steiza merged 1 commit into
sigstore:mainfrom
aaronlew02:delete-ephemeral

Conversation

@aaronlew02

Copy link
Copy Markdown
Contributor

Summary

This change removes the unused ephemeralSigner implementation.

Signed-off-by: Aaron Lew <64337293+aaronlew02@users.noreply.github.com>
@aaronlew02
aaronlew02 requested a review from a team as a code owner June 10, 2026 14:07
@codecov

codecov Bot commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.79%. Comparing base (2ef6022) to head (19144be).
⚠️ Report is 775 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4938      +/-   ##
==========================================
- Coverage   40.10%   38.79%   -1.31%     
==========================================
  Files         155      221      +66     
  Lines       10044    13581    +3537     
==========================================
+ Hits         4028     5269    +1241     
- Misses       5530     7547    +2017     
- Partials      486      765     +279     

☔ View full report in Codecov by Harness.
📢 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.

@steiza steiza 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.

LGTM!

@steiza

steiza commented Jun 10, 2026

Copy link
Copy Markdown
Member

It looks like unrelated to this pull request there might be an issue in https://github.com/sigstore/scaffolding?

From the failed e2e workflow run:

dependency failed to start: container rekor-tiles-witness-1 is unhealthy

@Hayden-IO

Copy link
Copy Markdown
Contributor

transparency-dev/witness#548 might have broken the test, since we're pulling in the witness from HEAD.

@Hayden-IO

Copy link
Copy Markdown
Contributor

I'm looking into the witness failure now, it's impacting local testing as well.

@Hayden-IO

Copy link
Copy Markdown
Contributor

oh hah it was just the poll_interval flag was dropped. We don't use that. Will fix in scaffolding.

@Hayden-IO

Copy link
Copy Markdown
Contributor

sigstore/rekor-tiles#826

@steiza
steiza merged commit b1dd2e9 into sigstore:main Jun 12, 2026
40 of 42 checks passed
@aaronlew02
aaronlew02 deleted the delete-ephemeral branch June 12, 2026 14:01
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