Bug 31645 – CommitQueue should fetch list of patches on every cycle (and not use PatchCollection)

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Eric Seidel (no email)]

Created attachment 43457 [details]
example of broken logging ui

workqueue.py results in totally broken commit-queue UI

Adam warned me about some functional changes as a result of the new workqueue.py.  But it looks like one of them needs to be immediately fixed.  commit-queue needs to reload the list of bugs after every failure.  I'll attach an example of hte broken UI.

[Eric Seidel (no email)]

Created attachment 43460 [details]
example of commit re-tries not working

Sad!  Looks like commit re-tries are broken as a result of this too. :(

[Eric Seidel (no email)]

I think the problem is more PatchCollection than the WorkQueue itself.  Well, maybe both are at fault.  We should be refreshing the patch collection after every iteration (at least for the commit-queue).  I'm actually not sure commit-queue really wants to use a PatchCollection at all.

[Eric Seidel (no email)]

Created attachment 43463 [details]
Patch

[Eric Seidel (no email)]

This bug was a bit overly dramatic when I opened it. :)  The sky is not falling.  But things are definitely better after restoring the old behavior.

[Darin Adler]

Comment on attachment 43463 [details]
Patch

rs=me

[WebKit Commit Bot]

Comment on attachment 43463 [details]
Patch

Clearing flags on attachment: 43463

Committed r51160: <http://trac.webkit.org/changeset/51160>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.