Intermittent e10s browser_LoopRooms_channel.js | Test timed out

RESOLVED INCOMPLETE

Status

P2
normal
Rank:
26
RESOLVED INCOMPLETE
3 years ago
2 years ago

People

(Reporter: philor, Unassigned)

Tracking

(Blocks: 1 bug, {intermittent-failure})

unspecified
mozilla44
x86_64
Linux
intermittent-failure
Points:
2
Dependency tree / graph
Bug Flags:
firefox-backlog +
qe-verify -

Firefox Tracking Flags

(e10s+, firefox44 fixed)

Details

(Whiteboard: [todo-akita])

Attachments

(2 attachments)

Comment hidden (empty)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Reporter)

Updated

3 years ago
Keywords: leave-open
Whiteboard: [test disabled on ASan e10s]
Tentatively taking.
Assignee: nobody → standard8
Created attachment 8663732 [details] [diff] [review]
Fix intermittent browser_LoopRooms_channel.js time out, set up the back channel earlier to avoid loading time/setup issues.

This should fix it - the idea here is to set up the channel once and use it where necessary so that it always exists before we do the page load. Then we also check if the data has already been sent back either before or after we're ready for it.

Try seems to like this approach:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=81fe3a5c6c21
Attachment #8663732 - Flags: review?(mdeboer)
Comment on attachment 8663732 [details] [diff] [review]
Fix intermittent browser_LoopRooms_channel.js time out, set up the back channel earlier to avoid loading time/setup issues.

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

Thanks for fixing this!

::: browser/components/loop/test/mochitest/browser_LoopRooms_channel.js
@@ +23,5 @@
>  
>  var fakeRoomList = new Map([[ ROOM_TOKEN, { roomToken: ROOM_TOKEN } ]]);
>  
> +function BackChannel(uri) {
> +  this.init(uri);

You might as well put the `init` function here, instead of a function on the prototype.

@@ +44,5 @@
> +      this.receivedData = data;
> +    });
> +  },
> +
> +  teardown: function() {

nit: somehow I'm used to 'tearDown' instead of 'teardown' :)

@@ +49,5 @@
> +    this.channel.stopListening();
> +  }
> +};
> +
> +var goodBackChannel;

nit: please make these `gGoodBackChannel` and `gBadBackChannel`?

