| Summary: | EWS should be able to search existing bugs for given test | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Aakash Jain <aakash_jain> | ||||
| Component: | Tools / Tests | Assignee: | Aakash Jain <aakash_jain> | ||||
| Status: | ASSIGNED --- | ||||||
| Severity: | Normal | CC: | aakash_jain, ap, jbedard, ryanhaddad, webkit-bot-watchers-bugzilla | ||||
| Priority: | P2 | ||||||
| Version: | Other | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=207845 | ||||||
| Bug Depends on: | |||||||
| Bug Blocks: | 207935 | ||||||
| Attachments: |
|
||||||
|
Description
Aakash Jain
2020-02-18 15:26:01 PST
Created attachment 391106 [details]
Patch
Comment on attachment 391106 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=391106&action=review > Tools/BuildSlaveSupport/ews-build/steps.py:498 > + def fetch_bug_id_for_flaky_test(self, flaky_test_name): Inspired from _lookup_bug_for_flaky_test https://trac.webkit.org/browser/webkit/trunk/Tools/Scripts/webkitpy/tool/bot/flakytestreporter.py#L70 Before landing this, I think we should have a discussion about if this actually the right way to achieve our goals. We had an offline discussion about this today, I wonder if we should start with an umbrella bug with the problem statement and more formally determine which solution we want before implementing a solution. This is required to migrate the existing functionality of commit-queue from old ews to new ews (irrespective of whether we decide for other EWS queue to automatically file bugs for flaky tests or not). I believe there isn't a disagreement about commit-queue filing bugs for flaky tests. (In reply to Aakash Jain from comment #4) > This is required to migrate the existing functionality of commit-queue from > old ews to new ews (irrespective of whether we decide for other EWS queue to > automatically file bugs for flaky tests or not). > > I believe there isn't a disagreement about commit-queue filing bugs for > flaky tests. Didn't realize we have commit queue doing this now. |