Bug 179506 - Use Buildbot for OpenSource EWS
Summary: Use Buildbot for OpenSource EWS
Status: RESOLVED FIXED
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:
Blocks:
 
Reported: 2017-11-09 13:57 PST by Aakash Jain
Modified: 2019-09-18 12:19 PDT (History)
10 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 2017-11-09 13:57:43 PST
We should use Buildbot for OpenSource EWS. Currently our code in QueueStatusServer and webkitpy is implementing functionality similar to buildbot as well as logic for building, testing etc. The codebase is very complicated to navigate and scale. Various enhancements planned for EWS, are already implemented by Buildbot or would be very easy to implement using Buildbot.

Current codebase is not very scalable. Adding a queue requires significant amount of code. For e.g.: https://bugs.webkit.org/show_bug.cgi?id=178480 has large patch to add webkitpy EWS, if using buildbot, this could easily be done in 10-20 lines of code.

Buildbot would make EWS more scalable and easier to work on since we already manage Buildbot instances and many engineers in WebKit have expertise with Buildbot.

Few examples of tasks which would be easier with Buildbot:

- Seeing failures logs in real-time would be automatic with Buildbot <rdar://problem/29490561> 

- API tests could be easily added as a buildbot step (https://bugs.webkit.org/show_bug.cgi?id=106596)

- Implementing Security EWS.

- Handling large logs would be easier as well <rdar://problem/34841885>

- Retrying a build would be easier as well.


See <rdar://problem/35345858>.
Comment 1 Daniel Bates 2018-06-19 15:31:14 PDT
(In reply to Aakash Jain from comment #0)
> - Seeing failures logs in real-time would be automatic with Buildbot
> <rdar://problem/29490561> 
> 

Filed bug #186823 to implement this feature in the current EWS architecture.

> - API tests could be easily added as a buildbot step
> (https://bugs.webkit.org/show_bug.cgi?id=106596)
> 
> - Implementing Security EWS.
> 

The patch for bug #186291 implements most of this feature (minus bug comments and uploading of layout test results) in the current EWS architecture.
Comment 2 Aakash Jain 2019-09-18 12:17:54 PDT
OpenSource EWS has been running Buildbot in production (with most of the queues moved from old EWS and few new queues like api-tests, services, webkitperl etc).

Rest of the work would be tracked in individual bugs (e.g.: https://bugs.webkit.org/show_bug.cgi?id=201934 https://bugs.webkit.org/show_bug.cgi?id=201932 https://bugs.webkit.org/show_bug.cgi?id=201930).