Bug 248042 – Background Queue monitor to remove outdated hashes of PR from queue and update timeline

• Status: NEW
• Component: Tools / Tests

[Ahmad Saleem]

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!

[Jonathan Bedard]

If we were going to do this, we should do it on hook reception instead of polling.

That being said, it'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's running on are stale.

[Ahmad Saleem]

(In reply to Jonathan Bedard from comment #1)
> If we were going to do this, we should do it on hook reception instead of
> polling.
> 
> That being said, it'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's running on are stale.

Indeed! Hence I said in first line (psychology point of view - might be zero tangible benefit).. 😆

[Radar WebKit Bug Importer]

<rdar://problem/102649650>