Bug 226353 - Print bot configuration in build.webkit.org builds
Summary: Print bot configuration in build.webkit.org builds
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: Aakash Jain
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2021-05-27 13:37 PDT by Aakash Jain
Modified: 2021-06-01 09:29 PDT (History)
7 users (show)

See Also:


Attachments
[fast-cq] Patch (17.11 KB, patch)
2021-05-27 13:42 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-05-27 13:37:58 PDT
Print bot configuration in build.webkit.org builds (e.g.: OS/xcode version, uptime, disk-space etc.), similar to how we print in EWS builds. This would help in debugging any bot related issues.
Comment 1 Aakash Jain 2021-05-27 13:42:43 PDT
Created attachment 429922 [details]
[fast-cq] Patch
Comment 2 Aakash Jain 2021-05-27 13:45:22 PDT
Sample runs: https://build.webkit-dev.org/#/builders/17/builds/65
Comment 3 Jonathan Bedard 2021-05-27 14:14:09 PDT
(In reply to Aakash Jain from comment #2)
> Sample runs: https://build.webkit-dev.org/#/builders/17/builds/65

I notice that the linked run prints out watchOS information, but the run is associated with the iPad queue? What's going on with that?
Comment 4 Aakash Jain 2021-05-27 15:14:14 PDT
Not sure. The bot might have multiple SDKs installed.
Comment 5 Jonathan Bedard 2021-05-27 15:25:39 PDT
(In reply to Aakash Jain from comment #4)
> Not sure. The bot might have multiple SDKs installed.

I didn't look at the entire log. You're dumping all installed SDKs, so that makes sense. Is the goal here just to have this information accessible if someone needs it?
Comment 6 Aakash Jain 2021-05-28 03:34:45 PDT
(In reply to Jonathan Bedard from comment #5)
> Is the goal here just to have this information accessible if someone needs it?
Yes
Comment 7 Alexey Proskuryakov 2021-05-28 17:25:10 PDT
Comment on attachment 429922 [details]
[fast-cq] Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=429922&action=review

> Tools/CISupport/build-webkit-org/steps_unittest.py:1203
> +            + ExpectShell.log('stdio', stdout='''Upon execvpe sw_vers ['sw_vers'] in environment id 7696545650400
> +:Traceback (most recent call last):
> +  File "/usr/lib/python2.7/site-packages/twisted/internet/process.py", line 445, in _fork
> +    environment)
> +  File "/usr/lib/python2.7/site-packages/twisted/internet/process.py", line 523, in _execChild
> +    os.execvpe(executable, args, environment)
> +  File "/usr/lib/python2.7/os.py", line 355, in execvpe
> +    _execvpe(file, args, env)
> +  File "/usr/lib/python2.7/os.py", line 382, in _execvpe
> +    func(fullname, *argrest)
> +OSError: [Errno 2] No such file or directory'''),

This seems surprising. Not sure if I understand the logic of the test.
Comment 8 Aakash Jain 2021-05-29 05:08:05 PDT
Comment on attachment 429922 [details]
[fast-cq] Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=429922&action=review

>> Tools/CISupport/build-webkit-org/steps_unittest.py:1203
>> +OSError: [Errno 2] No such file or directory'''),
> 
> This seems surprising. Not sure if I understand the logic of the test.

The exact error doesn't matter in this test. We are looking at the exit code (1) of the command, and ensure that the step result is 'FAILURE' and description (state_string) is 'Failed to print configuration'. 

This code is being copied from https://github.com/WebKit/WebKit/blob/main/Tools/CISupport/ews-build/steps_unittest.py#L3709
At that time I would have copied this particular error log from some bot having issues.

Note that long-term we want to share buildbot configuration between ews and build.webkit.org, but that would be a separate effort.
Comment 9 EWS 2021-06-01 09:28:30 PDT
Committed r278310 (238347@main): <https://commits.webkit.org/238347@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 429922 [details].
Comment 10 Radar WebKit Bug Importer 2021-06-01 09:29:20 PDT
<rdar://problem/78718168>