<?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>224181</bug_id>
          
          <creation_ts>2021-04-05 04:45:38 -0700</creation_ts>
          <short_desc>results.webkit.org displays svn revision even on selecting git hash</short_desc>
          <delta_ts>2021-04-05 08:20:32 -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>WONTFIX</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="Aakash Jain">aakash_jain</reporter>
          <assigned_to name="Jonathan Bedard">jbedard</assigned_to>
          <cc>jbedard</cc>
    
    <cc>ryanhaddad</cc>
    
    <cc>tsavell</cc>
    
    <cc>webkit-bot-watchers-bugzilla</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1746927</commentid>
    <comment_count>0</comment_count>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2021-04-05 04:45:38 -0700</bug_when>
    <thetext>results.webkit.org displays svn revision even on selecting git hash

To reproduce, open any test results, e.g.: https://results.webkit.org/?suite=layout-tests&amp;test=fast%2Fanimation%2Fanimation-display-style-adjustment.html
change commit representation to &apos;hash&apos;. 
Instead of hashes, revisions are displayed.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1746928</commentid>
    <comment_count>1</comment_count>
    <who name="Aakash Jain">aakash_jain</who>
    <bug_when>2021-04-05 04:46:35 -0700</bug_when>
    <thetext>If the hashes are not fully supported by the results.webkit.org UI yet, that option should not be shown.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1746946</commentid>
    <comment_count>2</comment_count>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2021-04-05 08:20:32 -0700</bug_when>
    <thetext>This is reflecting an issue that is ultimately temporary.

We compute the hash by consulting GitHub after retrieving the identifier. this process works but is apparently not consistent. The solution to this will be to report commits from your local checkout when running tests, we didn&apos;t want to do this with Subversion because retrieving commit messages in Subversion actually requires network operations. The reason I haven&apos;t made this change yet is because I have yet to deploy this change to the Internal results database, because I am populating branches before backfilling that database.

For the time being, I just manually backfilled the effected commits. I think periodically backfilling until we are reporting full commits is better than changing any code here.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>