<?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>170191</bug_id>
          
          <creation_ts>2017-03-28 12:52:12 -0700</creation_ts>
          <short_desc>Dashboard tests fails to fetch the revision from commits</short_desc>
          <delta_ts>2017-03-28 14:56:18 -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 Local Build</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugs.webkit.org/show_bug.cgi?id=168624</see_also>
    
    <see_also>https://bugs.webkit.org/show_bug.cgi?id=170158</see_also>
          <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="Aakash Jain">aakash_jain</reporter>
          <assigned_to name="Aakash Jain">aakash_jain</assigned_to>
          <cc>aakash_jain</cc>
    
    <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>ddkilzer</cc>
    
    <cc>jmarcell</cc>
    
    <cc>lforschler</cc>
    
    <cc>webkit</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1292013</commentid>
    <comment_count>0</comment_count>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2017-03-28 12:52:12 -0700</bug_when>
    <thetext>Many dashboard tests (e.g: nextRevision, indexOfRevision, commitsOnBranchLaterThanRevision, commitsOnBranchInRevisionRange) fails as they fails to fetch the revision from commits.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1292020</commentid>
    <comment_count>1</comment_count>
      <attachid>305625</attachid>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2017-03-28 12:58:05 -0700</bug_when>
    <thetext>Created attachment 305625
Proposed patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1292082</commentid>
    <comment_count>2</comment_count>
    <who name="Jason Marcell">jmarcell</who>
    <bug_when>2017-03-28 14:10:18 -0700</bug_when>
    <thetext>Confirmed that all tests are passing after applying 170191 &amp; 170193.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1292110</commentid>
    <comment_count>3</comment_count>
      <attachid>305625</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-03-28 14:56:14 -0700</bug_when>
    <thetext>Comment on attachment 305625
Proposed patch

Clearing flags on attachment: 305625

Committed r214496: &lt;http://trac.webkit.org/changeset/214496&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1292111</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2017-03-28 14:56:18 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>305625</attachid>
            <date>2017-03-28 12:58:05 -0700</date>
            <delta_ts>2017-03-28 14:56:14 -0700</delta_ts>
            <desc>Proposed patch</desc>
            <filename>patch_dashboard_fetch_revision</filename>
            <type>text/plain</type>
            <size>5004</size>
            <attacher name="Aakash Jain">aakash_jain</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDIxNDQ4NykKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDE1IEBACisyMDE3LTAzLTI4ICBBYWthc2ggSmFpbiAgPGFha2FzaF9qYWluQGFwcGxlLmNv
