Closed
Bug 1484401
Opened 8 years ago
Closed 8 years ago
[wpt-sync] Sync PR 12557 - [serve] Correct error detection and add more log messages
Categories
(Testing :: web-platform-tests, enhancement, P4)
Testing
web-platform-tests
Tracking
(firefox64 fixed)
RESOLVED
FIXED
mozilla64
| Tracking | Status | |
|---|---|---|
| firefox64 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 12557 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/12557
Details from upstream follow.
Mike Pennisi <mike@mikepennisi.com> wrote:
> [serve] Correct error detection and add more log messages
>
> A more advanced solution might involve attempting to re-start the failed
> subprocess(es). I think we'd need to understand the failure conditions before
> committing to something like that, though. Otherwise, we'll risk enabling a
> "continual restart" state if the problem can't be resolved by creating a new
> sub-process.
>
> I wrote the new test to a dedicated file because as an integration-level test,
> it's quite a bit slower than the existing tests. It's also a little messy
> because it stubs out a module method. I'm open to suggestions about better ways
> to test this!
>
>
| Assignee | ||
Comment 1•8 years ago
|
||
| Assignee | ||
Comment 2•8 years ago
|
||
| Assignee | ||
Comment 3•8 years ago
|
||
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
| Assignee | ||
Comment 4•8 years ago
|
||
| Assignee | ||
Comment 5•8 years ago
|
||
| Assignee | ||
Comment 6•8 years ago
|
||
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
| Assignee | ||
Comment 7•8 years ago
|
||
The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
| Assignee | ||
Comment 8•8 years ago
|
||
Result changes from PR not available.
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b57864454fb3
[wpt PR 12557] - [serve] Correct error detection and add more log messages, a=testonly
Comment 10•8 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•