<?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>228741</bug_id>
          
          <creation_ts>2021-08-03 10:08:06 -0700</creation_ts>
          <short_desc>[check-github-mirror-integrity] Handle slow Svn query</short_desc>
          <delta_ts>2021-08-03 10:49:13 -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>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jonathan Bedard">jbedard</reporter>
          <assigned_to name="Jonathan Bedard">jbedard</assigned_to>
          <cc>aakash_jain</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1781642</commentid>
    <comment_count>0</comment_count>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2021-08-03 10:08:06 -0700</bug_when>
    <thetext>Querying Svn for a commit is slow, and because we query Svn before GitHub, it&apos;s possible that GitHub is ahead of Svn. This breaks our logic in check-github-mirror-integrity, which only expects GitHub to be behind Svn. If we reverse the order we fetch commits in, we shouldn&apos;t have this problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1781643</commentid>
    <comment_count>1</comment_count>
      <attachid>434838</attachid>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2021-08-03 10:11:53 -0700</bug_when>
    <thetext>Created attachment 434838
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1781645</commentid>
    <comment_count>2</comment_count>
      <attachid>434839</attachid>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2021-08-03 10:18:24 -0700</bug_when>
    <thetext>Created attachment 434839
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1781646</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2021-08-03 10:20:05 -0700</bug_when>
    <thetext>&lt;rdar://problem/81467944&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1781653</commentid>
    <comment_count>4</comment_count>
    <who name="EWS">ews-feeder</who>
    <bug_when>2021-08-03 10:49:12 -0700</bug_when>
    <thetext>Committed r280602 (240220@main): &lt;https://commits.webkit.org/240220@main&gt;

All reviewed patches have been landed. Closing bug and clearing flags on attachment 434839.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>434838</attachid>
            <date>2021-08-03 10:11:53 -0700</date>
            <delta_ts>2021-08-03 10:18:22 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-228741-20210803101152.patch</filename>
            <type>text/plain</type>
            <size>1433</size>
            <attacher name="Jonathan Bedard">jbedard</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDI4MDU5OSkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDEzIEBACisyMDIxLTA4LTAzICBKb25hdGhhbiBCZWRhcmQgIDxqYmVkYXJkQGFwcGxlLmNv
bT4KKworICAgICAgICBbY2hlY2stZ2l0aHViLW1pcnJvci1pbnRlZ3JpdHldIEhhbmRsZSBzbG93
IFN2biBxdWVyeQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9MjI4NzQxCisgICAgICAgIAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEp
LgorCisgICAgICAgICogU2NyaXB0cy9jaGVjay1naXRodWItbWlycm9yLWludGVncml0eTogUHVs
bCBjb21taXQgZnJvbSBHaXRIdWIgYmVmb3JlIFN1YnZlcnNpb24uCisKIDIwMjEtMDgtMDMgIFBo
aWxpcHBlIE5vcm1hbmQgIDxwbm9ybWFuZEBpZ2FsaWEuY29tPgogCiAgICAgICAgIFtXUEVdW0dU
S10gU1ZOX1JFVklTSU9OIGRyaWZ0aW5nIGF3YXkgaWYgYm90cyBkb24ndCByZS1ydW4gY21ha2UK
SW5kZXg6IFRvb2xzL1NjcmlwdHMvY2hlY2stZ2l0aHViLW1pcnJvci1pbnRlZ3JpdHkKPT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PQotLS0gVG9vbHMvU2NyaXB0cy9jaGVjay1naXRodWItbWlycm9yLWludGVncml0eQkocmV2
aXNpb24gMjgwNTk5KQorKysgVG9vbHMvU2NyaXB0cy9jaGVjay1naXRodWItbWlycm9yLWludGVn
cml0eQkod29ya2luZyBjb3B5KQpAQCAtMzAsMTEgKzMwLDExIEBAIGZyb20gd2Via2l0c2NtcHkg
aW1wb3J0IHJlbW90ZQogCiAKIGRlZiBtYWluKCk6Ci0gICAgcmVwb3NpdG9yeSA9IHJlbW90ZS5T
dm4oJ2h0dHBzOi8vc3ZuLndlYmtpdC5vcmcvcmVwb3NpdG9yeS93ZWJraXQnKQogICAgIG1pcnJv
ciA9IHJlbW90ZS5HaXRIdWIoJ2h0dHBzOi8vZ2l0aHViLmNvbS9XZWJLaXQvV2ViS2l0JykKKyAg
ICByZXBvc2l0b3J5ID0gcmVtb3RlLlN2bignaHR0cHM6Ly9zdm4ud2Via2l0Lm9yZy9yZXBvc2l0
b3J5L3dlYmtpdCcpICAgIAogCi0gICAgY2Fub25pY2FsX3RpcCA9IHJlcG9zaXRvcnkuY29tbWl0
KGJyYW5jaD0ndHJ1bmsnKQogICAgIG1pcnJvcl90aXAgPSBtaXJyb3IuY29tbWl0KGJyYW5jaD0n
bWFpbicpCisgICAgY2Fub25pY2FsX3RpcCA9IHJlcG9zaXRvcnkuY29tbWl0KGJyYW5jaD0ndHJ1
bmsnKQogCiAgICAgaWYgbWlycm9yX3RpcC5pZGVudGlmaWVyID09IGNhbm9uaWNhbF90aXAuaWRl
bnRpZmllcjoKICAgICAgICAgcHJpbnQoJ0dpdEh1YiBhbmQgc3ZuLndlYmtpdC5vcmcgYXJlIGlu
IHN5bmMnKQo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>434839</attachid>
            <date>2021-08-03 10:18:24 -0700</date>
            <delta_ts>2021-08-03 10:49:13 -0700</delta_ts>
            <desc>Patch for landing</desc>
            <filename>bug-228741-20210803101823.patch</filename>
            <type>text/plain</type>
            <size>1420</size>
            <attacher name="Jonathan Bedard">jbedard</attacher>
            
              <data encoding="base64">SW5kZXg6IFRvb2xzL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29scy9DaGFuZ2VMb2cJKHJl
