Bug 222282

Summary: Switch build.webkit.org bots to github
Product: WebKit Reporter: Aakash Jain <aakash_jain>
Component: Tools / TestsAssignee: Aakash Jain <aakash_jain>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: don.olmstead, jbedard, ryanhaddad, stephan.szabo, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=224497
Bug Depends on: 222283, 222608    
Bug Blocks:    
Attachments:
Description Flags
Patch
none
Patch
jbedard: review+
Patch none

Aakash Jain
Reported 2021-02-22 12:12:04 PST
As part for transition to github, we should switch build.webkit.org bots to use github.com Similar to https://bugs.webkit.org/show_bug.cgi?id=220479.
Attachments
Patch (3.51 KB, patch)
2021-02-22 12:21 PST, Aakash Jain 		no flags
DetailsFormatted DiffDiff
Patch (5.74 KB, patch)
2021-02-22 12:32 PST, Aakash Jain 		jbedard: review+
DetailsFormatted DiffDiff
Patch (9.16 KB, patch)
2021-07-05 12:24 PDT, Aakash Jain 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Aakash Jain
Comment 1 2021-02-22 12:21:10 PST
Created attachment 421224 [details] Patch
Radar WebKit Bug Importer
Comment 2 2021-02-22 12:21:39 PST
<rdar://problem/74608323>
Aakash Jain
Comment 3 2021-02-22 12:25:42 PST
(In reply to Aakash Jain from comment #1) > Created attachment 421224 [details] Tested in https://build.webkit-dev.org/#/builders/32/builds/40
Jonathan Bedard
Comment 5 2021-02-22 12:31:06 PST
Comment on attachment 421224 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=421224&action=review > Tools/CISupport/build-webkit-org/steps.py:140 > + timeout=2 * 60 * 60, This is a 2 hour timeout, right? How did we determine the timeout before?
Aakash Jain
Comment 6 2021-02-22 12:32:50 PST
Created attachment 421225 [details] Patch
Aakash Jain
Comment 7 2021-02-22 12:34:13 PST
> This is a 2 hour timeout, right? How did we determine the timeout before? Yes, just matching it with ews code. So that in long-term we can share the code. Also updated ShowIdentifier step in updated patch.
Aakash Jain
Comment 8 2021-07-05 12:24:55 PDT
Created attachment 432893 [details] Patch Rebased on ToT and added few unit-tests. Still blocked by Bug 222283.
Aakash Jain
Comment 9 2022-04-26 07:45:12 PDT
*** This bug has been marked as a duplicate of bug 239072 ***
Note You need to log in before you can comment on or make changes to this bug.