[webkit-reviews] review denied: [Bug 216536] Add further temporary debug logging for wptserve : [Attachment 408838] Patch

[bugzilla-daemon at webkit.org]

Aakash Jain <aakash_jain at apple.com> has denied Sam Sneddon [:gsnedders]
<gsnedders at apple.com>'s request for review:
Bug 216536: Add further temporary debug logging for wptserve
https://bugs.webkit.org/show_bug.cgi?id=216536

Attachment 408838: Patch

https://bugs.webkit.org/attachment.cgi?id=408838&action=review




--- Comment #10 from Aakash Jain <aakash_jain at apple.com> ---
Comment on attachment 408838
  --> https://bugs.webkit.org/attachment.cgi?id=408838
Patch

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

>
LayoutTests/imported/w3c/web-platform-tests/tools/wptserve/wptserve/response.py
:257
> +		   self.logger.debug(subprocess.check_output(["ps", "aux"]))

This ('ps aux' and 'lsof -P +c0') will produce huge amount of output (tens of
thousands of lines). Are you sure you want to add that much output in the logs?
Maybe you just need a count of number of lines of these.

Also, when the machine is already running out of file handles, these commands
might just slow down/hang the machine further. This might explain why
https://ews-build.webkit.org/#/builders/30/builds/17561/steps/11/logs/stdio
hanged with this patch.