<?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>136793</bug_id>
          
          <creation_ts>2014-09-12 16:13:46 -0700</creation_ts>
          <short_desc>Multiple EWS or commit bots can pick up processing of the same patch</short_desc>
          <delta_ts>2014-09-12 16:33:25 -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>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Alexey Proskuryakov">ap</reporter>
          <assigned_to name="Alexey Proskuryakov">ap</assigned_to>
          <cc>rniwa</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1034993</commentid>
    <comment_count>0</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2014-09-12 16:13:46 -0700</bug_when>
    <thetext>Only one bot from each queue is supposed to be processing a given patch; havoc ensues otherwise.

But this happens pretty frequently. Turns out that the server doesn&apos;t let a bot chew on a patch for more than an hour, and releases it for other bots to take.

In worst case, a bot needs to clean, update, apply the patch, build, test, re-test, rollout, build, test, commit. That takes more than an hour.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1034994</commentid>
    <comment_count>1</comment_count>
      <attachid>238072</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2014-09-12 16:16:50 -0700</bug_when>
    <thetext>Created attachment 238072
proposed fix</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1035000</commentid>
    <comment_count>2</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2014-09-12 16:33:25 -0700</bug_when>
    <thetext>Committed &lt;http://trac.webkit.org/r173587&gt;.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>238072</attachid>
            <date>2014-09-12 16:16:50 -0700</date>
            <delta_ts>2014-09-12 16:27:07 -0700</delta_ts>
            <desc>proposed fix</desc>
            <filename>queueTimeout.txt</filename>
            <type>text/plain</type>
            <size>2347</size>
            <attacher name="Alexey Proskuryakov">ap</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDE3MzU4NCkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDE3IEBACisyMDE0LTA5LTEyICBBbGV4ZXkgUHJvc2t1cnlha292ICA8YXBAYXBwbGUuY29t
