<?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>248042</bug_id>
          
          <creation_ts>2022-11-17 09:02:02 -0800</creation_ts>
          <short_desc>Background Queue monitor to remove outdated hashes of PR from queue and update timeline</short_desc>
          <delta_ts>2022-11-24 09:02:16 -0800</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>Safari Technology Preview</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>NEW</bug_status>
          <resolution></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>
          
          <blocked>248037</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Ahmad Saleem">ahmad.saleem792</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>jbedard</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1913197</commentid>
    <comment_count>0</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2022-11-17 09:02:02 -0800</bug_when>
    <thetext>Hi Team,

Another improvement to EWS (at least to user from psychology point of view - might be zero tangible benefit), it would be to run another background script on all queue PRs for build requests and check every 10 minutes or 20 minutes (can be tweaked) to ensure that if any PR has build request but now it is update or outdated build hash then remove it from the queue.

This will help for those EWS system (e.g., mac-wk1 etc.), which take time or slow because with background script running and removing outdated hashes from queue, it will not show 18 hours queue to contributors and demotivate them.

With showing just six hours queue since now it has clean all outdated hashes from build pipeline, it would be good and smarter approach IMO.

Again - I am just suggesting but the pros and cons from inside knowledge can be validated by Tools team from Apple.

Thanks!</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1913220</commentid>
    <comment_count>1</comment_count>
    <who name="Jonathan Bedard">jbedard</who>
    <bug_when>2022-11-17 10:08:15 -0800</bug_when>
    <thetext>If we were going to do this, we should do it on hook reception instead of polling.

That being said, it&apos;s less of a benefit than you might think because the very first thing every queue does before running a job is check if the PR and commit it&apos;s running on are stale.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1913223</commentid>
    <comment_count>2</comment_count>
    <who name="Ahmad Saleem">ahmad.saleem792</who>
    <bug_when>2022-11-17 10:14:01 -0800</bug_when>
    <thetext>(In reply to Jonathan Bedard from comment #1)
&gt; If we were going to do this, we should do it on hook reception instead of
&gt; polling.
&gt; 
&gt; That being said, it&apos;s less of a benefit than you might think because the
&gt; very first thing every queue does before running a job is check if the PR
&gt; and commit it&apos;s running on are stale.

Indeed! Hence I said in first line (psychology point of view - might be zero tangible benefit).. 😆</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1914504</commentid>
    <comment_count>3</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2022-11-24 09:02:16 -0800</bug_when>
    <thetext>&lt;rdar://problem/102649650&gt;</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>