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

[e2e]: generic schedule main multi-subjects slsa3 #876

Closed
ianlewis opened this issue Sep 20, 2022 · 25 comments
Closed

[e2e]: generic schedule main multi-subjects slsa3 #876

ianlewis opened this issue Sep 20, 2022 · 25 comments
Assignees
Labels
area:generic Issue with the generic generator e2e e2e integration tests type:bug Something isn't working

Comments

@ianlewis ianlewis added e2e e2e integration tests type:bug Something isn't working area:generic Issue with the generic generator labels Sep 20, 2022
@ianlewis
Copy link
Member Author

Getting a rekor error:

Getting rekor entry error error verifying tlog entry: calculated root:
[24 12 212 170 213 2 56 52 155 205 63 112 44 220 2 1 231 110 52 155 94 169 129 26 157 212 5 37 124 99 64 196]
 does not match expected root:
[161 54 80 70 6 156 217 165 66 112 218 229 59 131 190 83 40 141 66 118 98 5 110 175 23 241 106 177 93 217 127 40]: verifying inclusion proof, trying Redis search index to find entries by subject digest
FAILED: SLSA verification failed: could not find a matching valid signature entry: got unexpected errors 

@ianlewis ianlewis self-assigned this Sep 21, 2022
@asraa
Copy link
Collaborator

asraa commented Sep 21, 2022

Rekor is stuck at v0.10.0, which never included my race condition fix, FYI!

They are working on it
sigstore/rekor#1055 (comment)

@haydentherapper
Copy link
Contributor

Rekor 0.11.0 has now been rolled out.

@ianlewis
Copy link
Member Author

Thanks @haydentherapper! Hopefully the next run of the test will be successful and this issue will close automatically.

@haydentherapper
Copy link
Contributor

@asraa Is this the client breaking change?

@asraa
Copy link
Collaborator

asraa commented Sep 22, 2022

@asraa Is this the client breaking change?

What do you mean? I don't think it's related to cosign sign-blob fixes

The current failure is a little hard to read...

@stephenfuqua
Copy link

I'm using slsa-framework/slsa-github-generator/.github/workflows/generator_generic_slsa3.yml@v1.2.0 and encountering message FAILED: SLSA verification failed: could not find a matching valid signature entry.

Based on the above information, it is hard for me to tell if this is expected right now or if perhaps I have done something wrong in my implementation.

Will keep an eye on this issue to see if/when there is a resolution.

@asraa
Copy link
Collaborator

asraa commented Sep 28, 2022

I'm using slsa-framework/slsa-github-generator/.github/workflows/generator_generic_slsa3.yml@v1.2.0 and encountering message FAILED: SLSA verification failed: could not find a matching valid signature entry.

Hi! I don't think this is the same issue given that the error here is from multi subjects. There was a recent regression just today in Rekor regarding blob verification, see similar issue here (sigstore/cosign#2294).

I'll check to see what is the cause and update you here.

@asraa
Copy link
Collaborator

asraa commented Sep 28, 2022

Correct! Your issue will be fixed when this is rolled into production: sigstore/rekor#1082

For reference: we aren't retrieving the Rekor log entry for the builder's attestation so the workflow is failing because it cannot verify the builder. This is because when we sharded, Rekor only retrieves log entries on the CURRENT shard, which is now rotated.

You can use the compile-builder: true option to mitigate in the meantime.

@ianlewis
Copy link
Member Author

ianlewis commented Oct 5, 2022

I think this was caused by a buggy tag check in the tests. The if failed and continues down the wrong code branch.

./.github/workflows/scripts/e2e-verify.common.sh: line 115: v1.1.2: command not found

It should be fixed by slsa-framework/example-package@87a5fb1

@ianlewis ianlewis closed this as completed Oct 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:generic Issue with the generic generator e2e e2e integration tests type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants