<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>239731</bug_id>
          
          <creation_ts>2022-04-25 10:02:51 -0700</creation_ts>
          <short_desc>`merge-block` label added by one EWS bot prevents other EWS bots from processing the patch</short_desc>
          <delta_ts>2022-05-06 08:30:54 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Tools / Tests</component>
          <version>WebKit Nightly Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>DUPLICATE</resolution>
          <dup_id>240169</dup_id>
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>239082</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Chris Dumez">cdumez</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>jbedard</cc>
    
    <cc>ysuzuki</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1864137</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2022-04-25 10:02:51 -0700</bug_when>
    <thetext>`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.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1864139</commentid>
    <comment_count>1</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2022-04-25 10:03:52 -0700</bug_when>
    <thetext>Also, I tried removing the `merge-blocked` keyword manually and it didn&apos;t seem to unblock the other EWS (I had to reupload my PR).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1864166</commentid>
    <comment_count>2</comment_count>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2022-04-25 11:42:08 -0700</bug_when>
    <thetext>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 ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1864922</commentid>
    <comment_count>3</comment_count>
    <who name="Yusuke Suzuki">ysuzuki</who>
    <bug_when>2022-04-27 13:51:18 -0700</bug_when>
    <thetext>Nice!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1867202</commentid>
    <comment_count>4</comment_count>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2022-05-06 08:29:50 -0700</bug_when>
    <thetext>

*** This bug has been marked as a duplicate of bug 240169 ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1867204</commentid>
    <comment_count>5</comment_count>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2022-05-06 08:30:54 -0700</bug_when>
    <thetext>My comment about &quot;patch based EWS doing this too with cq-&quot; is incorrect. Patch based EWS does this with r-, the important difference being that bots don&apos;t ever automatically apply r- to patches like they do merging-blocked to PRs.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>