Intermittent test_storage_queue.js | test failed Error: Queue is empty

RESOLVED FIXED

Status

defect
RESOLVED FIXED
7 years ago
8 months ago

People

(Reporter: anant, Assigned: gps)

Tracking

({intermittent-failure})

Details

(Whiteboard: [blocking-aitc+][qa-])

Attachments

(1 attachment)

Reporter

Description

7 years ago
The xpcshell test for AITC's storage-registry seems to be failing in a few environemnts. I've noticed 2 failures so far, one in Linux 64 opt (https://tbpl.mozilla.org/php/getParsedLog.php?id=12323978&tree=Services-Central#error0) and the other in OS 64 debug (https://tbpl.mozilla.org/php/getParsedLog.php?id=12323859&tree=Services-Central)

Investigate. Not sure if it's a 64 bit specific issue, or just an intermittent bug with an as-yet-unknown case. My hunch is that it might have something to do with the async write.
Assignee

Comment 1

7 years ago
I temporarily commented out this test because of intermittent oranges. I deemed this acceptable because the feature being tested is not enabled by default and leaving the test active would just burden people who watch the trees.

https://hg.mozilla.org/services/services-central/rev/a0b10aaad563
Whiteboard: [fixed in services][leave open]
Assignee

Comment 2

7 years ago
https://hg.mozilla.org/mozilla-central/rev/a0b10aaad563
Whiteboard: [fixed in services][leave open] → [leave open]
Whiteboard: [leave open] → [leave open] [blocking-aitc]
Whiteboard: [leave open] [blocking-aitc] → [leave open] [blocking-aitc+]
Reporter

Updated

7 years ago
OS: Linux → All
Hardware: x86 → All
Reporter

Comment 3

7 years ago
Bug 760896 (where this test was enabled again) landed and tbpl looks promising: https://tbpl.mozilla.org/?tree=Services-Central&rev=5099eb0d0287

We'll wait for a few more runs, but I suspect this has been fixed.
Reporter

Comment 4

7 years ago
Ok, bug 760896 merged to m-c, and there have been several pushes to m-i with no oranges for this test. Closing, we can re-open if we find an orange later on.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Reporter

Updated

7 years ago
Whiteboard: [leave open] [blocking-aitc+] → [blocking-aitc+]
Whiteboard: [blocking-aitc+] → [blocking-aitc+], [qa-]
Assignee

Comment 5

7 years ago
(In reply to Anant Narayanan [:anant] from comment #4)
> Ok, bug 760896 merged to m-c, and there have been several pushes to m-i with
> no oranges for this test. Closing, we can re-open if we find an orange later
> on.

Bug 760896 only recently landed in m-c. If you look at services-central, there were oranges after this patch landed. I think we still have this issue.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee

Updated

7 years ago
Whiteboard: [blocking-aitc+], [qa-] → [blocking-aitc+][qa-][orangefactor]
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Assignee

Comment 55

7 years ago
Feel free to punt to mconnor.
Assignee: nobody → gps
Status: REOPENED → ASSIGNED
Attachment #641117 - Flags: review?(rnewman)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment on attachment 641117 [details] [diff] [review]
Change bounds to prevent equivalence

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

::: services/aitc/tests/unit/test_storage_queue.js
@@ +53,5 @@
>    let items = [{test:"object"}, "teststring", 42];
>  
>    // Two random numbers: how many items to queue and how many to remove.
> +  let num = Math.floor(Math.random() * 100 + 20);
> +  let rem = Math.floor(Math.random() * 15 + 5);

Can we add an explanatory assertion or comment here that rem < num, and that we're doing it to avoid emptying the queue?
Attachment #641117 - Flags: review?(rnewman) → review+
Comment hidden (Legacy TBPL/Treeherder Robot)
Assignee

Comment 65

7 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/425a2086db6d

Included new comment and additional do_check tests for bounds sanity. Pushed to inbound so this orange can hopefully disappear sooner.
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
https://hg.mozilla.org/mozilla-central/rev/425a2086db6d
Status: ASSIGNED → RESOLVED
Closed: 7 years ago7 years ago
Resolution: --- → FIXED
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Comment hidden (Legacy TBPL/Treeherder Robot)
Blocks: 438871
Whiteboard: [blocking-aitc+][qa-][orangefactor] → [blocking-aitc+][qa-][orange]
Whiteboard: [blocking-aitc+][qa-][orange] → [blocking-aitc+][qa-]

Updated

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