Intermittent browser_rules_edit-property-increments.js | This test exceeded the timeout threshold. It should be rewritten or split up.

RESOLVED FIXED in Firefox 52

Status

P1
normal
RESOLVED FIXED
3 years ago
8 months ago

People

(Reporter: philor, Assigned: ochameau)

Tracking

({intermittent-failure})

unspecified
Firefox 53
intermittent-failure
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(firefox51 unaffected, firefox52 fixed, firefox53 fixed)

Details

Attachments

(1 attachment)

Inspector bug triage (filter on CLIMBING SHOES)
Priority: -- → P3
Comment hidden (Intermittent Failures Robot)
Inspector bug triage (filter on CLIMBING SHOES).
Priority: P3 → P2
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
Comment hidden (Intermittent Failures Robot)
:pbro, this bug really spiked on November 14th (maybe the 13th based on how you read the data and comment dates)- can you take a look at this? It is one of the top issues remaining on the trees.

this is 97% on linux32 (the win7 stuff could be mis filed) and it is a timeout- I am not sure if we get a lot of value out of devtools on linux32, so possibly a disabling is easier than a longer investigation.
Flags: needinfo?(pbrosset)
Comment hidden (Intermittent Failures Robot)
(In reply to Joel Maher ( :jmaher) from comment #14)
> :pbro, this bug really spiked on November 14th (maybe the 13th based on how
> you read the data and comment dates)- can you take a look at this? It is one
> of the top issues remaining on the trees.
> 
> this is 97% on linux32 (the win7 stuff could be mis filed) and it is a
> timeout- I am not sure if we get a lot of value out of devtools on linux32,
> so possibly a disabling is easier than a longer investigation.
Thanks Joel. This one is another case of a test that passes but started to take a few more seconds than the allowed threshold recently, for some obscure reasons.
You're right that we could disable it (or maybe just increase the timeout), but looking at the test, it seems to be one that can be very easily split up in several parts. It seems to be really long and composed of fairly independent parts. So I'd like us to do this instead. If after this, some parts keep on running for too long, we can then start disabling on linux debug 32 as suggested.

Alex: do you have some time to look into this simple test splitting bug? r=test-only would be fine for something like this.
Flags: needinfo?(pbrosset) → needinfo?(poirot.alex)
Priority: P2 → P1
Comment hidden (Intermittent Failures Robot)
(Assignee)

Comment 18

2 years ago
Do we ever had a linux32-only bug to fix? Or may be it helped us fix some real world races?
If not, is there value in running linux32 devtools at all?
This test is all about increments, if we split it, it would be quite arbitrary of split in many this time very small piece. I would be more in favor of incresing the timeout while looking at possible performance tweaks.

What is this timeout? 30s? It runs in 10s locally.

Also, the rule view throws some errors to the console.
This is related to: (from bug 1069829):
https://hg.mozilla.org/mozilla-central/rev/99fbe8621fa7
I imagine it can slow the test down. This test runs 20-30% faster by reverting that.
But that's not what recently regressed as it landed long time ago.
Comment hidden (mozreview-request)
(Assignee)

Comment 20

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/b2a7cf1f8f467495d0a3d1f06935ee76d147b466
Bug 1275446 - Bump timeout threshold for browser_rules_edit-property-increments.js on linux32. r=test-only
(In reply to Alexandre Poirot [:ochameau] from comment #18)
> 
> What is this timeout? 30s? It runs in 10s locally.

The timeout is 45 seconds (http://searchfox.org/mozilla-central/rev/b4e6fa3527436bdbcd9dd2e1982382fe423431f3/testing/mochitest/browser-test.js#3)
Assignee: nobody → poirot.alex
Status: NEW → ASSIGNED

Comment 22

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b2a7cf1f8f46
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 53
Comment hidden (Intermittent Failures Robot)
(Assignee)

Updated

2 years ago
Flags: needinfo?(poirot.alex)
status-firefox51: --- → unaffected
status-firefox52: --- → affected

Comment 24

2 years ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/761f6ef1cf34
status-firefox52: affected → fixed
Flags: in-testsuite-

Updated

8 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.