bT4KKworICAgICAgICBEYXNoYm9hcmQgdGVzdHMgZmFpbHMgdG8gZmV0Y2ggdGhlIHJldmlzaW9u
IGZyb20gY29tbWl0cworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5j
Z2k/aWQ9MTcwMTkxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAg
ICAgICAgKiBCdWlsZFNsYXZlU3VwcG9ydC9idWlsZC53ZWJraXQub3JnLWNvbmZpZy9wdWJsaWNf
aHRtbC9kYXNoYm9hcmQvU2NyaXB0cy90ZXN0cy9yZXNvdXJjZXMvTW9ja1RyYWMuanM6CisgICAg
ICAgIEFkZGVkIHJlY29yZGVkQ29tbWl0SW5kaWNlc0J5UmV2aXNpb25OdW1iZXIuCisgICAgICAg
ICogQnVpbGRTbGF2ZVN1cHBvcnQvYnVpbGQud2Via2l0Lm9yZy1jb25maWcvcHVibGljX2h0bWwv
ZGFzaGJvYXJkL1NjcmlwdHMvdGVzdHMvcmVzb3VyY2VzL3Rlc3RzLmpzOgorICAgICAgICAoc2V0
dXApOiBQb3B1bGF0ZWQgdHJhYy5yZWNvcmRlZENvbW1pdEluZGljZXNCeVJldmlzaW9uTnVtYmVy
IHNpbmNlIHRoZSBkYXNoYm9hcmQgY29kZSB1c2VzIHRoaXMgdmFyaWFibGUuCisKIDIwMTctMDMt
MjggIEpvbmF0aGFuIEJlZGFyZCAgPGpiZWRhcmRAYXBwbGUuY29tPgogCiAgICAgICAgIHdlYmtp
dHB5OiBVc2UgaG9zdCBwYXR0ZXJuIGZvciBkZXZpY2VzCkluZGV4OiBUb29scy9CdWlsZFNsYXZl
U3VwcG9ydC9idWlsZC53ZWJraXQub3JnLWNvbmZpZy9wdWJsaWNfaHRtbC9kYXNoYm9hcmQvU2Ny
aXB0cy90ZXN0cy9yZXNvdXJjZXMvTW9ja1RyYWMuanMKPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gVG9vbHMvQnVp
bGRTbGF2ZVN1cHBvcnQvYnVpbGQud2Via2l0Lm9yZy1jb25maWcvcHVibGljX2h0bWwvZGFzaGJv
YXJkL1NjcmlwdHMvdGVzdHMvcmVzb3VyY2VzL01vY2tUcmFjLmpzCShyZXZpc2lvbiAyMTQyMzcp
CisrKyBUb29scy9CdWlsZFNsYXZlU3VwcG9ydC9idWlsZC53ZWJraXQub3JnLWNvbmZpZy9wdWJs
aWNfaHRtbC9kYXNoYm9hcmQvU2NyaXB0cy90ZXN0cy9yZXNvdXJjZXMvTW9ja1RyYWMuanMJKHdv
cmtpbmcgY29weSkKQEAgLTEwNiw0ICsxMDYsMTUgQEAgTW9ja1RyYWMuRVhBTVBMRV9UUkFDX0NP
TU1JVFMgPSBbCiAgICAgICAgICJjb250YWluc0JyYW5jaExvY2F0aW9uIjogdHJ1ZSwKICAgICAg
ICAgImJyYW5jaGVzIjogWyJ0cnVuayJdCiAgICAgfQotXTsKXCBObyBuZXdsaW5lIGF0IGVuZCBv
ZiBmaWxlCitdOworCitNb2NrVHJhYy5yZWNvcmRlZENvbW1pdEluZGljZXNCeVJldmlzaW9uTnVt
YmVyID0geworICAgIDMzMDE4OiAwLAorICAgIDMzMDE5OiAxLAorICAgIDMzMDIwOiAyLAorICAg
IDMzMDIxOiAzLAorICAgIDMzMDIyOiA0LAorICAgIDMzMDIzOiA1LAorICAgIDMzMDI0OiA2LAor
ICAgIDMzMDI1OiA3Cit9OwpJbmRleDogVG9vbHMvQnVpbGRTbGF2ZVN1cHBvcnQvYnVpbGQud2Vi
a2l0Lm9yZy1jb25maWcvcHVibGljX2h0bWwvZGFzaGJvYXJkL1NjcmlwdHMvdGVzdHMvcmVzb3Vy
Y2VzL3Rlc3RzLmpzCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFRvb2xzL0J1aWxkU2xhdmVTdXBwb3J0L2J1aWxk
LndlYmtpdC5vcmctY29uZmlnL3B1YmxpY19odG1sL2Rhc2hib2FyZC9TY3JpcHRzL3Rlc3RzL3Jl
c291cmNlcy90ZXN0cy5qcwkocmV2aXNpb24gMjE0NDg2KQorKysgVG9vbHMvQnVpbGRTbGF2ZVN1
cHBvcnQvYnVpbGQud2Via2l0Lm9yZy1jb25maWcvcHVibGljX2h0bWwvZGFzaGJvYXJkL1Njcmlw
dHMvdGVzdHMvcmVzb3VyY2VzL3Rlc3RzLmpzCSh3b3JraW5nIGNvcHkpCkBAIC05MSw2ICs5MSw3
IEBAIHRlc3QoIl9wYXJzZVJldmlzaW9uRnJvbVVSTCIsIGZ1bmN0aW9uKCkKIHRlc3QoIm5leHRS
ZXZpc2lvbiIsIGZ1bmN0aW9uKCkKIHsKICAgICB0aGlzLnRyYWMucmVjb3JkZWRDb21taXRzID0g
TW9ja1RyYWMuRVhBTVBMRV9UUkFDX0NPTU1JVFM7CisgICAgdGhpcy50cmFjLnJlY29yZGVkQ29t
bWl0SW5kaWNlc0J5UmV2aXNpb25OdW1iZXIgPSBNb2NrVHJhYy5yZWNvcmRlZENvbW1pdEluZGlj
ZXNCeVJldmlzaW9uTnVtYmVyOwogICAgIHN0cmljdEVxdWFsKHRoaXMudHJhYy5uZXh0UmV2aXNp
b24oInRydW5rIiwgIjMzMDIwIiksICIzMzAyMiIsICJuZXh0UmV2aXNpb24gc2FtZSBicmFuY2gi
KTsKICAgICBzdHJpY3RFcXVhbCh0aGlzLnRyYWMubmV4dFJldmlzaW9uKCJ0cnVuayIsICIzMzAx
OSIpLCAiMzMwMjAiLCAibmV4dFJldmlzaW9uIGRpZmZlcmVudCBicmFuY2giKTsKIH0pOwpAQCAt
OTgsMTIgKzk5LDE0IEBAIHRlc3QoIm5leHRSZXZpc2lvbiIsIGZ1bmN0aW9uKCkKIHRlc3QoImlu
ZGV4T2ZSZXZpc2lvbiIsIGZ1bmN0aW9uKCkKIHsKICAgICB0aGlzLnRyYWMucmVjb3JkZWRDb21t
aXRzID0gTW9ja1RyYWMuRVhBTVBMRV9UUkFDX0NPTU1JVFM7CisgICAgdGhpcy50cmFjLnJlY29y
ZGVkQ29tbWl0SW5kaWNlc0J5UmV2aXNpb25OdW1iZXIgPSBNb2NrVHJhYy5yZWNvcmRlZENvbW1p
dEluZGljZXNCeVJldmlzaW9uTnVtYmVyOwogICAgIHN0cmljdEVxdWFsKHRoaXMudHJhYy5pbmRl
eE9mUmV2aXNpb24oIjMzMDIwIiksIDIsICJpbmRleE9mUmV2aXNpb24iKTsKIH0pOwogCiB0ZXN0
KCJjb21taXRzT25CcmFuY2hMYXRlclRoYW5SZXZpc2lvbiIsIGZ1bmN0aW9uKCkKIHsKICAgICB0
aGlzLnRyYWMucmVjb3JkZWRDb21taXRzID0gTW9ja1RyYWMuRVhBTVBMRV9UUkFDX0NPTU1JVFM7
CisgICAgdGhpcy50cmFjLnJlY29yZGVkQ29tbWl0SW5kaWNlc0J5UmV2aXNpb25OdW1iZXIgPSBN
b2NrVHJhYy5yZWNvcmRlZENvbW1pdEluZGljZXNCeVJldmlzaW9uTnVtYmVyOwogICAgIHZhciBj
b21taXRzID0gdGhpcy50cmFjLmNvbW1pdHNPbkJyYW5jaExhdGVyVGhhblJldmlzaW9uKCJ0cnVu
ayIsICIzMzAyMCIpOwogICAgIGVxdWFsKGNvbW1pdHMubGVuZ3RoLCAxLCAiZ3JlYXRlciB0aGFu
IDMzMDIwIik7CiB9KTsKQEAgLTExMSw2ICsxMTQsNyBAQCB0ZXN0KCJjb21taXRzT25CcmFuY2hM
YXRlclRoYW5SZXZpc2lvbiIsCiB0ZXN0KCJjb21taXRzT25CcmFuY2hMYXRlclRoYW5SZXZpc2lv
biBubyBjb21taXRzIiwgZnVuY3Rpb24oKQogewogICAgIHRoaXMudHJhYy5yZWNvcmRlZENvbW1p
dHMgPSBNb2NrVHJhYy5FWEFNUExFX1RSQUNfQ09NTUlUUzsKKyAgICB0aGlzLnRyYWMucmVjb3Jk
ZWRDb21taXRJbmRpY2VzQnlSZXZpc2lvbk51bWJlciA9IE1vY2tUcmFjLnJlY29yZGVkQ29tbWl0
SW5kaWNlc0J5UmV2aXNpb25OdW1iZXI7CiAgICAgdmFyIGNvbW1pdHMgPSB0aGlzLnRyYWMuY29t
bWl0c09uQnJhbmNoTGF0ZXJUaGFuUmV2aXNpb24oInNvbWVPdGhlckJyYW5jaCIsICIzMzAyMSIp
OwogICAgIGVxdWFsKGNvbW1pdHMubGVuZ3RoLCAwLCAiZ3JlYXRlciB0aGFuIDMzMDIxIik7CiB9
KTsKQEAgLTExOCw2ICsxMjIsNyBAQCB0ZXN0KCJjb21taXRzT25CcmFuY2hMYXRlclRoYW5SZXZp
c2lvbiBuCiB0ZXN0KCJjb21taXRzT25CcmFuY2hJblJldmlzaW9uUmFuZ2UiLCBmdW5jdGlvbigp
CiB7CiAgICAgdGhpcy50cmFjLnJlY29yZGVkQ29tbWl0cyA9IE1vY2tUcmFjLkVYQU1QTEVfVFJB
Q19DT01NSVRTOworICAgIHRoaXMudHJhYy5yZWNvcmRlZENvbW1pdEluZGljZXNCeVJldmlzaW9u
TnVtYmVyID0gTW9ja1RyYWMucmVjb3JkZWRDb21taXRJbmRpY2VzQnlSZXZpc2lvbk51bWJlcjsK
ICAgICB2YXIgY29tbWl0cyA9IHRoaXMudHJhYy5jb21taXRzT25CcmFuY2hJblJldmlzaW9uUmFu
Z2UoInRydW5rIiwgIjMzMDIwIiwgIjMzMDIyIik7CiAgICAgZXF1YWwoY29tbWl0cy5sZW5ndGgs
IDIsICJpbiByYW5nZSAzMzAyMCwgMzMwMjIiKTsKIH0pOwpAQCAtMTU2LDYgKzE2MSw3IEBAIG1v
ZHVsZSgiQnVpbGRCb3RRdWV1ZVZpZXciLCB7CiAgICAgc2V0dXA6IGZ1bmN0aW9uKCkgewogICAg
ICAgICB0aGlzLnRyYWMgPSBuZXcgTW9ja1RyYWMoKTsKICAgICAgICAgdGhpcy50cmFjLnJlY29y
ZGVkQ29tbWl0cyA9IE1vY2tUcmFjLkVYQU1QTEVfVFJBQ19DT01NSVRTOworICAgICAgICB0aGlz
LnRyYWMucmVjb3JkZWRDb21taXRJbmRpY2VzQnlSZXZpc2lvbk51bWJlciA9IE1vY2tUcmFjLnJl
Y29yZGVkQ29tbWl0SW5kaWNlc0J5UmV2aXNpb25OdW1iZXI7CiAgICAgICAgIHRoaXMucXVldWUg
PSBuZXcgTW9ja0J1aWxkYm90UXVldWUoKTsKICAgICAgICAgdGhpcy50cnVua0JyYW5jaCA9IHsK
ICAgICAgICAgICAgIG5hbWU6ICJ0cnVuayIsCkBAIC0zODIsNiArMzg4LDcgQEAgbW9kdWxlKCJC
dWlsZEJvdFF1ZXVlIiwgewogICAgIHNldHVwOiBmdW5jdGlvbigpIHsKICAgICAgICAgRGFzaGJv
YXJkLlJlcG9zaXRvcnkuT3BlblNvdXJjZS50cmFjID0gbmV3IE1vY2tUcmFjKCk7CiAgICAgICAg
IERhc2hib2FyZC5SZXBvc2l0b3J5Lk9wZW5Tb3VyY2UudHJhYy5yZWNvcmRlZENvbW1pdHMgPSBN
b2NrVHJhYy5FWEFNUExFX1RSQUNfQ09NTUlUUzsKKyAgICAgICAgRGFzaGJvYXJkLlJlcG9zaXRv
cnkuT3BlblNvdXJjZS50cmFjLnJlY29yZGVkQ29tbWl0SW5kaWNlc0J5UmV2aXNpb25OdW1iZXIg
PSBNb2NrVHJhYy5yZWNvcmRlZENvbW1pdEluZGljZXNCeVJldmlzaW9uTnVtYmVyOwogICAgICAg
ICB0aGlzLnF1ZXVlID0gbmV3IE1vY2tCdWlsZGJvdFF1ZXVlKCk7CiAgICAgICAgIHRoaXMucXVl
dWUuYnJhbmNoZXMgPSBbewogICAgICAgICAgICAgbmFtZTogInRydW5rIiwK
</data>

          </attachment>
      

    </bug>

</bugzilla>