@@ +70,5 @@
>      url: uri.spec + "#" + hash
>    }, () => waitForChannelPromise);
>  }
>  
> +add_task(function* setup() {

You *could* make this an `add_test`, but that will require more boilerplate to actually run it, so... it's better this way :) You can add a `yield undefined` at the bottom if needed.
Attachment #8663732 - Flags: review?(mdeboer) → review+
Iteration: --- → 44.1 - Oct 5
Points: --- → 2
Keywords: leave-open
Rank: 25
Flags: qe-verify-
Flags: firefox-backlog+
Priority: -- → P2
https://hg.mozilla.org/mozilla-central/rev/1e82a58c1761
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox44: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
Comment hidden (Treeherder Robot)
(Reporter)

Updated

3 years ago
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Whiteboard: [test disabled on ASan e10s]
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
Created attachment 8667261 [details] [diff] [review]
Second attempt to fix intermittent failure in browser_LoopRooms_channel.js - ensure previous states are cleared when they are used.

I think this should be better - I realised that we weren't clearing out the old values of resolve/data from the channel after they were used.
Attachment #8667261 - Flags: review?(mdeboer)
Comment on attachment 8667261 [details] [diff] [review]
Second attempt to fix intermittent failure in browser_LoopRooms_channel.js - ensure previous states are cleared when they are used.

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

Aha! ok, worth a shot ;)
Attachment #8667261 - Flags: review?(mdeboer) → review+
https://hg.mozilla.org/mozilla-central/rev/ab2a3c581f58
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago3 years ago
Resolution: --- → FIXED
Comment hidden (Treeherder Robot)
Comment hidden (Treeherder Robot)
(Reporter)

Comment 22

3 years ago
We actually managed to hit it on Linux32 debug yesterday, the first non-ASan instance, so apparently the ASan connection is just that it's a race requiring a slow running build.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Summary: Intermittent browser_LoopRooms_channel.js | Test timed out → Intermittent e10s browser_LoopRooms_channel.js | Test timed out
8 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 7
* mozilla-central: 1

Platform breakdown:
* linux64: 7
* linux32: 1

For more details, see:
http://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2015-09-28&endday=2015-10-04&tree=all
Ok, I give up. I have no idea why this is still failing. I've tried ensuring the channel is definitely present before the page loads and that we reset things between tests.

Since I copied this from some of the existing web channel tests, which I assume aren't intermittent, I'll needinfo markh and see if he has any ideas.
Assignee: standard8 → nobody
Flags: needinfo?(markh)
Mark, do you want to back these patches out and mark them obsolete? Or do you still want them in m-c?
Flags: needinfo?(standard8)
(In reply to Liz Henry (:lizzard) (needinfo? me) from comment #25)
> Mark, do you want to back these patches out and mark them obsolete? Or do
> you still want them in m-c?

We still need them on m-c. I believe they are improvements on what was there, they just haven't 100% fixed the issue.
Flags: needinfo?(standard8)
7 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-aurora: 3
* mozilla-inbound: 2
* try: 1
* fx-team: 1

Platform breakdown:
* linux64: 7

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2015-10-05&endday=2015-10-11&tree=all
I'm afraid I can't see anything either :( The failure logs do seem to have "Error: operation not possible on dead CPOW" which I suspect is the root, but without a traceback it's difficult to see what might be causing it - a quick scan of the test and test .html implies it might well be in the webchannel itself - so I guess I'd be looking to instrument WebChannel.jsm at a few points to narrow this down - but the fact it only happened ~5 times in the last week makes even that trickier than it should be :(
Flags: needinfo?(markh)

Updated

3 years ago
Blocks: 984139
tracking-e10s: --- → +
6 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* fx-team: 3
* mozilla-inbound: 1
* mozilla-aurora: 1
* b2g-inbound: 1

Platform breakdown:
* linux64: 6

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2015-10-12&endday=2015-10-18&tree=all
Iteration: 44.1 - Oct 5 → ---
This hasn't been seen since December, so I guess something fixed it. Marking as fixed, as we did fix a few things here.
Assignee: nobody → standard8
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago3 years ago
Resolution: --- → FIXED
(Reporter)

Comment 31

3 years ago
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457 says three times this week.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Rank: 25 → 26
5 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* try: 2
* mozilla-inbound: 1
* mozilla-aurora: 1
* fx-team: 1

Platform breakdown:
* linux64: 5

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-02-08&endday=2016-02-14&tree=all
9 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* fx-team: 4
* try: 2
* mozilla-inbound: 2
* mozilla-central: 1

Platform breakdown:
* linux64: 9

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-02-29&endday=2016-03-06&tree=all
14 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 8
* fx-team: 4
* try: 1
* mozilla-beta: 1

Platform breakdown:
* linux64: 14

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-03-07&endday=2016-03-13&tree=all
6 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 4
* mozilla-beta: 1
* mozilla-aurora: 1

Platform breakdown:
* linux64: 6

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-03-14&endday=2016-03-20&tree=all
I'm not working on this at the moment.
Assignee: standard8 → nobody
5 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 2
* fx-team: 2
* mozilla-aurora: 1

Platform breakdown:
* linux64: 5

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-03-21&endday=2016-03-27&tree=all
10 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-aurora: 4
* mozilla-inbound: 3
* fx-team: 2
* try: 1

Platform breakdown:
* linux64: 10

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-03-28&endday=2016-04-03&tree=all
6 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 4
* mozilla-aurora: 1
* fx-team: 1

Platform breakdown:
* linux64: 6

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-04-04&endday=2016-04-10&tree=all
14 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 5
* mozilla-aurora: 5
* fx-team: 2
* mozilla-central: 1
* mozilla-beta: 1

Platform breakdown:
* linux64: 12
* linux32: 2

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-04-11&endday=2016-04-17&tree=all
Whiteboard: [akita-todo]
13 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-aurora: 5
* try: 3
* mozilla-inbound: 3
* mozilla-central: 1
* fx-team: 1

Platform breakdown:
* linux64: 8
* linux32: 5

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-04-18&endday=2016-04-24&tree=all
6 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 3
* try: 1
* mozilla-aurora: 1
* fx-team: 1

Platform breakdown:
* linux64: 6

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-05-10&endday=2016-05-16&tree=all
13 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 6
* fx-team: 5
* try: 1
* mozilla-central: 1

Platform breakdown:
* linux64: 13

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-05-16&endday=2016-05-22&tree=all
Whiteboard: [akita-todo] → [todo-akita]
17 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 9
* mozilla-aurora: 2
* fx-team: 2
* try: 1
* mozilla-central: 1
* mozilla-beta: 1
* ash: 1

Platform breakdown:
* linux64: 17

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-05-23&endday=2016-05-29&tree=all
15 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 8
* mozilla-aurora: 3
* mozilla-central: 2
* fx-team: 2

Platform breakdown:
* linux64: 14
* linux32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-05-30&endday=2016-06-05&tree=all
7 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 4
* fx-team: 3

Platform breakdown:
* linux64: 7

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-06-06&endday=2016-06-12&tree=all
5 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 4
* mozilla-central: 1

Platform breakdown:
* linux64: 5

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-06-13&endday=2016-06-19&tree=all
14 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 4
* mozilla-beta: 4
* try: 2
* fx-team: 2
* mozilla-central: 1
* mozilla-aurora: 1

Platform breakdown:
* linux64: 10
* linux32: 4

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-06-20&endday=2016-06-26&tree=all
19 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 7
* autoland: 5
* mozilla-aurora: 3
* fx-team: 2
* mozilla-release: 1
* ash: 1

Platform breakdown:
* linux64: 16
* linux32: 3

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-06-27&endday=2016-07-03&tree=all
18 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* autoland: 9
* mozilla-inbound: 7
* mozilla-central: 1
* mozilla-aurora: 1

Platform breakdown:
* linux64: 18

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-07-04&endday=2016-07-10&tree=all
22 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 6
* autoland: 5
* fx-team: 4
* try: 2
* mozilla-central: 2
* mozilla-beta: 2
* mozilla-aurora: 1

Platform breakdown:
* linux64: 22

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-07-11&endday=2016-07-17&tree=all
20 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 9
* autoland: 7
* fx-team: 3
* mozilla-central: 1

Platform breakdown:
* linux64: 19
* linux32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-07-18&endday=2016-07-24&tree=all
7 automation job failures were associated with this bug in the last 7 days.

Repository breakdown:
* mozilla-inbound: 3
* autoland: 2
* mozilla-central: 1
* mozilla-aurora: 1

Platform breakdown:
* linux64: 6
* linux32: 1

For more details, see:
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1206457&startday=2016-07-25&endday=2016-07-31&tree=all
Support for Hello/Loop has been discontinued.

https://support.mozilla.org/kb/hello-status

Hence closing the old bugs. Thank you for your support.
Status: REOPENED → RESOLVED
Last Resolved: 3 years ago2 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.