android jsreftests should run in 3 chunks instead of 2

RESOLVED FIXED

Status

Release Engineering
General
RESOLVED FIXED
7 years ago
5 years ago

People

(Reporter: jmaher, Assigned: bear)

Tracking

({intermittent-failure})

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [mobile_unittests][android_tier_1])

Attachments

(1 attachment)

currently we run android jsreftests in 2 chunks.  It is hard to reproduce the problems, but a strong suspicion is that we are hitting memory limitations.

to do this we need to change our parameters to jsreftest from '--total-chunks=2 --this-chunk=1' to '--total-chunks=3 --this-chunk=1'.
(Assignee)

Updated

7 years ago
Depends on: 697473
(Assignee)

Updated

7 years ago
Blocks: 697473
No longer depends on: 697473
Bear, any chance you could look into this while you are looking at bug 696356.  We could possibly dup this bug to that one
(Assignee)

Updated

7 years ago
Assignee: nobody → bear
(Assignee)

Comment 2

7 years ago
Created attachment 570449 [details] [diff] [review]
add a third chunk to jsreftests and split crashtests into 2 chunks

lumped in bug 696356 since it's all buildbot-config changes

I need to find out where to make the tbpl change
Attachment #570449 - Flags: review?(aki)
Attachment #570449 - Flags: feedback?(jmaher)
Depends on: 698177

Comment 3

7 years ago
Comment on attachment 570449 [details] [diff] [review]
add a third chunk to jsreftests and split crashtests into 2 chunks

Yeah, trychooser and tbpl need updating too.
Attachment #570449 - Flags: review?(aki) → review+
(Assignee)

Comment 4

7 years ago
(In reply to Aki Sasaki [:aki] from comment #3)
> Comment on attachment 570449 [details] [diff] [review] [diff] [details] [review]
> add a third chunk to jsreftests and split crashtests into 2 chunks
> 
> Yeah, trychooser and tbpl need updating too.

philor said in IRC he would work up the tbpl change and I have a patch for trychooser over in bug 697473
Comment on attachment 570449 [details] [diff] [review]
add a third chunk to jsreftests and split crashtests into 2 chunks

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

it would be cool if we could have a master list of test suites that trychooser and tbpl would read from, then we don't have to file bugs for 3 different things!
Attachment #570449 - Flags: feedback?(jmaher) → feedback+
(Assignee)

Comment 6

7 years ago
Comment on attachment 570449 [details] [diff] [review]
add a third chunk to jsreftests and split crashtests into 2 chunks

committed changeset 4948:67dc88e2d6e2
Attachment #570449 - Flags: checked-in+
(Assignee)

Updated

7 years ago
Flags: needs-reconfig?
(Assignee)

Comment 7

7 years ago
reconfig landed and this is now live
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Sadly, http://build.mozilla.org/builds/builds-4hr.js.gz, and thus tbpl, still refuses to admit that crashtest-1, crashtest-2 or jsreftest-3 exist.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Going by the log for the job that inserts the finished jobs into the statusdb (test-master01:~cltbld/update.log) there have been no crashtest-[1,2] or jsreftest-3 inserted at all. Hence nothing in builds-4hr.js.gz.
this is running in staging for a last minute sanity check before rolling into production.
(Assignee)

Comment 11

7 years ago
joel - the new chunk setup went live earlier today.  the new issue is that for some reason the buildmaster isn't writing the proper files that are then picked up to build the tbpl-consumed data file.

working on that now
(Assignee)

Comment 12

7 years ago
forced a reconfig of test-master01 and it's now generated the data file that is collected to create builds-4hr.js.gz
Status: REOPENED → RESOLVED
Last Resolved: 7 years ago7 years ago
Resolution: --- → FIXED
(Assignee)

Updated

7 years ago
Flags: needs-reconfig?

Updated

6 years ago
Blocks: 438871
Keywords: intermittent-failure
Whiteboard: [orange][mobile_unittests][android_tier_1] → [mobile_unittests][android_tier_1]
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.