RESOLVED FIXED 219016
BuildbotTriggerable should not process a completed build request from a completed test group. 
https://bugs.webkit.org/show_bug.cgi?id=219016
Summary BuildbotTriggerable should not process a completed build request from a compl...
dewei_zhu
Reported 2020-11-16 18:47:17 PST
BuildBotTriggerable should not process a completed build request from a completed test group.
Attachments
Patch (8.51 KB, patch)
2020-11-16 19:04 PST, dewei_zhu 		rniwa: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
dewei_zhu
Comment 1 2020-11-16 19:04:11 PST
Created attachment 414298 [details] Patch
Ryosuke Niwa
Comment 2 2020-11-16 19:30:19 PST
Comment on attachment 414298 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=414298&action=review > Websites/perf.webkit.org/ChangeLog:9 > + BuildBotTriggerable._pullBuildbotOnAllSyncers assumes BuildRequest.all() retuns all build requests from triggerables. > + And it relies on BuildRequest.findById to filter out buildbot build entries and corresponding build requests from a This explanation is rather misleading. The issue here isn't really that _pullBuildbotOnAllSyncers assumes that but it's rather that it relies on buildReqeustsByGroup to contain all build requests found by BuildRequest.findById. > Websites/perf.webkit.org/tools/js/buildbot-triggerable.js:182 > + if (!info) > + continue; We should probably assert here that in this case, request's test group had finished running.
dewei_zhu
Comment 3 2020-11-16 20:33:50 PST
Landed in r269892.
Note You need to log in before you can comment on or make changes to this bug.