Bug 218643 – REGRESSION(r268930): It broke the http server of run-benchmark

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Carlos Alberto Lopez Perez]

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

[Carlos Alberto Lopez Perez]

Created attachment 413390 [details]
Patch

[Carlos Alberto Lopez Perez]

Created attachment 413395 [details]
Patch

v2: refactor the loop to avoid repeating the code twice

[Carlos Alberto Lopez Perez]

Created attachment 413396 [details]
Patch

cosmetic changes: remove empty line

[Carlos Alberto Lopez Perez]

Created attachment 413428 [details]
Patch

fix style errors

[Jonathan Bedard]

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.

[Carlos Alberto Lopez Perez]

(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!

[Carlos Alberto Lopez Perez]

Comment on attachment 413428 [details]
Patch

Clearing flags on attachment: 413428

Committed r269514: <https://trac.webkit.org/changeset/269514>

[Carlos Alberto Lopez Perez]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/71121500>