Bug 247900 - Add support for skip-ews GitHub label
Summary: Add support for skip-ews GitHub label
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jonathan Bedard
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-11-14 08:11 PST by Aakash Jain
Modified: 2023-01-26 11:26 PST (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Aakash Jain 2022-11-14 08:11:28 PST
Add support for Skip-EWS GitHub label. This label can be applied to a PR to skip EWS runs. There might be various scenarios in which people might want to apply such a label.

Foe e.g:
- Some engineers might want to abort EWS as soon as the they know that the change is going to be bad (e.g: an important ews on their patch failed, and they don't want to keep running on all other ewses).
- Someone just re-uploaded the change after doing a trivial change (add reviewer name, fix typo in PR title etc.)
- Admins wants to stop a bad PR (which might be breaking EWS bots) from further processing
Comment 1 Aakash Jain 2022-11-14 08:14:05 PST
Note that, in bugzilla world, there are three ways to do similar thing:
1) r- the patch
2) mark the patch as obsolete from bugzilla UI
3) --no-ews while uploading the patch

In GitHub world, we don't have any way to do this.

Equivalent of r- in GitHub is marking the PR as Request changes, which still triggers the EWS. There is no way to mark a commit as obsolete in GitHub UI without uploading a new commit.
Comment 2 Aakash Jain 2022-11-14 08:19:44 PST
Pull request: https://github.com/WebKit/WebKit/pull/6476
Comment 3 EWS 2022-11-16 17:26:13 PST
Committed 256760@main (57ba798e7aa4): <https://commits.webkit.org/256760@main>

Reviewed commits have been landed. Closing PR #6476 and removing active labels.
Comment 4 Radar WebKit Bug Importer 2022-11-16 17:27:17 PST
<rdar://problem/102444473>
Comment 5 Jonathan Bedard 2022-11-17 16:25:11 PST
Re-opening for pull request https://github.com/apple/WebKit/pull/216
Comment 6 EWS 2022-11-18 08:37:23 PST
Committed 252432.837@safari-7614-branch (8dbf19696f48): <https://commits.webkit.org/252432.837@safari-7614-branch>

Reviewed commits have been landed. Closing PR #216 and removing active labels.