| Summary: | [ews] Notify bot watchers about pre-existing and flaky test failures for JSC tests | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||
| Component: | Tools / Tests | Assignee: | Aakash Jain <aakash_jain> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | aakash_jain, ap, jbedard, ryanhaddad, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | Other | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| See Also: |
https://bugs.webkit.org/show_bug.cgi?id=215949 https://bugs.webkit.org/show_bug.cgi?id=207475 https://bugs.webkit.org/show_bug.cgi?id=216889 |
||||||
| Attachments: |
|
||||||
|
Description
Aakash Jain
2020-08-28 13:33:59 PDT
Created attachment 407958 [details]
Patch
Comment on attachment 407958 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=407958&action=review > Tools/BuildSlaveSupport/ews-build/steps.py:1781 > + def send_email_for_flaky_failure(self, test_name): Same deal as https://bugs.webkit.org/show_bug.cgi?id=215949, the only variability here seems to be the suite, should we consider basing all of these analysis steps on the same base class? (In reply to Jonathan Bedard from comment #2) > Same deal as https://bugs.webkit.org/show_bug.cgi?id=215949, the only variability here seems to be the suite, should we consider basing all of these analysis steps on the same base class? Yeah, will do that in a separate patch. commit-queue failed to commit attachment 407958 [details] to WebKit repository.
Committed r266629: <https://trac.webkit.org/changeset/266629> All reviewed patches have been landed. Closing bug and clearing flags on attachment 407958 [details]. |