dmlzaW9uIDI4MDU5OSkKKysrIFRvb2xzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDEyIEBACisyMDIxLTA4LTAzICBKb25hdGhhbiBCZWRhcmQgIDxqYmVkYXJkQGFwcGxlLmNv
bT4KKworICAgICAgICBbY2hlY2stZ2l0aHViLW1pcnJvci1pbnRlZ3JpdHldIEhhbmRsZSBzbG93
IFN2biBxdWVyeQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9MjI4NzQxCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgQWFrYXNoIEphaW4uCisKKyAgICAgICAg
KiBTY3JpcHRzL2NoZWNrLWdpdGh1Yi1taXJyb3ItaW50ZWdyaXR5OiBQdWxsIGNvbW1pdCBmcm9t
IEdpdEh1YiBiZWZvcmUgU3VidmVyc2lvbi4KKwogMjAyMS0wOC0wMyAgUGhpbGlwcGUgTm9ybWFu
ZCAgPHBub3JtYW5kQGlnYWxpYS5jb20+CiAKICAgICAgICAgW1dQRV1bR1RLXSBTVk5fUkVWSVNJ
T04gZHJpZnRpbmcgYXdheSBpZiBib3RzIGRvbid0IHJlLXJ1biBjbWFrZQpJbmRleDogVG9vbHMv
U2NyaXB0cy9jaGVjay1naXRodWItbWlycm9yLWludGVncml0eQo9PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBUb29s
cy9TY3JpcHRzL2NoZWNrLWdpdGh1Yi1taXJyb3ItaW50ZWdyaXR5CShyZXZpc2lvbiAyODA1OTkp
CisrKyBUb29scy9TY3JpcHRzL2NoZWNrLWdpdGh1Yi1taXJyb3ItaW50ZWdyaXR5CSh3b3JraW5n
IGNvcHkpCkBAIC0zMCwxMSArMzAsMTEgQEAgZnJvbSB3ZWJraXRzY21weSBpbXBvcnQgcmVtb3Rl
CiAKIAogZGVmIG1haW4oKToKLSAgICByZXBvc2l0b3J5ID0gcmVtb3RlLlN2bignaHR0cHM6Ly9z
dm4ud2Via2l0Lm9yZy9yZXBvc2l0b3J5L3dlYmtpdCcpCiAgICAgbWlycm9yID0gcmVtb3RlLkdp
dEh1YignaHR0cHM6Ly9naXRodWIuY29tL1dlYktpdC9XZWJLaXQnKQorICAgIHJlcG9zaXRvcnkg
PSByZW1vdGUuU3ZuKCdodHRwczovL3N2bi53ZWJraXQub3JnL3JlcG9zaXRvcnkvd2Via2l0Jykg
ICAgCiAKLSAgICBjYW5vbmljYWxfdGlwID0gcmVwb3NpdG9yeS5jb21taXQoYnJhbmNoPSd0cnVu
aycpCiAgICAgbWlycm9yX3RpcCA9IG1pcnJvci5jb21taXQoYnJhbmNoPSdtYWluJykKKyAgICBj
YW5vbmljYWxfdGlwID0gcmVwb3NpdG9yeS5jb21taXQoYnJhbmNoPSd0cnVuaycpCiAKICAgICBp
ZiBtaXJyb3JfdGlwLmlkZW50aWZpZXIgPT0gY2Fub25pY2FsX3RpcC5pZGVudGlmaWVyOgogICAg
ICAgICBwcmludCgnR2l0SHViIGFuZCBzdm4ud2Via2l0Lm9yZyBhcmUgaW4gc3luYycpCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>