Bug 149942

Summary: Perf dashboard erroneously shows an old OS build in A/B testing range
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: Perf DashboardAssignee: Ryosuke Niwa <rniwa>
Status: RESOLVED FIXED    
Severity: Normal CC: barraclough, cdumez, dewei_zhu, jond, kling, koivisto, rniwa, slewis, timothy
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Fixes the bug
none
Removed a stale comment darin: review+

Ryosuke Niwa
Reported 2015-10-08 20:12:46 PDT
Ordering OS builds lexicologically turned out be a bad idea since 15A25 falls between 15A242 and 15A251. Use a fake/synthetic timestamp to force the commonly understood total order.
Attachments
Fixes the bug (8.92 KB, patch)
2015-10-08 20:27 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
Removed a stale comment (8.83 KB, patch)
2015-10-08 20:28 PDT, Ryosuke Niwa 		darin: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2015-10-08 20:27:37 PDT
Created attachment 262736 [details] Fixes the bug
Ryosuke Niwa
Comment 2 2015-10-08 20:28:41 PDT
Created attachment 262737 [details] Removed a stale comment
Ryosuke Niwa
Comment 3 2015-10-08 20:29:16 PDT
Comment on attachment 262737 [details] Removed a stale comment View in context: https://bugs.webkit.org/attachment.cgi?id=262737&action=review > Websites/perf.webkit.org/tools/pull-os-versions.py:24 > + parser.add_argument('--seconds-to-sleep', type=float, default=43200, help='The seconds to sleep between iterations') We only update OS builds every 12 hours.
Ryosuke Niwa
Comment 4 2015-10-09 14:00:57 PDT
Committed r190817: <http://trac.webkit.org/changeset/190817>
Note You need to log in before you can comment on or make changes to this bug.