-
Notifications
You must be signed in to change notification settings - Fork 46
No Edge or Safari stable results in >24 hours #607
Comments
Results are now available for 2018-09-24 and without my intervention. They delay may have been caused by gh-605, which I only took steps to resolve yesterday afternoon. To be honest, though, Sauce Labs builds are so irregular that I don't have a good sense for when they typically complete. I wrote a script (included below) to get some data:
I was surprised by the amount of variation until I remembered that prior to the resolution of gh-602, some Firefox builds were taking over 6 hours to complete. That's relevant due to the way workers are scheduled. Workers may only make one collection attempt at a time. A worker that is actively testing Firefox may be assigned a Sauce Labs job, though. In that case, the job will be deferred until the worker is ready, even if other workers become available in the mean time. If a worker takes 6 hours to complete a Firefox build, then any Sauce Labs build assigned to it will be severely delayed. We can't upload until all the jobs are complete, so this delay is apparent in the result's This is clearly a deficiency of the scheduling algorithm: I didn't consider the possibility of 6-hour jobs when building it. However, this is a failure condition. We want the system to be robust, of course, but in terms of current priorities, it's probably best for now to fix the underlying issue. I've filed gh-608 to track the enhancement. 2018-09-25-timing.pyimport json
import sys
from dateutil import parser
runs = json.loads(sys.stdin.read())
def compare(a, b):
a_created = parser.parse(a['created_at'])
b_created = parser.parse(b['created_at'])
return cmp(a_created, b_created)
by_date = {}
for run in runs:
date_str = run['created_at'].split('T')[0]
if not date_str in by_date:
by_date[date_str] = {}
by_date[date_str][run['browser_name']] = run['created_at']
grouped = [by_date[date_str] for date_str in sorted(by_date.keys())]
print 'edge | safari'
print '-----|-------'
for runs in grouped:
print '%s | %s' % (runs.get('edge', 'missing'), runs.get('safari', 'missing')) |
https://wpt.fyi/test-runs has not seem results for Edge or Safari since 2018-09-24T01:15:29.790252Z and 2018-09-24T03:17:40.904069Z respective. That is now >24 hours ago, while the full runs take something <12 hours, so something is probably wrong.
@jugglinmike @mariestaver FYI
The text was updated successfully, but these errors were encountered: