New issue
Advanced search Search tips

Issue 601751 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

CQ error message due to open dependencies should be improved

Project Member Reported by rmis...@chromium.org, Apr 8 2016

Issue description

If there is an open dependency on a CL the CQ fails with the following error message:
This CL has an open dependency (Issue 9901001 Patch 1). Please resolve the dependency and try again.

As discussed in https://groups.google.com/a/chromium.org/d/msg/chromium-dev/yS3sV0Wgseg/KZWXOI5WLgAJ there are many usecases where developers would want to skip these checks. The ways to skip the checks are documented at 
https://docs.google.com/document/d/1KZGFKZpOPvco81sYVRCzwlnjGctup71RAzY0MSb0ntc/edit#heading=h.i94lf8av9mh3 but this is not easy to find.

We should extend the CQ error message to list those options to make them easier to find.

 
Summary: CQ error message due to open dependencies should be improved (was: CQ error message due to open depenencies should be improved)
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 14 2016

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/infra_internal.git/+/af776d8497b891bf67ff96d9cd2e5c86e781d238

commit af776d8497b891bf67ff96d9cd2e5c86e781d238
Author: rmistry <rmistry@google.com>
Date: Thu Apr 14 11:20:59 2016

Comment 3 by rmis...@google.com, Apr 14 2016

Status: Fixed (was: Assigned)
The change is now live. Tested it with: https://codereview.chromium.org/1888773002/#msg5

Marking as fixed.

Comment 4 by aga...@chromium.org, Apr 26 2016

Components: Infra>CQ
Labels: -Infra-CommitQueue

Sign in to add a comment