Bug 235377 - [EWS] ConfigureBuild should display PR information instead of ValidateChange
Summary: [EWS] ConfigureBuild should display PR information instead of ValidateChange
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jonathan Bedard
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-01-19 12:50 PST by Aakash Jain
Modified: 2022-01-25 05:19 PST (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Aakash Jain 2022-01-19 12:50:10 PST 
ConfigureBuild and ValidateChange steps both display same information, about the PR. For e.g.: in https://ews-build.webkit-uat.org/#/builders/1/builds/1687 
ConfigureBuild display: "Pull request 78", and ValidateChange display: "[PR 78] [webkitbugspy] Support automted comments" Both link to same url, and duplicate the information displayed. Either one of the step should display this info. 

Ideally ConfigureBuild should display all these information if it can. In bugzilla work flow ConfigureBuild didn't had bug title information, that's why it couldn't display it and ValidatePatch display it. But in git workflow ConfigureBuild would probably have the required information. So we can display all these information together in ConfigureBuild step and not display such info in ValidateChange step. This would be more valuable in the builds in which RelevanceCheck fails.
Comment 1 Radar WebKit Bug Importer 2022-01-20 15:29:08 PST 
<rdar://problem/87853245>
Comment 2 Jonathan Bedard 2022-01-20 15:42:04 PST 
Pull-request: https://github.com/WebKit/WebKit/pull/84
Comment 3 Jonathan Bedard 2022-01-21 10:55:53 PST 
Landed 246269@main (r288367)