PgorCisgICAgICAgIE11bHRpcGxlIEVXUyBvciBjb21taXQgYm90cyBjYW4gcGljayB1cCBwcm9j
ZXNzaW5nIG9mIHRoZSBzYW1lIHBhdGNoCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD0xMzY3OTMKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P
UFMhKS4KKworICAgICAgICBDaGFuZ2UgdGltZW91dCBmcm9tIG9uZSBob3VyIHRvIHR3byBob3Vy
cy4gVGhpcyB3aWxsIGxldCBib3RzIGZpbmlzaCwgYW5kIGluIHRoZQorICAgICAgICB1bmxpa2Vs
eSBldmVudCBvZiBhIGJvdCBiZWluZyBmcm96ZW4sIHdpbGwgb25seSBkZWxheSBwYXRjaCBwcm9j
ZXNzaW5nIHJlc3RhcnQgYnkKKyAgICAgICAgb25lIGhvdXIuCisKKyAgICAgICAgKiBRdWV1ZVN0
YXR1c1NlcnZlci9hcHAueWFtbDoKKyAgICAgICAgKiBRdWV1ZVN0YXR1c1NlcnZlci9tb2RlbC9h
Y3RpdmV3b3JraXRlbXMucHk6IChBY3RpdmVXb3JrSXRlbXMuZGVhY3RpdmF0ZV9leHBpcmVkKToK
KwogMjAxNC0wOS0xMSAgQW5keSBFc3RlcyAgPGFlc3Rlc0BhcHBsZS5jb20+CiAKICAgICAgICAg
W1dlYktpdDJdIFdLSW1hZ2VDcmVhdGVDR0ltYWdlIGNyYXNoZXMgd2hlbiBwYXNzZWQgYSBudWxs
IFdLSW1hZ2UKSW5kZXg6IFRvb2xzL1F1ZXVlU3RhdHVzU2VydmVyL2FwcC55YW1sCj09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT0KLS0tIFRvb2xzL1F1ZXVlU3RhdHVzU2VydmVyL2FwcC55YW1sCShyZXZpc2lvbiAxNzM1ODQp
CisrKyBUb29scy9RdWV1ZVN0YXR1c1NlcnZlci9hcHAueWFtbAkod29ya2luZyBjb3B5KQpAQCAt
MSw1ICsxLDUgQEAKIGFwcGxpY2F0aW9uOiB3ZWJraXQtcXVldWVzCi12ZXJzaW9uOiAxNjIzNTgg
IyBCdWd6aWxsYSBidWcgSUQgb2YgbGFzdCBtYWpvciBjaGFuZ2UKK3ZlcnNpb246IDE3MzU4NSAj
IEJ1Z3ppbGxhIGJ1ZyBJRCBvZiBsYXN0IG1ham9yIGNoYW5nZQogcnVudGltZTogcHl0aG9uCiBh
cGlfdmVyc2lvbjogMQogCkluZGV4OiBUb29scy9RdWV1ZVN0YXR1c1NlcnZlci9tb2RlbC9hY3Rp
dmV3b3JraXRlbXMucHkKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gVG9vbHMvUXVldWVTdGF0dXNTZXJ2ZXIvbW9k
ZWwvYWN0aXZld29ya2l0ZW1zLnB5CShyZXZpc2lvbiAxNzM1ODQpCisrKyBUb29scy9RdWV1ZVN0
YXR1c1NlcnZlci9tb2RlbC9hY3RpdmV3b3JraXRlbXMucHkJKHdvcmtpbmcgY29weSkKQEAgLTc5
LDggKzc5LDEwIEBAIGNsYXNzIEFjdGl2ZVdvcmtJdGVtcyhkYi5Nb2RlbCwgUXVldWVQcm8KICAg
ICAgICAgcmV0dXJuIGRiLnJ1bl9pbl90cmFuc2FjdGlvbihzZWxmLl9leHBpcmVfaXRlbSwgc2Vs
Zi5rZXkoKSwgaXRlbV9pZCkKIAogICAgIGRlZiBkZWFjdGl2YXRlX2V4cGlyZWQoc2VsZiwgbm93
KToKLSAgICAgICAgb25lX2hvdXJfYWdvID0gdGltZS5ta3RpbWUoKG5vdyAtIHRpbWVkZWx0YSht
aW51dGVzPTYwKSkudGltZXR1cGxlKCkpCi0gICAgICAgIG5vbmV4cGlyZWRfcGFpcnMgPSBbcGFp
ciBmb3IgcGFpciBpbiBzZWxmLl9pdGVtX3RpbWVfcGFpcnMoKSBpZiBwYWlyWzFdID4gb25lX2hv
dXJfYWdvXQorICAgICAgICAjIElmIHRoZSBwYXRjaCBpcyBzdGlsbCBhY3RpdmUgYWZ0ZXIgdGhp
cyBtdWNoIHRpbWUsIHRoZW4gYSBib3QgbXVzdCBoYXZlIGZyb3plbiBvciByZWJvb3RlZCwKKyAg
ICAgICAgIyBhbmQgZHJvcHBlZCB0aGUgcGF0Y2ggb24gdGhlIGZsb29yLiBMZXQgYW5vdGhlciBi
b3QgcGljayBpdCB1cC4KKyAgICAgICAgdHdvX2hvdXJzX2FnbyA9IHRpbWUubWt0aW1lKChub3cg
LSB0aW1lZGVsdGEobWludXRlcz0xMjApKS50aW1ldHVwbGUoKSkKKyAgICAgICAgbm9uZXhwaXJl
ZF9wYWlycyA9IFtwYWlyIGZvciBwYWlyIGluIHNlbGYuX2l0ZW1fdGltZV9wYWlycygpIGlmIHBh
aXJbMV0gPiB0d29faG91cnNfYWdvXQogICAgICAgICBzZWxmLl9zZXRfaXRlbV90aW1lX3BhaXJz
KG5vbmV4cGlyZWRfcGFpcnMpCiAKICAgICBkZWYgbmV4dF9pdGVtKHNlbGYsIHdvcmtfaXRlbV9p
ZHMsIG5vdyk6Cg==
</data>
<flag name="review"
          id="262833"
          type_id="1"
          status="+"
          setter="rniwa"
    />
          </attachment>
      

    </bug>

</bugzilla>