[haiku-commits] Re: haiku: hrev53344 - /

  • From: "Alexander von Gluck IV" <kallisti5@xxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 06 Aug 2019 13:52:53 +0000

August 5, 2019 6:16 PM, "Alexander von Gluck IV" <kallisti5@xxxxxxxxxxx> wrote:

August 5, 2019 4:54 PM, "waddlesplash" <waddlesplash@xxxxxxxxx> wrote:

Appears kallisti5 completely broke the notifier. Don't worry, master
is still fine; though there's now a hrev tag pointing to these
commits... I'll try to delete it.

-waddlesplash

On Mon, Aug 5, 2019 at 5:51 PM waddlesplash <waddlesplash@xxxxxxxxx> wrote:

hrev53344 adds 2 changesets to branch 'master'
old head: 7391b10ca5e1033ba2bf6ba6a35303b437b9077e
new head: 57d9a08982e5fc8be742cac87c50b4630709c499
overview: 
https://git.haiku-os.org/haiku/log/?qt=range&q=57d9a08982e5+^7391b10ca5e1

Thanks :-)

So, the "big perl notifier" has been erroring out for a while and not-tagging 
things.

Turns out, gerrit submits a ref of "refs/changes/76/1223/meta" to the hooks 
while
the non-code-review one was a simple "refs/heads/(branch)"

All of the logic finding the branch was based around extracting the branch 
name
from the ref.. this is no longer a valid method to get it (causing the issues 
with
tags and notifications we have been seeing from gerrit code review accepted
commits)

I "fixed" the branch parsing, now we're "working" on the ref.

... except as you point out, the "ref" is what we're tagging now and logging.

Investigating.

Ok, so after much debugging and hacking of solutions... it turns one we have 
tumbled
over *additional* undocumented behavior of Gerrit.

The ref-updated hooks can get called "multiple times, simultaneously"
  https://bugs.chromium.org/p/gerrit/issues/detail?id=11281

Pretty much these scripts were written expecting "a ref-updated to a branch on 
commit"
with locking, and not "multiple ref-updated's run across multiple refs at the 
same time"

I've added some short-circuits around the undesirable ref's, and improved the 
detection
of "refs we don't yet know about yet"

The next person to merge a Gerrit change review will be the crash test dummy.

  -- Alex

Other related posts: