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

Failure in TUF-on-CI online signing #1349

Closed
sigstore-bot opened this issue Sep 2, 2024 · 2 comments · Fixed by sigstore/community#484
Closed

Failure in TUF-on-CI online signing #1349

sigstore-bot opened this issue Sep 2, 2024 · 2 comments · Fixed by sigstore/community#484

Comments

@sigstore-bot
Copy link
Member

Workflow run failed for TUF-on-CI online signing.

Failed run: https://github.com/sigstore/root-signing/actions/runs/10666187470

CC @sigstore/tuf-root-signing-codeowners, please have a look.

@jku
Copy link
Member

jku commented Sep 2, 2024

This failure happened after (the second) attempt at merging initial signing event: looks like the signing succceeds but push to main does not.

Online sign (snapshot & timestamp)
[main 1fb8aaa] Online sign (snapshot & timestamp)
 2 files changed, 11 insertions(+), 26 deletions(-)
remote: error: GH006: Protected branch update failed for refs/heads/main.        
remote: error: 2 of 2 required status checks are expected.        
To https://github.com/sigstore/root-signing
 ! [remote rejected] HEAD -> main (protected branch hook declined)

The reason seems to be that there are required status checks (DCO and yamllint). This was fine in the previous system where the commit was included in the PR (which was then automatically merged) but does not work in new system where the commit is just automatically merged to main -- status checks are not available in this case

@sigstore-bot
Copy link
Member Author

Workflow run failed for TUF-on-CI online signing.

Failed run: https://github.com/sigstore/root-signing/actions/runs/10668039366

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants