TEST-UNEXPECTED-FAIL | test_mozprocess_poll.ProcTestPoll.test_poll_while_running | AssertionError: Detected process is not running, process output: []

RESOLVED FIXED in Firefox 44

Status

Testing
Mozbase
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: ewong, Assigned: parkouss)

Tracking

unspecified
mozilla44
Points:
---

Firefox Tracking Flags

(firefox44 fixed)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

2 years ago
Getting this on SeaMonkey Linux32 comm-central build's make check:

TEST-UNEXPECTED-FAIL | test_mozprocess_poll.ProcTestPoll.test_poll_while_running | AssertionError: Detected process is not running, process output: []
Traceback (most recent call last):
  File "/builds/slave/c-cen-t-lnx/build/mozilla/testing/mozbase/mozprocess/tests/test_mozprocess_poll.py", line 42, in test_poll_while_running
    True)
  File "/builds/slave/c-cen-t-lnx/build/mozilla/testing/mozbase/mozprocess/tests/proctest.py", line 64, in determine_status
    self.assertTrue(detected, "Detected process is not running, process output: %s" % output)
TEST-INFO took 7ms
This actually happens in Firefox in automation too, but due to bug 1139911 it wasn't turning the builds orange.

Also test_mozprocess_wait.ProcTestWait.test_waittimeout has the same error:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c2df0cf0235c

I'll take a quick look.. if the solution isn't trivial, I vote we disable them for now so we can land bug 1139911 ASAP.
Blocks: 1139911
(Assignee)

Comment 2

2 years ago
Ok, did a quick fix, seems to work locally, so pushed to try:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=c91ee19e3759

That should probably be fixed in a better way - using the process id instead of parsing the command line of all processes to see if a process is running :( But this involve a bigger change (not much, but still) and anyway it was like this before we broke the tests.

Original issue comes from my patch in bug 1139487.
(Assignee)

Comment 3

2 years ago
Created attachment 8676382 [details]
MozReview Request: Bug 1215756 - TEST-UNEXPECTED-FAIL | test_mozprocess_poll.ProcTestPoll.test_poll_while_running | AssertionError: Detected process is not running, process output: []. r=ahal

Bug 1215756 - TEST-UNEXPECTED-FAIL | test_mozprocess_poll.ProcTestPoll.test_poll_while_running | AssertionError: Detected process is not running, process output: []. r=ahal
Attachment #8676382 - Flags: review?(ahalberstadt)
(Assignee)

Comment 4

2 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=32d8a57c0654
Comment on attachment 8676382 [details]
MozReview Request: Bug 1215756 - TEST-UNEXPECTED-FAIL | test_mozprocess_poll.ProcTestPoll.test_poll_while_running | AssertionError: Detected process is not running, process output: []. r=ahal

https://reviewboard.mozilla.org/r/22685/#review20147

Thanks for investigating, lgtm!
Attachment #8676382 - Flags: review?(ahalberstadt) → review+
(Assignee)

Comment 6

2 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=354e5272b836
(Assignee)

Comment 7

2 years ago
Created attachment 8676673 [details] [diff] [review]
1215756.patch

So, previous patch was not working for mac. Here is an improved version, which now check processes in a better and secure way.

Can't push that to try until the end of the day (CEST), feel free to do so if you want to.
Assignee: nobody → j.parkouss
Attachment #8676382 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8676673 - Flags: review?
(Assignee)

Updated

2 years ago
Attachment #8676673 - Flags: review? → review?(ahalberstadt)
Comment on attachment 8676673 [details] [diff] [review]
1215756.patch

Review of attachment 8676673 [details] [diff] [review]:
-----------------------------------------------------------------

Wow, nice cleanup! r+ assuming it works.. I'll push it to try.

::: testing/mozbase/mozprocess/tests/proctest.py
@@ +25,5 @@
> +        returncode = proc.proc.returncode
> +        didtimeout = proc.didTimeout
> +        detected = psutil.pid_exists(proc.pid)
> +        output = ''
> +        # ProcessHandler have output when store_output is set to True in the constructor

s/have/has
Attachment #8676673 - Flags: review?(ahalberstadt) → review+
Try run:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c75da30a8275
(Assignee)

Comment 10

2 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c68fa4bb22f5
(Assignee)

Comment 11

2 years ago
Created attachment 8677018 [details] [diff] [review]
fixed the nits

Ok, fixed the nit. (this was the patch on the last try).
Attachment #8676673 - Attachment is obsolete: true
Attachment #8677018 - Flags: review+
(Assignee)

Comment 12

2 years ago
So, green on try. try is closed, so asking for checkin-needed.

Thanks!
Keywords: checkin-needed

Comment 13

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/72bf30ffd771
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/72bf30ffd771
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox44: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in before you can comment on or make changes to this bug.