New issue
Advanced search Search tips

Issue 780359 link

Starred by 1 user
Status: WontFix
Owner: ----
Closed: Nov 2017
Cc:
rmis...@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Gerrit drops LGTM for some code review

Project Member Reported by zhongyi@chromium.org, Nov 1 2017 
When reviewing a CL, CQ drops the approval I grant at 7:45pm.

first time approval: https://screenshot.googleplex.com/fFCccOcZLC8

then got lost, have to reapprove: https://screenshot.googleplex.com/sFmJaX9VroM

Comment 1 by tandrii@chromium.org, Nov 1 2017

Cc: -zhongyi@chromium.org
Owner: zhongyi@chromium.org
Status: Assigned (was: Untriaged)
which CL?

Why do you think it's CQ, and not your Gerrit config that doesn't keep CR vote sticky between patchsets?

Comment 2 by zhongyi@chromium.org, Nov 1 2017

Owner: ----
Status: Available (was: Assigned)
heh...I meant to say Gerrit not the CQ

https://chromium-review.googlesource.com/c/chromium/src/+/748425

we didn't encounter this issue before, not sure which team this issue should assign to, feel free to route to the correct one or close if you think it's not a issue.

Comment 3 by tandrii@chromium.org, Nov 1 2017

Components: -Infra>Platform>CQ Infra>Codereview>Gerrit
Status: untrig (was: Available)
if Gerrit, then please don't file it under Infra>Platform>CQ next time :(

Comment 4 by tandrii@chromium.org, Nov 1 2017

Status: Untriaged (was: Untrig)

Comment 5 by zhongyi@chromium.org, Nov 1 2017

Status: WontFix (was: Untriaged)
I messed up two CLs, closing this one. Sorry about the noise.
►

Sign in to add a comment