-
Notifications
You must be signed in to change notification settings - Fork 50
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
[CI] Integration failure: staging instance #889
Comments
Context: this failed because we failed to parse the checkpoint record in Rekor's response. Triaging: looks like Rekor 1.3.5 removed the timestamp field from the checkpoint: sigstore/rekor#1888 This should be trivial for us to fix, per convo in the PR above. |
Opened #891 for this. |
Yes, even the "spec" on transparency-dev says:
So we're not compliant when we require 4 lines. |
Yep, opened #891 with the fix. |
Whoops, sorry that broke tests! Yea, the timestamp is part of extra data (which should be parsed, just not required), and it served no purpose in the ecosystem. |
No problem at all! This was definitely us not reading the spec closely enough 🙂 |
We should do a release in near future so we're ahead of this changing in production rekor. |
Yeah, agreed. We're mid-burn for 3.0, so we may have to cut another 2.x release and backport this. |
I've created https://github.com/sigstore/sigstore-python/tree/series/2.1.x to track a point release to 2.1.x. |
I've cut 2.1.2 as a backport release. We can continue to use |
Staging instance failure
A scheduled test against Sigstore's staging instance has failed.
This suggests one of three conditions:
sigstore-python
;The full CI failure can be found here:
https://github.com/sigstore/sigstore-python/actions/runs/7772634242
The text was updated successfully, but these errors were encountered: