Summary: | last-green-revision should report the revision that succeeded on all bots | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryosuke Niwa <rniwa> | ||||
Component: | Tools / Tests | Assignee: | Ryosuke Niwa <rniwa> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, dpranke, eric, ojan, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Bug Depends on: | 76011 | ||||||
Bug Blocks: | |||||||
Attachments: |
|
Description
Ryosuke Niwa
2012-01-11 15:09:31 PST
Created attachment 122107 [details]
Adds the feature
With this patch, sheriffbot can tell us the last known green revision as in: rniwa-macpro:webkit4 rniwa$ Tools/Scripts/webkit-patch last-green-revision Qt The last known green revision is 104740 Qt Linux Release: 104744 Qt Linux Release minimal: 104749 Qt Linux ARMv7 Release: 104749 Qt Windows 32-bit Release: 104749 Qt Windows 32-bit Debug: 104749 Qt SnowLeopard Release: 104740 Qt Linux MIPS Release: 104748 Qt Linux SH4 Release: 104748 When there is no such a revision, it'll still give us the last known green revision per bot as in: rniwa-macpro:webkit4 rniwa$ Tools/Scripts/webkit-patch last-green-revision Chromium Chromium Win Release: 104748 Chromium Win Release (Tests) has had no green revision in the last 100 runs Chromium Mac Release: 104747 Chromium Mac Release (Tests): 104744 Chromium Linux Release: 104749 Chromium Linux Release (Tests): 104746 Chromium Linux Release (Grid Layout): 104746 Chromium Android Release: 104649 Comment on attachment 122107 [details] Adds the feature View in context: https://bugs.webkit.org/attachment.cgi?id=122107&action=review > Tools/Scripts/webkitpy/common/net/buildbot/buildbot.py:462 > + # Once we found X, check if remaining builders succeeded in the neiborhood of X. typo: neiborhood Thanks for the review! (In reply to comment #4) > (From update of attachment 122107 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=122107&action=review > > > Tools/Scripts/webkitpy/common/net/buildbot/buildbot.py:462 > > + # Once we found X, check if remaining builders succeeded in the neiborhood of X. > > typo: neiborhood Oops, will fix. Committed r104754: <http://trac.webkit.org/changeset/104754> |