New issue
Advanced search Search tips

Issue 923455 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Auto-submit did not set CQ+2 via the reply dialog

Project Member Reported by mbarowsky@google.com, Jan 18 (4 days ago)

Issue description

What steps will reproduce the problem?
(1) Upload a change for review and turn on the "Auto-submit" tag from within the Gerrit UI: crrev.com/c/1415890
(2) Receive +1 from an owner

What is the expected result?
CQ process begins automatically

What happens instead?
Gerrit still shows the option to "Submit to CQ" and doesn't appear to have added Commit Bot to the CL.




 

Comment 1 by rmis...@google.com, Jan 18 (4 days ago)

I cannot reproduce. When I CR+1 in https://chromium-review.googlesource.com/c/chromium/src/+/1415890, CQ+2 is selected in the UI.
Perhaps the reviewer specifically unchecked the CQ+2?

Comment 2 by mbarow...@chromium.org, Jan 18 (4 days ago)

The reviewer said they did not specifically uncheck the CQ+2. 

Comment 3 by rmis...@google.com, Jan 18 (4 days ago)

Cc: crouleau@chromium.org
+reviewer

crouleau@ could you describe how you gave approval to https://chromium-review.googlesource.com/c/chromium/src/+/1415890 ?

I know the Auto-Submit feature has been working: https://chromium-review.googlesource.com/q/label:Auto-Submit+repo:chromium/src+is:merged
so maybe this was a timing issue.

Comment 4 by crouleau@chromium.org, Jan 18 (4 days ago)

I just hit LGTM and didn't do anything else.

Comment 5 by rmis...@google.com, Jan 18 (4 days ago)

There are 2 ways to give approval: The quick approve "Code-Review+1" button on the main page and via the "Reply" dialog popup.
You hit the "Code-Review+1" button?

Comment 6 by crouleau@chromium.org, Jan 18 (4 days ago)

I think via the Reply dialog, but I'm not certain.

Comment 7 by ajp@chromium.org, Jan 18 (4 days ago)

Owner: rmis...@google.com
Status: Assigned (was: Untriaged)

Comment 8 by rmis...@google.com, Jan 19 (4 days ago)

Summary: Auto-submit did not set CQ+2 via the reply dialog (was: Auto-submit not working via UI)
I would guess the reply dialog, that might be the only place there could be Gerrit lag while automatically setting the necessary votes. Please let us know if you see this happen again.

Let's use this bug to dup against if there are similar reports.

Sign in to add a comment