Closed
Bug 1263149
Opened 9 years ago
Closed 9 years ago
Add mulet debug builds on pine
Categories
(Firefox OS Graveyard :: Runtime, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gerard-majax, Assigned: gerard-majax)
References
Details
(Whiteboard: fixed-in-pine)
Attachments
(2 files, 3 obsolete files)
(deleted),
application/json
|
Details | |
(deleted),
patch
|
gerard-majax
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•9 years ago
|
||
Taskcluster graph JSON file with the patches, as much as I can say it includes both opt and dbg builds.
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8739420 -
Flags: review?(garndt)
Comment 3•9 years ago
|
||
Comment on attachment 8739420 [details] [diff] [review]
Add Mulet Linux debug builds to pine
Review of attachment 8739420 [details] [diff] [review]:
-----------------------------------------------------------------
Looking good, but I think there are a few things that need to be fixed before this is pushed. Probably because of the changes to the mozconfig for this build, you might want to push to try to see what that build looks like for you. Because this was also added to the try flags, you don't need to transplant any changes you made for pine into the try job_flags...it should already work.
::: b2g/dev/config/mozconfigs/linux64/mulet_dbg
@@ +1,1 @@
> +MOZ_AUTOMATION_L10N_CHECK=0
I'm not an expert in mozconfigs, so you might want to have someone else review this piece to ensure it's doing what you want it to.
::: testing/taskcluster/tasks/branches/base_jobs.yml
@@ +11,5 @@
> - Mulet Linux
> types:
> opt:
> task: tasks/builds/mulet_linux.yml
> + debug:
Adding it here will make it scheduled on all pushes to branches inheriting this file, if that's the intent.
::: testing/taskcluster/tasks/branches/try/job_flags.yml
@@ +44,5 @@
> opt:
> task: tasks/builds/opt_linux64.yml
> debug:
> task: tasks/builds/dbg_linux64.yml
> + linux64-mulet:
Because this file inherits from base/job_flags.yml, and you added the debug build there, I do not think you need to add any changes here.
::: testing/taskcluster/tasks/builds/mulet_linux_dbg.yml
@@ +10,5 @@
> +
> + workerType: mulet-debug
> +
> + routes:
> + - 'index.buildbot.branches.{{project}}.linux64-mulet'
I'm not even sure if these buildbot index routes need to be here anymore, but other tasks have it so I suppose this isn't the right time to fix that.
I would just check to make sure mulet-dbg is getting indexed the way you would want it to. Looking over the attached graph json it appears that it should be indexed correctly.
@@ +20,5 @@
> + - 'docker-worker:relengapi-proxy:tooltool.download.public'
> +
> + payload:
> + cache:
> + level-{{level}}-{{project}}-build-mulet-dbg-linux-workspace: '/home/worker/workspace'
Cache name here is:
level-{{level}}-{{project}}-build-mulet-dbg-linux-workspace
But the scope here is:
level-{{level}}-{{project}}-build-mulet-linux-workspace' <--- missing dbg
I'm not exactly sure about he build specifics if a separate cache is necessary for debug and opt builds. I know we've done that in the past, but I noticed that the linux desktop builds do not separate them out. Just something I noticed.
The cache name and the name in the scope must match, no matter what's decided about splitting debug and opt out.
@@ +44,5 @@
> + extra:
> + treeherderEnv:
> + - production
> + - staging
> + treeherder:
You'll need to add task.extra.treeherder.collection.debug = true here for it to show up as a debug build
Similar to this:
https://dxr.mozilla.org/mozilla-central/source/testing/taskcluster/tasks/builds/dbg_linux64.yml#26
Attachment #8739420 -
Flags: review?(garndt) → review-
Assignee | ||
Comment 4•9 years ago
|
||
Attachment #8739483 -
Flags: review?(garndt)
Comment 5•9 years ago
|
||
Comment on attachment 8739483 [details] [diff] [review]
Add Mulet Linux debug builds to pine
Review of attachment 8739483 [details] [diff] [review]:
-----------------------------------------------------------------
::: testing/taskcluster/tasks/branches/try/job_flags.yml
@@ +48,5 @@
> + linux64-mulet:
> + platforms:
> + - Mulet Linux
> + types:
> + opt:
So in the base/job_flags.yml that this file inherits from, mulet opt is already defined:
https://dxr.mozilla.org/mozilla-central/source/testing/taskcluster/tasks/branches/base_jobs.yml#13
Do we want to enable debug only for try then and that's why we're not adding debug to the base/job_flags.yml?
Assignee | ||
Comment 6•9 years ago
|
||
Additions to try are for being able to push to try from pine. The job is also defined on pine/job_flags, so it should run for any job on pine, right?
Assignee | ||
Updated•9 years ago
|
Attachment #8739420 -
Attachment is obsolete: true
Comment 7•9 years ago
|
||
Yea if it's in pine/job_flags then it should run
Comment 8•9 years ago
|
||
Comment on attachment 8739483 [details] [diff] [review]
Add Mulet Linux debug builds to pine
Review of attachment 8739483 [details] [diff] [review]:
-----------------------------------------------------------------
This looks good to me and should work. I understand if you want to hold off on landing this until you're able to push to try again and verify it behaves how you would like it to.
Attachment #8739483 -
Flags: review?(garndt) → review+
Comment 9•9 years ago
|
||
Assignee | ||
Comment 10•9 years ago
|
||
Attachment #8739483 -
Attachment is obsolete: true
Attachment #8739537 -
Flags: review+
Assignee | ||
Comment 11•9 years ago
|
||
Everything looks good, removing "tier" definition since it seems inconsistent
with the rest of the use (mulet opt has no such field and yet it is tier 3).
Attachment #8739537 -
Attachment is obsolete: true
Attachment #8739563 -
Flags: review+
Assignee | ||
Comment 12•9 years ago
|
||
triggering properly and treeherder filtering has been updated to add mulet debug as tier 3: https://treeherder.mozilla.org/#/jobs?repo=try&revision=eafe7ffebff0&filter-tier=1&filter-tier=2&filter-tier=3
Assignee | ||
Comment 13•9 years ago
|
||
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: fixed-in-pine
You need to log in
before you can comment on or make changes to this bug.
Description
•