Bug 235033

Summary: [EWS] Accept GitHub hooks
Product: WebKit Reporter: Jonathan Bedard <jbedard>
Component: Tools / TestsAssignee: Jonathan Bedard <jbedard>
Status: RESOLVED FIXED    
Severity: Normal CC: aakash_jain, ews-watchlist, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=234847
https://bugs.webkit.org/show_bug.cgi?id=235913
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
none
Patch none

Description Jonathan Bedard 2022-01-10 09:33:37 PST
ewe-build.webkit.org should accept hooks from GitHub.
Comment 1 Radar WebKit Bug Importer 2022-01-10 09:33:57 PST
<rdar://problem/87345269>
Comment 2 Jonathan Bedard 2022-01-10 09:39:07 PST
PR: https://github.com/WebKit/WebKit/pull/66
Comment 3 Jonathan Bedard 2022-01-12 09:20:15 PST
Created attachment 448953 [details]
Patch
Comment 4 Jonathan Bedard 2022-01-12 09:30:25 PST
Created attachment 448955 [details]
Patch
Comment 5 Jonathan Bedard 2022-01-12 11:43:17 PST
Created attachment 448979 [details]
Patch
Comment 6 Jonathan Bedard 2022-01-12 12:00:33 PST
Created attachment 448982 [details]
Patch
Comment 7 Jonathan Bedard 2022-01-12 13:09:51 PST
Created attachment 448988 [details]
Patch
Comment 8 Jonathan Bedard 2022-01-12 13:19:10 PST Comment hidden (obsolete)
Comment 9 Aakash Jain 2022-01-12 15:20:35 PST
Comment on attachment 448988 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=448988&action=review

> Tools/CISupport/ews-build/master.cfg:57
>      admin_username = os.getenv('EWS_ADMIN_USERNAME')

would be good to change other os.getenv in this file to load_password as well (in subsequent patch).
Comment 10 Aakash Jain 2022-01-17 14:07:46 PST
(In reply to Aakash Jain from comment #9)
> would be good to change other os.getenv in this file to load_password as well (in subsequent patch).
Doing this in Bug 235296.
Comment 11 Aakash Jain 2022-01-17 14:09:55 PST
(In reply to Jonathan Bedard from comment #8)
> Landed: 245976@main (r287947)
Above comment was incorrect. Correct commit id was: https://commits.webkit.org/245978@main (287949)