Bug 239731 - `merge-block` label added by one EWS bot prevents other EWS bots from processing the patch
Summary: `merge-block` label added by one EWS bot prevents other EWS bots from process...
Status: RESOLVED DUPLICATE of bug 240169
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks: 239082
  Show dependency tree / graph
 
Reported: 2022-04-25 10:02 PDT by Chris Dumez
Modified: 2022-05-06 08:30 PDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2022-04-25 10:02:51 PDT 
`merge-block` label added by one EWS bot prevents other EWS bots from processing the patch.

I upload a PR, it fails on one of the EWS bots (due to flakiness or just a fluke). The `merge-block` label will get added to my PR and other EWS bots will just not process my PR any longer. This is super annoying.
I need feedback from other bots.

One example where it is particularly bad is when the style bot is failing, why should it prevent testing? There are plenty of cases where we choose the ignore the style bot because it is wrong.
Comment 1 Chris Dumez 2022-04-25 10:03:52 PDT 
Also, I tried removing the `merge-blocked` keyword manually and it didn't seem to unblock the other EWS (I had to reupload my PR).
Comment 2 Jonathan Bedard 2022-04-25 11:42:08 PDT 
Patch based EWS does this too with cq-!

This problem is much worse with PRs because of https://bugs.webkit.org/show_bug.cgi?id=239411, which I do intend to resolve shortly.

*** This bug has been marked as a duplicate of bug 239411 ***
Comment 3 Yusuke Suzuki 2022-04-27 13:51:18 PDT 
Nice!
Comment 4 Jonathan Bedard 2022-05-06 08:29:50 PDT 

*** This bug has been marked as a duplicate of bug 240169 ***
Comment 5 Jonathan Bedard 2022-05-06 08:30:54 PDT 
My comment about "patch based EWS doing this too with cq-" is incorrect. Patch based EWS does this with r-, the important difference being that bots don't ever automatically apply r- to patches like they do merging-blocked to PRs.