Bug 278470

Summary: [ews-build] Do not print worker environment variables in find-modified-layout-tests step
Product: WebKit Reporter: Aakash Jain <aakash_jain>
Component: Tools / TestsAssignee: Aakash Jain <aakash_jain>
Status: RESOLVED FIXED    
Severity: Normal CC: bfan2, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: Unspecified   
OS: Unspecified   
See Also: https://bugs.webkit.org/show_bug.cgi?id=197319
Attachments:
Description Flags
build page with find-modified-layout-tests logs none

Aakash Jain
Reported 2024-08-21 10:26:09 PDT
[ews-build] Do not print worker environment variables in find-modified-layout-tests step. It makes the buildbot build page hard to read (or not so nice), especially this step is skipped most of the times, and buildbot expands it by default showing all these unnecessary details on front. This causes users to have to scroll lot more to see the relevant things. See attached screenshot. In almost all other steps we have it disabled. Similar to https://bugs.webkit.org/show_bug.cgi?id=197319
Attachments
build page with find-modified-layout-tests logs (341.37 KB, image/png)
2024-08-21 10:27 PDT, Aakash Jain 		no flags
Details
View All Add attachment proposed patch, testcase, etc.
Aakash Jain
Comment 1 2024-08-21 10:27:06 PDT
Created attachment 472258 [details] build page with find-modified-layout-tests logs
Aakash Jain
Comment 2 2024-08-21 10:32:10 PDT
Pull request: https://github.com/WebKit/WebKit/pull/32535
EWS
Comment 3 2024-08-21 11:23:33 PDT
Committed 282571@main (b215773cfd9d): <https://commits.webkit.org/282571@main> Reviewed commits have been landed. Closing PR #32535 and removing active labels.
Radar WebKit Bug Importer
Comment 4 2024-08-21 11:24:13 PDT
<rdar://problem/134432320>
Note You need to log in before you can comment on or make changes to this bug.