1276672 - browser_mozloop_telemetry.js creates a zombie process when running on windows7 VM

Status: RESOLVED FIXED

(Hello (Loop) :: Client, defect, P4)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

we are trying to move all our windows7 tests to run on a AWS VM instead of hardware in our datacenter.

This is going well, but a few tests are problematic.  In this case we get a zombiecheck failure in browser-chrome tests on opt tests (https://treeherder.mozilla.org/#/jobs?repo=try&revision=dfb24c916a3abf215ebc7de2e8c9d95e60648038) in this log file:
http://archive.mozilla.org/pub/firefox/try-builds/[e-mail]-dfb24c916a3abf215ebc7de2e8c9d95e60648038/try-win32/try_win7_vm_test-mochitest-browser-chrome-1-bm139-tests1-windows-build85.txt.gz

I did a bunch of try pushes to bisect this down and found browser_mozLoop_telemetry.js to be the root cause of these zombie process issues.

I plan to disable this test for win7_opt, with the goal of it becoming undisabled in the near future.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: temporarily disable test for zombieleak failures

Comment 2

[Ryan VanderMeulen [:RyanVM]]

Comment on attachment 8757942 [details] [diff] [review]
temporarily disable test for zombieleak failures

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

::: browser/extensions/loop/chrome/test/mochitest/browser.ini
@@ +17,5 @@
>  [browser_mozLoop_socialShare.js]
>  [browser_panel_privateBrowsing.js]
>  [browser_mozLoop_sharingListeners.js]
>  [browser_mozLoop_telemetry.js]
> +skip-if = os == win && !debug #  zombiecheck | child process 1228 still alive after shutdown (on win7-vm specifically)

Please add a bug number here.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b780d702673a
disable browser_mozLoop_telemetry.js on win7_opt for zombieleak failures. r=RyanVM

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b780d702673a

Comment 5

[Mark Banner (:standard8)]

Also landed the disabling in the loop repository:

https://github.com/mozilla/loop/commit/5ac0ae0edc258f1ea44860f00a47e6372718fa71

Comment 6

[Joel Maher ( :jmaher ) (UTC -8)]

this test case is removed

Comment 7

[Firefox Bug Husbandry Bot]

Removing leave-open keyword from resolved bugs, per :sylvestre.