New issue
Advanced search Search tips

Issue 674256 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 666841


Previous locations:
gerrit:5139


Sign in to add a comment

CQ Dry Run button is shown for Chrome OS changes.

Project Member Reported by davidri...@chromium.org, Dec 14 2016

Issue description

Affected Version:

What steps will reproduce the problem?
1. View change https://chromium-review.googlesource.com/c/399478/

What is the expected output?

The CQ Dry Run button doesn't seem to make sense for Chrome OS and shouldn't exist.

What do you see instead?

It exists.


Please provide any additional information below.

 
Labels: -Restrict-View-Google
Project: chromium
Moved issue gerrit:5139 to now be  issue chromium:674256 .
Components: Infra>Codereview>Gerrit
Labels: Milestone-Dogfood Proj-Gerrit-Migration Pri-2 Type-Bug
Owner: aga...@chromium.org
Status: Assigned (was: New)
Also, the "Submit to CQ Button" should not exist.

Comment 5 by aga...@chromium.org, Jan 11 2017

Blocking: 666841

Comment 6 by aga...@chromium.org, Jan 12 2017

Let me make sure my understanding is correct, and please help me fill in the gaps:

The ChromeOS CQ should have a single button:
  * Applies CQ+1 label
  * Is the primary action (button is blue)
  * Says "Send to pre-CQ" (something better?) when available
  * Says "In pre-CQ" (something better?) when already set
And there should be no button which applies CQ+2.

Is that correct?

Comment 7 by aga...@chromium.org, Jan 17 2017

Status: Started (was: Assigned)
Out for review: cl/144758651

Comment 8 by aga...@chromium.org, Jan 18 2017

The above CL has been landed. It will go live with the next deployment.

Leaving this bug open since it is the sole blocker for making PG the default for our gerrit hosts, so keeping it open is a more accurate indication of that bug's state.
Status: Fixed (was: Started)
This is now fixed: https://screenshot.googleplex.com/BGAjrZqAeLu

Sign in to add a comment