On https://build.webkit.org/builders/GTK-Linux-64-bit-Release-Perf/builds/10332/steps/benchmark-test/logs/stdio the benchmarks are all failing with: Traceback (most recent call last): File "/home/buildbot/worker/gtk-linux-64-release-perf-tests/build/Tools/Scripts/webkitpy/browserperfdash/browserperfdash_runner.py", line 169, in run runner.execute() File "/home/buildbot/worker/gtk-linux-64-release-perf-tests/build/Tools/Scripts/webkitpy/benchmark_runner/benchmark_runner.py", line 120, in execute self._run_benchmark(int(self._plan['count']), web_root) File "/home/buildbot/worker/gtk-linux-64-release-perf-tests/build/Tools/Scripts/webkitpy/benchmark_runner/benchmark_runner.py", line 90, in _run_benchmark result = self._run_one_test(web_root, self._plan['entry_point']) File "/home/buildbot/worker/gtk-linux-64-release-perf-tests/build/Tools/Scripts/webkitpy/benchmark_runner/webserver_benchmark_runner.py", line 43, in _run_one_test raise error AccessDenied: psutil.AccessDenied (pid=12593) This is caused because the http server launched via a subprocess exits as soon as is launched with an import error. So then the call to "psutil.Process(self._server_process.pid).connections()" tries to access the PID of a zombie in /proc causing the AccessDenied error. The http server exits with this error: Traceback (most recent call last): File "/home/clopez/webkit/webkit/Tools/Scripts/webkitpy/benchmark_runner/http_server_driver/http_server/twisted_http_server.py", line 17, in <module> from webkitpy.autoinstalled import twisted ImportError: No module named webkitpy.autoinstalled Which in turn is caused because Tools/Scripts is not on PYTHONPATH and it can't find webkitpy.autoinstalled
Created attachment 413390 [details] Patch
Created attachment 413395 [details] Patch v2: refactor the loop to avoid repeating the code twice
Created attachment 413396 [details] Patch cosmetic changes: remove empty line
Created attachment 413428 [details] Patch fix style errors
Comment on attachment 413428 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=413428&action=review > Tools/Scripts/webkitpy/benchmark_runner/http_server_driver/simple_http_server_driver.py:39 > + for attempt in range(max_attempt): While these changes aren't required to fix the regression, they do seem to be an improvement to readability.
(In reply to Jonathan Bedard from comment #5) > Comment on attachment 413428 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=413428&action=review > > > Tools/Scripts/webkitpy/benchmark_runner/http_server_driver/simple_http_server_driver.py:39 > > + for attempt in range(max_attempt): > > While these changes aren't required to fix the regression, they do seem to > be an improvement to readability. Right.. my first patch was not doing this changes.. but after I uploaded I looked at it and it looked ugly enough to have to repeat the code twice that I ended doing this small changes to try to improve the code quality Thanks for the review!
Comment on attachment 413428 [details] Patch Clearing flags on attachment: 413428 Committed r269514: <https://trac.webkit.org/changeset/269514>
All reviewed patches have been landed. Closing bug.
<rdar://problem/71121500>