OMG: Gerrit is very buggy since the recent major updates

#211
Opened by tazjin at 2022-10-13T18·49+00

We've recently upgraded Gerrit through a few major versions, which has some important user-facing changes:

Related CLs to this are e.g. cl/6505, cl/6660, cl/6665, cl/6685

As a result of this, we now have a variety of bugs in Gerrit that didn't exist before, for example:

This list is inexhaustive, it's just noticeable in everyday flows that Gerrit is now kind of wonky.

I'm not sure what to do about this at the moment, filing issue for tracking and discussion.

  1. Another one:

    • Gerrit will randomly mark commits in a chain as "Ready to submit" when they are not. Clicking the Submit button then causes an error.

    tazjin at 2022-10-16T10·14+00

  2. Score copying randomly doesn't work, leading to situations where people e.g. change a commit message and lose Verified+1 (with no way to get it back, since there is no build to re-run for that patchset!) (e.g. cl/6948)

    Happened with https://cl.tvl.fyi/c/depot/+/7053/2 as well.

    flokli at 2022-10-21T16·25+00

  3. I think some of the problems have been resolved:

    • Change indexing should work again, mostly, after cl/7172
    • Gerrit bump to the 3.7.0 RC has pulled in https://gerrit-review.googlesource.com/337968 which should fix vote copying - basically the problem was that the vote copying would happen, THEN CI would vote on a previous version, which wouldn't get copied forward.

    lukegb at 2022-11-06T16·57+00