New issue
Advanced search Search tips

Issue 773552 link

Starred by 2 users

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

CL Had CQ +1 cleared incorrectly.

Project Member Reported by dgarr...@chromium.org, Oct 11 2017

Issue description

This CQ run failed with an Infra error and should not have rejected any CLs.

https://luci-milo.appspot.com/buildbot/chromeos/master-paladin/16544


It contained these two CLs that depended on each other:

https://crrev.com/c/709776
https://crrev.com/c/709738

At the end of the run, the annotataions indicate that both CLs were rejected because the other didn't have CQ +1.
 

Comment 1 by nxia@chromium.org, Oct 11 2017

Owner: nxia@chromium.org

Comment 2 by nxia@chromium.org, Oct 11 2017


17:39:05: INFO: Change CL:709738 is verified with reasons ['strategy:cq-submit-partial-pool-ignored-stages', 'strategy:cq-submit-partial-pool-builds-passed'], choose the final reason strategy:cq-submit-partial-pool-ignored-stages.


17:39:05: INFO: Failed to verify change CL:709776: relevant build reef-paladin failed with not ignorable failures in current run and didn't pass in history



http://shortn/_Q2gxLq89tJ

CQ verified 709738 but not 709776, when it tried to submit 709738 it found 709776 wasn't verified, so reject them. This needs fixing.



Comment 4 by nxia@chromium.org, Oct 19 2017

Cc: akes...@chromium.org nxia@chromium.org davidjames@chromium.org
 Issue 776362  has been merged into this issue.
Cc: -davidjames@chromium.org
Components: -Infra>Client>ChromeOS Infra>Client>ChromeOS>CI
Owner: ----

Comment 7 by nxia@chromium.org, Jun 8 2018

Cc: -nxia@chromium.org

Sign in to add a comment