-
Notifications
You must be signed in to change notification settings - Fork 63
Update staging-root.json #594
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
Conversation
Change hardcoded root to the new root from sigstore/root-signing#756 Signed-off-by: Hayden B <hblauzvern@google.com>
Note this hasn't been rolled out to staging yet, so it may fail. |
/gcbrun |
It'll be ~10 minutes til I update staging |
Updated the root, please kick off GHA tests again |
Still have a lot of test failures here, haven't looked into them yet: https://github.com/sigstore/sigstore-python/actions/runs/4601687825/jobs/8130389601?pr=594 |
A couple notes on things that have changed:
|
Also I do have a backup of the original TUF repo if need be |
/gcbrun |
Didn't run the test suite on these changes when they were pending in sigstore/root-signing 🙃 The tests are failing because our offline tests depend on a local copy of the relevant TUF targets. Unfortunately, we can't just pull them down and shove them in because of the consistent snapshot hash format. I'm working on adapting the tests now and I'll update when I make some progress |
Have been running into issues signing w/ staging:
Seems like the issue is happening with TUF refresh:
The offline tests w/ mock fetcher seem to |
Never mind, I figured it out: I had to delete the data dir in |
@haydentherapper I've updated the requisite files in https://github.com/trail-of-forks/sigstore-python/tree/ap/wip-staging-root-update, if you'd like to pull from that (I can't update this PR directly because I only have triage perms) |
@tnytown Given that this is currently failing the integration tests, I'd say you can go ahead and just create a separate PR rather than trying to coordinate merges between forks here. |
Rolled into #602. Thanks @haydentherapper! |
Change hardcoded root to the new root from sigstore/root-signing#756
Summary
Release Note
Documentation