[Webkit-unassigned] [Bug 141191] New: [Win] Attempt to improve reliability of HTTP service

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=141191

            Bug ID: 141191
           Summary: [Win] Attempt to improve reliability of HTTP service
    Classification: Unclassified
           Product: WebKit
           Version: 528+ (Nightly build)
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Tools / Tests
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: bfulgham at webkit.org

The Windows test bots are periodically getting stuck in a mode where the httpd server is leaving a pidfile around. When this happens on Windows, the existing recovery code doesn't really work properly; I think this is primarily because webkitpy doesn't provide code to check for a running Windows PID, so the fallback code that tries to decide if a server is really running is just a no-op.

This patch does two things:

1. Instead of using the Cygwin /tmp folder to hold the PID file, it switches to using the XAMPP's standard file location. Since Apache is running as a Windows Service there is some confusing interactions with Windows vs. Cygwin file ownership. I am hoping that leaving this file where the XAMPP build (and Windows service) expects it to live.

2. Since the Windows webkitpy implementation doesn't actually check whether the PID is actually running, we skip the code path where the abandoned PID file is deleted. This patch makes sure we delete the abandoned file in the case where asking httpd to stop (which will do nothing if it wasn't running) doesn't cause the file to go away.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150203/36dee6b8/attachment-0002.html>