Bug 222282 - Switch build.webkit.org bots to github
Summary: Switch build.webkit.org bots to github
Status: RESOLVED DUPLICATE of bug 239072
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Aakash Jain
URL:
Keywords: InRadar
Depends on: 222283 222608
Blocks:
  Show dependency treegraph
 
Reported: 2021-02-22 12:12 PST by Aakash Jain
Modified: 2022-04-26 07:45 PDT (History)
5 users (show)

See Also:


Attachments
Patch (3.51 KB, patch)
2021-02-22 12:21 PST, Aakash Jain
no flags Details | Formatted Diff | Diff
Patch (5.74 KB, patch)
2021-02-22 12:32 PST, Aakash Jain
jbedard: review+
Details | Formatted Diff | Diff
Patch (9.16 KB, patch)
2021-07-05 12:24 PDT, Aakash Jain
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Aakash Jain 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.
Comment 1 Aakash Jain 2021-02-22 12:21:10 PST
Created attachment 421224 [details]
Patch
Comment 2 Radar WebKit Bug Importer 2021-02-22 12:21:39 PST
<rdar://problem/74608323>
Comment 3 Aakash Jain 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
Comment 5 Jonathan Bedard 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?
Comment 6 Aakash Jain 2021-02-22 12:32:50 PST
Created attachment 421225 [details]
Patch
Comment 7 Aakash Jain 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.
Comment 8 Aakash Jain 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.
Comment 9 Aakash Jain 2022-04-26 07:45:12 PDT

*** This bug has been marked as a duplicate of bug 239072 ***