-
Notifications
You must be signed in to change notification settings - Fork 74
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
DCO bot is stuck in "waiting for status reported" #69
Comments
Looks like #67 might have introduced a bug? cc @hiimbex
|
Hi all, Can someone please help me checking if something is causing this delay again? Thank you so much! |
This one was stuck and we ended up closing it and opening it as a new item to get past DCO being stuck. hyperledger-archives/sawtooth-core#1572 |
another PR stuck on DCO: hyperledger-archives/sawtooth-core#1554 |
Hi, We are still seeing this issue. edgexfoundry/blackbox-testing#24 |
Sorry for the delay and thanks for your patience with this issue. I think we resolved some of the causes of the "hanging" in #72, but for older PRs the DCO might need to be re-triggered to run on the PR. Opening new PRs to re-trigger the bot is definitely the first step. And I think for the PRs opened more than a week or two ago, future PRs should be fine. The only PR here that doesn't fit that criterion is @JPWKU issue, which I think may have been due to the first commit having |
Hi @hiimbex , That indeed was the issue. weichou1229 closed the old PR and resigned the commits with the same git user and resubmitting. It checked out after that. Thanks for looking into it. |
@hiimbex I agree that it seemed a weird case for it to hang though. |
We are still seeing the DCO bot stuck, even on a newly uploaded PR. hyperledger-labs/private-data-objects#11 Since none of us have the ability to override the DCO bot check, we are completely stuck and unable to merge PRs. Help would be much appreciated. |
This is definitely the exact same issue @JPWKU just described, so I'd recommend that Mic Bowman (the original committer) associate his github account with those commits by editing his GitHub global configs locally and recommitting. If this is a new contributor though, doing all of that git meddling might be confusing in which case I'd recommend just copying the changes and making a new commit once their global git config is set up. Also you can remove DCO as a 'required' check in the settings page. But that does need to be done by someone with admin permissions. I am looking into this issue and hope to have a solution soon, since this is obviously not our intended behavior. |
See: #77 for the solution to this issue. |
This still happens. See this PR for an example. Please advise ASAP as I cannot ask this user to leave this PR in this state for long. He needs to move on and get the PR landed. |
🎉 This issue has been resolved in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
envoyproxy/envoy#3021
Any idea what's up?
The text was updated successfully, but these errors were encountered: