WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
53220
back-during-onload-hung-page.php causes Chromium WebKit bot to fail
https://bugs.webkit.org/show_bug.cgi?id=53220
Summary
back-during-onload-hung-page.php causes Chromium WebKit bot to fail
Charles Reis
Reported
2011-01-26 19:49:06 PST
One of the Chromium WebKit bots is frequently failing because the PHP process for back-during-onload-hung-page.php doesn't exit for over 600 seconds. The tests that rely on this file are passing, but they just need it not to respond during the test. They would still pass if we reduced the timeout from 1000 seconds to something more reasonable, like 60. URL to the bot:
http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Win%20(deps
)
Attachments
Patch
(1.19 KB, patch)
2011-01-26 19:51 PST
,
Charles Reis
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Charles Reis
Comment 1
2011-01-26 19:51:24 PST
Created
attachment 80290
[details]
Patch
Eric Seidel (no email)
Comment 2
2011-01-26 22:33:53 PST
I take it we don't have code in run-webkit-tests/new-run-webkit-tests to kill run-away http servers (or in this case php processes?) Should we?
Charles Reis
Comment 3
2011-01-27 09:57:59 PST
(In reply to
comment #2
)
> I take it we don't have code in run-webkit-tests/new-run-webkit-tests to kill run-away http servers (or in this case php processes?) Should we?
I don't know much about those scripts, but they appear to try to kill the process but fail in this case. Here's one of the error messages from the bot, which looks like it tried and failed to kill the PHP process (according to Nicolas, who found out which process it was after the fact): command timed out: 600 seconds without output, killing pid 19192 SIGKILL failed to kill process using fake rc=-1 program finished with exit code -1 remoteFailed: [Failure instance: Traceback from remote host -- Traceback (most recent call last): Failure: buildbot.slave.commands.TimeoutError: SIGKILL failed to kill process ] (From:
http://build.chromium.org/p/chromium.webkit/builders/Webkit%20Win%20(deps)/builds/334/steps/webkit_tests/logs/stdio
) Maybe we just need a more effective approach for killing such processes? We only see the problem on one of the bots, so it might be specific to something in the config.
Ryosuke Niwa
Comment 4
2011-01-27 11:12:02 PST
Comment on
attachment 80290
[details]
Patch Clearing flags on attachment: 80290 Committed
r76816
: <
http://trac.webkit.org/changeset/76816
>
Ryosuke Niwa
Comment 5
2011-01-27 11:12:07 PST
All reviewed patches have been landed. Closing bug.
Tony Chang
Comment 6
2011-01-27 11:54:31 PST
(In reply to
comment #3
)
> (In reply to
comment #2
) > > I take it we don't have code in run-webkit-tests/new-run-webkit-tests to kill run-away http servers (or in this case php processes?) Should we? > > I don't know much about those scripts, but they appear to try to kill the process but fail in this case. Here's one of the error messages from the bot, which looks like it tried and failed to kill the PHP process (according to Nicolas, who found out which process it was after the fact): > > command timed out: 600 seconds without output, killing pid 19192 > SIGKILL failed to kill process > using fake rc=-1 > program finished with exit code -1
Maybe I misunderstand, but it looks like the buildbot process is trying to kill the NRWT process. Maybe it's failing to kill the NRWT because it's holding on to the lighttpd process which holds on to the php process? I bet it wouldn't be too hard to add a "kill all php.exe processes" when stopping the httpd, but it might cause collateral damage. I bet apache handles this better. Another reason to try and switch to apache (although I'm not volunteering). Nice detective work!
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug