New issue
Advanced search Search tips

Issue 740155 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----

Blocking:
issue 685318


Previous locations:
gerrit:6607


Sign in to add a comment

CQ: Provide a tag when unvoting on Gerrit label

Project Member Reported by jgruber@chromium.org, Jun 29 2017

Issue description

Even after clicking 'Show comments only', the messages section is spammed by all kinds of irrelevant auto-generated comments such as 'Commit-Queue+1'.

See attached image for an example where the single real comment is spammed with 3 auto-generated comments.

And as an aside, 'Show comments only'/'Expand all' should really be the default.
 
gerrit-show-comments-only.png
33.2 KB View Download
Blocking: chromium:685318

Comment 2 by logan@google.com, Jul 7 2017

Labels: -Priority-3 Milestone-Chromium-Afterglow Priority-2
Owner: logan@google.com
Status: Accepted (was: New)
Summary: Add tag to DeleteVoteInput (was: 'Show comments only' is still too noisy)
The API endpoint that Commit-Bot uses to clean up other users' Commit-Queue votes when it processes them doesn't support the tag feature that Gerrit depends on to identify non-comment messages.

Comment 3 by logan@google.com, Jul 7 2017

Labels: -Type-Bug -Priority-2 -Milestone-Chromium-Afterglow Proj-Gerrit-Migration
Owner: ----
Status: New (was: Accepted)
Summary: CQ: Provide a tag when unvoting on Gerrit label (was: Add tag to DeleteVoteInput)
I dug into the code for Commit-Bot. I see that it doesn't actually delete votes this way, it uses a different endpoint that *does* support tag. I don't see Commit-Bot providing a tag, however:

https://cs.corp.google.com/chromium_infra_internal/infra_internal/services/cq/pending_manager/gerrit.py?l=170-173

Comment 4 by logan@google.com, Jul 7 2017

Project: chromium
Moved issue gerrit:6607 to now be  issue chromium:740155 .

Comment 5 by logan@google.com, Jul 10 2017

Summary: CQ: Provide a tag when unvoting on Gerrit label (was: Add tag to DeleteVoteInput)
My summary change seems to have been reverted by the issue move, see issue monorail:2711.

Comment 6 by logan@google.com, Jul 10 2017

Owner: ----

Comment 7 by logan@google.com, Jul 10 2017

Status: Untriaged (was: Accepted)

Comment 8 by aga...@chromium.org, Jul 10 2017

Cc: tandrii@chromium.org
Labels: Milestone-Afterglow
Owner: aga...@chromium.org
Status: Started (was: Untriaged)
https://chrome-internal-review.googlesource.com/409069
Project Member

Comment 9 by bugdroid1@chromium.org, Jul 10 2017

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

commit 910ea7e96ca0ea188f33aaf92ff99d70bd8a1df3
Author: Aaron Gable <agable@chromium.org>
Date: Mon Jul 10 20:07:29 2017

Status: Fixed (was: Started)

Sign in to add a comment