Closed
Bug 1175934
Opened 9 years ago
Closed 9 years ago
[B2G] Add support to build blobfree images
Categories
(Release Engineering :: Applications: MozharnessCore, defect, P2)
Release Engineering
Applications: MozharnessCore
ARM
Gonk (Firefox OS)
Tracking
(firefox42 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox42 | --- | fixed |
People
(Reporter: wcosta, Assigned: wcosta)
References
Details
Attachments
(7 files, 7 obsolete files)
40 bytes,
text/x-review-board-request
|
garndt
:
review+
|
Details |
40 bytes,
text/x-review-board-request
|
jlund
:
review+
|
Details |
501 bytes,
patch
|
garndt
:
review+
|
Details | Diff | Splinter Review |
5.83 MB,
text/x-log
|
Details | |
3.08 KB,
patch
|
jlund
:
review+
|
Details | Diff | Splinter Review |
7.85 KB,
patch
|
jlund
:
review+
garndt
:
review+
|
Details | Diff | Splinter Review |
1.86 KB,
patch
|
garndt
:
review+
|
Details | Diff | Splinter Review |
B2G can create blob free images, through the command "./build.sh blobfree"
Assignee | ||
Updated•9 years ago
|
OS: Unspecified → Gonk (Firefox OS)
Priority: -- → P2
Hardware: Unspecified → ARM
Flags: needinfo?(nhirata.bugzilla)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → wcosta
Assignee | ||
Updated•9 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•9 years ago
|
||
https://tools.taskcluster.net/task-inspector/#pMUiijd5ROmPOLSMH2woiw/ https://tools.taskcluster.net/task-inspector/#D3DRXM4_SFej--ONZ0DZPw/ https://tools.taskcluster.net/task-inspector/#seBBa_-VTVWNHhpyB6nqew/ https://tools.taskcluster.net/task-inspector/#YPpjZqx6QjmWeYlmQyFfpw/ https://tools.taskcluster.net/task-inspector/#X0K833ZXQLaFI95uEf-K1Q/ https://tools.taskcluster.net/task-inspector/#KUbzqRgaRCGgaMwI2IinWw/
Assignee | ||
Comment 2•9 years ago
|
||
Bug 1175934: Add blob free build for flame and spark devices. r=garndt
Attachment #8627521 -
Flags: review?(garndt)
Assignee | ||
Comment 3•9 years ago
|
||
I cannot run task-graph to completely test this patch due to Bug 1177183, however.
Comment 4•9 years ago
|
||
Comment on attachment 8627521 [details] MozReview Request: Bug 1175934 part 1: Copy blobfree zip to public upload dir. r?garndt https://reviewboard.mozilla.org/r/12259/#review10755 ::: testing/taskcluster/tasks/builds/b2g_aries_spark_debug_blobfree.yml:11 (Diff revision 1) > + name: '[TC] B2G Aries Debug (blobfree)' I wasn't sure anymore, but do we need [TC] in these names still? lightsofapollo might know and I'm curious to understand what we should be doing here. ::: testing/taskcluster/tasks/builds/b2g_aries_spark_debug_blobfree.yml:33 (Diff revision 1) > + symbol: B We should clarify with the teams what the group symbol and symbol should be for all of these. As it stands now, these share the same symbols as blob builds. So on a push to m-c, this would appear on the device image debug line in treeherder: Aries( B B ) where one B is for the blob free build and the other with blobs. Could get confusing when trying to find specific jobs. Gets even worse if any of them are retriggered. ::: testing/taskcluster/tasks/builds/b2g_aries_spark_eng_blobfree.yml:39 (Diff revision 1) > + img: 'public/build/aries.zip' Would the validate script break when processing this? https://dxr.mozilla.org/mozilla-central/source/testing/docker/phone-builder/bin/validate_task.py#42 ::: testing/taskcluster/tasks/builds/b2g_aries_spark_debug_blobfree.yml:34 (Diff revision 1) > + groupSymbol: Aries I'm not sure wwhat the cleaner, straight forward route is for this, but either the symbols could indicate if they are blob free or not, or the group symbol could be difference. Maybe Aries() and Aries-Blob-Free() or something. I'm not sure, I would defer to others for their opinion on what they want to see on TH. ::: testing/taskcluster/tasks/builds/b2g_aries_spark_debug_blobfree.yml:30 (Diff revision 1) > + - production Since these are newer builds and we're not sure how well it will go, do we want to enable these on production right from the start?
Attachment #8627521 -
Flags: review?(garndt)
Assignee | ||
Comment 5•9 years ago
|
||
Bug 1175934: Add blob free build configs. r=garndt The buildfree target builds phone images without blobs, allowing them to be freely available.
Attachment #8628506 -
Flags: review?(garndt)
Assignee | ||
Comment 6•9 years ago
|
||
Comment on attachment 8627521 [details] MozReview Request: Bug 1175934 part 1: Copy blobfree zip to public upload dir. r?garndt Bug 1175934 part 1: Add blob free build for flame and spark devices. r=garndt
Attachment #8627521 -
Attachment description: MozReview Request: Bug 1175934: Add blob free build for flame and spark devices. r=garndt → MozReview Request: Bug 1175934 part 1: Add blob free build for flame and spark devices. r=garndt
Attachment #8627521 -
Flags: review?(garndt)
Assignee | ||
Comment 7•9 years ago
|
||
Bug 1175934 part 2: Allow blob free images go on public. r=garndt Die, blobs! Die!
Attachment #8628526 -
Flags: review?(garndt)
Assignee | ||
Comment 8•9 years ago
|
||
https://tools.taskcluster.net/task-inspector/#979cqKtyRkK0dOCCMthwnQ/ https://tools.taskcluster.net/task-inspector/#pAwXY-zZRSm0ve4PAoZw0g/ https://tools.taskcluster.net/task-inspector/#uuDSlbkvRyaYrZfqYtc3TA/ https://tools.taskcluster.net/task-inspector/#l2ZPTSUbTsy-Oxa9lmRMwg/ https://tools.taskcluster.net/task-inspector/#BFVdNWt7T2GzyCRpvYW3BQ/ https://tools.taskcluster.net/task-inspector/#EiX-Wci0REKF3jexAYn2QA/
Comment 9•9 years ago
|
||
I don't see the |build.sh blobfree| statement in the logs of those runs :(
Assignee | ||
Comment 10•9 years ago
|
||
(In reply to Alexandre LISSY :gerard-majax from comment #9) > I don't see the |build.sh blobfree| statement in the logs of those runs :( Indeed there was a mistake in the mozharness patch, I am testing the fix, thanks. I wonder why I didn't get a full flash on aries while testing...
Comment 11•9 years ago
|
||
Comment on attachment 8627521 [details] MozReview Request: Bug 1175934 part 1: Copy blobfree zip to public upload dir. r?garndt https://reviewboard.mozilla.org/r/12259/#review10929 Ship It!
Attachment #8627521 -
Flags: review?(garndt) → review+
Comment 12•9 years ago
|
||
Comment on attachment 8628526 [details] MozReview Request: Bug 1175934 part 2: Support blobfree images. r?garndt https://reviewboard.mozilla.org/r/12395/#review10931 Ship It!
Attachment #8628526 -
Flags: review?(garndt) → review+
Comment 13•9 years ago
|
||
https://reviewboard.mozilla.org/r/12391/#review10935 I gave this a quick look over and it seems ok, but I'm no MH config expert.
Comment 14•9 years ago
|
||
Comment on attachment 8628506 [details] MozReview Request: Bug 1175934 part 1: Allow build target configuration from config file. r=jlund https://reviewboard.mozilla.org/r/12393/#review10937 Ship It!
Attachment #8628506 -
Flags: review?(garndt) → review+
Assignee | ||
Comment 15•9 years ago
|
||
https://tools.taskcluster.net/task-inspector/#zOkjQ-6-TKWl7PD8JQ8nSw/1 https://tools.taskcluster.net/task-inspector/#jt5d5ViARV6o9bjg7MNSpw/1 https://tools.taskcluster.net/task-inspector/#3GR-Su5hRbG1yc1cfZdMmQ/1 https://tools.taskcluster.net/task-inspector/#aBjfl3-SRtuoGTSOpQi9gQ/1 https://tools.taskcluster.net/task-inspector/#ZSQverbWSYqNnmWRKq5FVA/1 https://tools.taskcluster.net/task-inspector/#TpiVr7__SFm7asnQI-iDaw/1
Assignee | ||
Updated•9 years ago
|
Attachment #8628506 -
Attachment description: MozReview Request: Bug 1175934: Add blob free build configs. r=garndt → MozReview Request: Bug 1175934 part 1: Allow build target configuration from config file. r=jlund
Attachment #8628506 -
Flags: review+ → review?(jlund)
Assignee | ||
Comment 16•9 years ago
|
||
Comment on attachment 8628506 [details] MozReview Request: Bug 1175934 part 1: Allow build target configuration from config file. r=jlund Bug 1175934 part 1: Allow build target configuration from config file. r=jlund For now, we are going to provide builds with and without blobs. Instead of going into the pain of creating new devices targets just to add blobfree in config.json, we allow configuration of build targets in the mozharness config file.
Assignee | ||
Comment 17•9 years ago
|
||
Bug 1175934 part 2: Add blob free build configs. The buildfree target builds phone images without blobs, allowing them to be freely available.
Attachment #8629078 -
Flags: review?(garndt)
Comment 18•9 years ago
|
||
Comment on attachment 8629078 [details] MozReview Request: Bug 1175934 part 2: Add blob free build configs. https://reviewboard.mozilla.org/r/12533/#review11017 Ship It!
Attachment #8629078 -
Flags: review?(garndt) → review+
Comment 19•9 years ago
|
||
Comment on attachment 8628506 [details] MozReview Request: Bug 1175934 part 1: Allow build target configuration from config file. r=jlund https://reviewboard.mozilla.org/r/12393/#review11087 Ship It!
Attachment #8628506 -
Flags: review?(jlund) → review+
Assignee | ||
Comment 20•9 years ago
|
||
https://hg.mozilla.org/build/mozharness/rev/1d9d27ca8b31 https://hg.mozilla.org/build/mozharness/rev/0e613ece6d93
Comment 21•9 years ago
|
||
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/production
Comment 22•9 years ago
|
||
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/production
Assignee | ||
Comment 23•9 years ago
|
||
url: https://hg.mozilla.org/integration/b2g-inbound/rev/5b4aed5949f1e9135b07f2efc82261ee9469c3a9 changeset: 5b4aed5949f1e9135b07f2efc82261ee9469c3a9 user: Wander Lairson Costa <wcosta@mozilla.com> date: Mon Jul 13 07:55:22 2015 -0300 description: Bug 1175934 part 1: Add blob free build for flame and spark devices. r=garndt url: https://hg.mozilla.org/integration/b2g-inbound/rev/83348c0ea47bcc4e0d91150c2cb45c295bcaca50 changeset: 83348c0ea47bcc4e0d91150c2cb45c295bcaca50 user: Wander Lairson Costa <wcosta@mozilla.com> date: Mon Jul 13 07:55:22 2015 -0300 description: Bug 1175934 part 2: Allow blob free images go on public. r=garndt Die, blobs! Die!
Assignee | ||
Comment 24•9 years ago
|
||
Originally the part 2 contained the version update to 0.0.l7, but after rebase, it was removed because another commit already contained it. This late patch upgrade de docker image version.
Attachment #8632768 -
Flags: review?(garndt)
Updated•9 years ago
|
Attachment #8632768 -
Flags: review?(garndt) → review+
Assignee | ||
Comment 25•9 years ago
|
||
url: https://hg.mozilla.org/integration/b2g-inbound/rev/b7d35cc9ea8c30cba1bb78df0c6a078715841ecc changeset: b7d35cc9ea8c30cba1bb78df0c6a078715841ecc user: Wander Lairson Costa <wcosta@mozilla.com> date: Mon Jul 13 10:21:02 2015 -0300 description: Bug 1175934 part 3: Update phone-builder version. r=garndt
https://hg.mozilla.org/mozilla-central/rev/5b4aed5949f1 https://hg.mozilla.org/mozilla-central/rev/83348c0ea47b https://hg.mozilla.org/mozilla-central/rev/b7d35cc9ea8c
Flags: needinfo?(nhirata.bugzilla)
Comment 27•9 years ago
|
||
Sorry to ask dumb question, but I still don't see blobfree dist zip file on recent tasks, like https://tools.taskcluster.net/task-inspector/#PAUBPHW2QXqQjkyrmjX_aA/0
Flags: needinfo?(wcosta)
Assignee | ||
Comment 28•9 years ago
|
||
(In reply to Alexandre LISSY :gerard-majax from comment #27) > Sorry to ask dumb question, but I still don't see blobfree dist zip file on > recent tasks, like > https://tools.taskcluster.net/task-inspector/#PAUBPHW2QXqQjkyrmjX_aA/0 It is running "./build.sh -j8 blobfree". Is there any additional step besides that to generate blob free?
Flags: needinfo?(wcosta) → needinfo?(lissyx+mozillians)
Comment 29•9 years ago
|
||
(In reply to Wander Lairson Costa [:wcosta] from comment #28) > (In reply to Alexandre LISSY :gerard-majax from comment #27) > > Sorry to ask dumb question, but I still don't see blobfree dist zip file on > > recent tasks, like > > https://tools.taskcluster.net/task-inspector/#PAUBPHW2QXqQjkyrmjX_aA/0 > > It is running "./build.sh -j8 blobfree". Is there any additional step > besides that to generate blob free? No but the file we want to make easily and publicly accessible is <device>.blobfree-dist.zip
Flags: needinfo?(lissyx+mozillians)
Assignee | ||
Updated•9 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 30•9 years ago
|
||
url: https://hg.mozilla.org/integration/b2g-inbound/rev/e810d4d7ad99942b31dbe68c38f78c98f09e4172 changeset: e810d4d7ad99942b31dbe68c38f78c98f09e4172 user: Wander Lairson Costa <wcosta@mozilla.com> date: Mon Jul 20 10:50:23 2015 -0300 description: Bug 1175934: backout revision b7d35cc9ea8c. No need to create new builds. r=me url: https://hg.mozilla.org/integration/b2g-inbound/rev/47d33944a71962482a7d4c89cc419b0ac84f29a4 changeset: 47d33944a71962482a7d4c89cc419b0ac84f29a4 user: Wander Lairson Costa <wcosta@mozilla.com> date: Mon Jul 20 11:01:24 2015 -0300 description: Bug 1175934: backout revision 83348c0ea47b. No need to create new builds. r=me url: https://hg.mozilla.org/integration/b2g-inbound/rev/f7d9b77a7afb616b1a7cce12185256da99cba9be changeset: f7d9b77a7afb616b1a7cce12185256da99cba9be user: Wander Lairson Costa <wcosta@mozilla.com> date: Mon Jul 20 11:29:13 2015 -0300 description: Bug 1175934: backout revision 5b4aed5949f1. No need to create new builds. r=me
Assignee | ||
Comment 31•9 years ago
|
||
https://tools.taskcluster.net/task-inspector/#gn_54CSGQL-n6Ht7PHtmWA/ https://tools.taskcluster.net/task-inspector/#buxaKigKR5CHW-HAGanx3A/ https://tools.taskcluster.net/task-inspector/#AGKxoOGPSiW5EhIQLVw16g/ https://tools.taskcluster.net/task-inspector/#RiRwiOqQQ6CPDaHpAMy6tA/ https://tools.taskcluster.net/task-inspector/#lAtt-JkXTGWlT_TRus_dtw/ https://tools.taskcluster.net/task-inspector/#nMS4ZrAmSq27t8LybY8bYA/ https://tools.taskcluster.net/task-inspector/#i7nAwjdhTSifwnJQmjUk_w/ https://tools.taskcluster.net/task-inspector/#gNRqGnoVTvygLHYSzGLsGg/ https://tools.taskcluster.net/task-inspector/#B48dKjokTrKhq02Uqwq9hg/ https://tools.taskcluster.net/task-inspector/#07isnvmHS06WwE6MIKsNMQ/ https://tools.taskcluster.net/task-inspector/#fSG3li4WQoyluLfKEsicBA/
Comment 32•9 years ago
|
||
(In reply to Wander Lairson Costa [:wcosta] from comment #31) > https://tools.taskcluster.net/task-inspector/#gn_54CSGQL-n6Ht7PHtmWA/ > https://tools.taskcluster.net/task-inspector/#buxaKigKR5CHW-HAGanx3A/ > https://tools.taskcluster.net/task-inspector/#AGKxoOGPSiW5EhIQLVw16g/ > https://tools.taskcluster.net/task-inspector/#RiRwiOqQQ6CPDaHpAMy6tA/ > https://tools.taskcluster.net/task-inspector/#lAtt-JkXTGWlT_TRus_dtw/ > https://tools.taskcluster.net/task-inspector/#nMS4ZrAmSq27t8LybY8bYA/ > https://tools.taskcluster.net/task-inspector/#i7nAwjdhTSifwnJQmjUk_w/ > https://tools.taskcluster.net/task-inspector/#gNRqGnoVTvygLHYSzGLsGg/ > https://tools.taskcluster.net/task-inspector/#B48dKjokTrKhq02Uqwq9hg/ > https://tools.taskcluster.net/task-inspector/#07isnvmHS06WwE6MIKsNMQ/ > https://tools.taskcluster.net/task-inspector/#fSG3li4WQoyluLfKEsicBA/ So it looks like those actually do contains blobs. Sorry about this, but the proper command should be: |./build.sh && ./build.sh blobfree|, without any -j involved.
Comment 33•9 years ago
|
||
That is the expected output.
Assignee | ||
Comment 34•9 years ago
|
||
https://tools.taskcluster.net/task-inspector/#YnehR_V6Q3WGrsJt9SBBqA/ https://tools.taskcluster.net/task-inspector/#YnehR_V6Q3WGrsJt9SBBqA/ https://tools.taskcluster.net/task-inspector/#gm3YpX6iQ3O9GmC3oGxKIw/ https://tools.taskcluster.net/task-inspector/#SRqdbQ1JR7uIXNFo-6Wlzw/ https://tools.taskcluster.net/task-inspector/#NlDeE_4vS0emXR9ZFN43ug/ https://tools.taskcluster.net/task-inspector/#cdLZUjC1QxeFhM_tukh6hQ/ https://tools.taskcluster.net/task-inspector/#L-M8OxcVQ1ueqW0k8Z8aMg/ https://tools.taskcluster.net/task-inspector/#rtxbU02lSguNtpk41--itw/ https://tools.taskcluster.net/task-inspector/#MBERlFkhRqu4-iVrnDG8Zg/ https://tools.taskcluster.net/task-inspector/#dNLut3fJQsm3GxogjPM5TQ/ New round of tests. Alexandre, could you please pick one build and Aries and one for Flame and make sure the blobfree packages are actually blobfree?
Flags: needinfo?(lissyx+mozillians)
Assignee | ||
Updated•9 years ago
|
Attachment #8627521 -
Attachment description: MozReview Request: Bug 1175934 part 1: Add blob free build for flame and spark devices. r=garndt → MozReview Request: Bug 1175934 part 1: Copy blobfree zip to public upload dir. r?garndt
Assignee | ||
Comment 35•9 years ago
|
||
Comment on attachment 8627521 [details] MozReview Request: Bug 1175934 part 1: Copy blobfree zip to public upload dir. r?garndt Bug 1175934 part 1: Copy blobfree zip to public upload dir. r?garndt If we ./build.sh blobfree, move the zip file to public upload dir. Also, we don't need a blobfree specific config file anymore.
Assignee | ||
Comment 36•9 years ago
|
||
Comment on attachment 8628526 [details] MozReview Request: Bug 1175934 part 2: Support blobfree images. r?garndt Bug 1175934 part 2: Support blobfree images. r?garndt When we ./build.sh blobfree, we have a new zip file called <target>.blobfree-dist.zip which contains a blobfree image. We copy this file to public artifact. We also refactor the build scripts a little bit to deduplicate the post build code.
Attachment #8628526 -
Attachment description: MozReview Request: Bug 1175934 part 2: Allow blob free images go on public. r=garndt → MozReview Request: Bug 1175934 part 2: Support blobfree images. r?garndt
Assignee | ||
Updated•9 years ago
|
Attachment #8628526 -
Attachment is obsolete: true
Assignee | ||
Updated•9 years ago
|
Attachment #8629078 -
Attachment is obsolete: true
Assignee | ||
Comment 37•9 years ago
|
||
If we ./build.sh blobfree, move the zip file to public upload dir. Also, we don't need a blobfree specific config file anymore.
Assignee | ||
Comment 38•9 years ago
|
||
When we ./build.sh blobfree, we have a new zip file called <target>.blobfree-dist.zip which contains a blobfree image. We copy this file to public artifact. We also refactor the build scripts a little bit to deduplicate the post build code.
Assignee | ||
Updated•9 years ago
|
Attachment #8640226 -
Flags: review?(garndt)
Assignee | ||
Updated•9 years ago
|
Attachment #8640227 -
Flags: review?(garndt)
Comment 39•9 years ago
|
||
(In reply to Wander Lairson Costa [:wcosta] from comment #34) > https://tools.taskcluster.net/task-inspector/#YnehR_V6Q3WGrsJt9SBBqA/ > https://tools.taskcluster.net/task-inspector/#YnehR_V6Q3WGrsJt9SBBqA/ > https://tools.taskcluster.net/task-inspector/#gm3YpX6iQ3O9GmC3oGxKIw/ > https://tools.taskcluster.net/task-inspector/#SRqdbQ1JR7uIXNFo-6Wlzw/ > https://tools.taskcluster.net/task-inspector/#NlDeE_4vS0emXR9ZFN43ug/ > https://tools.taskcluster.net/task-inspector/#cdLZUjC1QxeFhM_tukh6hQ/ > https://tools.taskcluster.net/task-inspector/#L-M8OxcVQ1ueqW0k8Z8aMg/ > https://tools.taskcluster.net/task-inspector/#rtxbU02lSguNtpk41--itw/ > https://tools.taskcluster.net/task-inspector/#MBERlFkhRqu4-iVrnDG8Zg/ > https://tools.taskcluster.net/task-inspector/#dNLut3fJQsm3GxogjPM5TQ/ > > New round of tests. Alexandre, could you please pick one build and Aries and > one for Flame and make sure the blobfree packages are actually blobfree? Right, I'll finish checking once in the office but from the train, the file sizes are more consistent with blobfree :)
Flags: needinfo?(lissyx+mozillians)
Assignee | ||
Updated•9 years ago
|
Attachment #8640226 -
Flags: review?(garndt) → review?(jlund)
Comment 40•9 years ago
|
||
That looks good except I've noticed some discrepencies in bin/, xbin/ and lack of bootanimation. But those are files built so it looks good.
Assignee | ||
Updated•9 years ago
|
Attachment #8640226 -
Attachment is obsolete: true
Attachment #8640226 -
Flags: review?(jlund)
Assignee | ||
Updated•9 years ago
|
Attachment #8640227 -
Attachment is obsolete: true
Attachment #8640227 -
Flags: review?(garndt)
Assignee | ||
Comment 41•9 years ago
|
||
https://tools.taskcluster.net/task-inspector/#mpEcRgCjROqMUMLExSzqrw/ https://tools.taskcluster.net/task-inspector/#k93xza3hSw2v4C6FPS2Cag/ https://tools.taskcluster.net/task-inspector/#-0lHs_EdQfuuQx2O6D46kg/ https://tools.taskcluster.net/task-inspector/#g7iBA9UPT1OCOnFCNzyofA/ https://tools.taskcluster.net/task-inspector/#LEa8U2YcTMWOFrxRGbVCCw/ https://tools.taskcluster.net/task-inspector/#exFGdFqaRWCD9ckl-hFHWg/ https://tools.taskcluster.net/task-inspector/#AbHzu_TcToOJiMofjMy79w/ https://tools.taskcluster.net/task-inspector/#Nk3f0G9VShWPEIAM5_CBwA/ https://tools.taskcluster.net/task-inspector/#lJ13YaSVQMego0I4b2k49Q/ https://tools.taskcluster.net/task-inspector/#-QGP-1vyR6qXSxSiGZEy1w/ https://tools.taskcluster.net/task-inspector/#zEKp1gEhQy-7uqOkxfyD0w/ https://tools.taskcluster.net/task-inspector/#bfd57vtfRxiX54OdQ7YzHA/ As we talked in IRC, these builds add MOZILLA_OFICIAL and ENABLE_DEFAULT_BOOTANIMATION to all phone builds. Alexandre, would you mind checking a couple of builds again?
Flags: needinfo?(lissyx+mozillians)
Assignee | ||
Comment 42•9 years ago
|
||
If we ./build.sh blobfree, move the zip file to public upload dir. Also, we don't need a blobfree specific config file anymore.
Assignee | ||
Comment 43•9 years ago
|
||
When we ./build.sh blobfree, we have a new zip file called <target>.blobfree-dist.zip which contains a blobfree image. We copy this file to public artifact. We also refactor the build scripts a little bit to deduplicate the post build code.
Assignee | ||
Comment 44•9 years ago
|
||
All phone builds must have MOZILLA_OFFICIAL and ENABLE_DEFAULT_BOOTANIMATION build flags.
Comment 45•9 years ago
|
||
Checking https://tools.taskcluster.net/task-inspector/#LEa8U2YcTMWOFrxRGbVCCw/0 ...
Flags: needinfo?(lissyx+mozillians)
Comment 46•9 years ago
|
||
(In reply to Alexandre LISSY :gerard-majax from comment #45) > Checking > https://tools.taskcluster.net/task-inspector/#LEa8U2YcTMWOFrxRGbVCCw/0 ... Looks much better except it lacks updater binary :). But from a blobfree point of view, it's okay !
Assignee | ||
Updated•9 years ago
|
Attachment #8640473 -
Attachment is obsolete: true
Assignee | ||
Updated•9 years ago
|
Attachment #8640474 -
Attachment is obsolete: true
Assignee | ||
Updated•9 years ago
|
Attachment #8640475 -
Attachment is obsolete: true
Assignee | ||
Comment 47•9 years ago
|
||
New round of tests, hopefully the last one (famous last words). https://tools.taskcluster.net/task-inspector/#m7nikZwBSdmVpJyu1dDh8g/0 https://tools.taskcluster.net/task-inspector/#u17brfLtTjuXGwFP1haA6Q/0 https://tools.taskcluster.net/task-inspector/#PehFfYwZTpqt3BVBAdhnIw/0 https://tools.taskcluster.net/task-inspector/#9cNzJbmeSwKuXaqGycIZxA/0 https://tools.taskcluster.net/task-inspector/#wU_m2yPASuKC57don987zw/0 https://tools.taskcluster.net/task-inspector/#nbdZbzCtTiWj1oDMS7esRg/4 https://tools.taskcluster.net/task-inspector/#v7hDz6iXQr-8XuxnOgnl8w/1 https://tools.taskcluster.net/task-inspector/#WF4IIZYxS0CDV7IjYzZ1cg/0 https://tools.taskcluster.net/task-inspector/#XaXKs5pfS86NByGecs2pzw/0 https://tools.taskcluster.net/task-inspector/#zXxrFb-GSJGZ2ljY1VfXlQ/0 https://tools.taskcluster.net/task-inspector/#rVgdUoGRSVCRCduvkYMCWA/0 https://tools.taskcluster.net/task-inspector/#H_xISZfLT7KM879nHFZtRw/0
Assignee | ||
Comment 48•9 years ago
|
||
Alexandre, could you give a check on those?
Flags: needinfo?(lissyx+mozillians)
Assignee | ||
Comment 49•9 years ago
|
||
If we ./build.sh blobfree, move the zip file to public upload dir. Also, we don't need a blobfree specific config file anymore.
Assignee | ||
Comment 50•9 years ago
|
||
When we ./build.sh blobfree, we have a new zip file called <target>.blobfree-dist.zip which contains a blobfree image. We copy this file to public artifact. We also refactor the build scripts a little bit to deduplicate the post build code.
Assignee | ||
Comment 51•9 years ago
|
||
All phone builds must have B2G_UPDATER, MOZILLA_OFFICIAL and ENABLE_DEFAULT_BOOTANIMATION build flags.
Assignee | ||
Updated•9 years ago
|
Attachment #8641646 -
Flags: review?(jlund)
Assignee | ||
Updated•9 years ago
|
Attachment #8641647 -
Flags: review?(jlund)
Attachment #8641647 -
Flags: review?(garndt)
Assignee | ||
Updated•9 years ago
|
Attachment #8641648 -
Flags: review?(garndt)
Comment 52•9 years ago
|
||
Comment on attachment 8641647 [details] [diff] [review] part 2: Support blobfree images. Review of attachment 8641647 [details] [diff] [review]: ----------------------------------------------------------------- The taskcluster pieces of this look good to me.
Attachment #8641647 -
Flags: review?(garndt) → review+
Comment 53•9 years ago
|
||
Comment on attachment 8641648 [details] [diff] [review] part 3: Add more build flags to phone builds. Review of attachment 8641648 [details] [diff] [review]: ----------------------------------------------------------------- I'm not sure what those options do, but as long as they don't change the behaviors in the engineering build (in regards to flashing devices and having ADB enabled), then this looks good to me.
Attachment #8641648 -
Flags: review?(garndt) → review+
Comment 54•9 years ago
|
||
Comment on attachment 8641646 [details] [diff] [review] part 1: Copy blobfree zip to public upload dir. Review of attachment 8641646 [details] [diff] [review]: ----------------------------------------------------------------- sweet. ship it ::: testing/mozharness/scripts/b2g_build.py @@ +710,5 @@ > files.append(f) > if base_pattern in public_upload_patterns: > public_files.append(f) > > + device_name = self.config['target'].split('-')[0] this seems fragile. should we have another config item for this hard coded or do you think it's safe?
Attachment #8641646 -
Flags: review?(jlund) → review+
Comment 55•9 years ago
|
||
Comment on attachment 8641647 [details] [diff] [review] part 2: Support blobfree images. Review of attachment 8641647 [details] [diff] [review]: ----------------------------------------------------------------- mozharness pieces look sane ::: testing/mozharness/scripts/b2g_build.py @@ +521,5 @@ > self.run_command(["diff", "-u", sourcesfile_orig, sourcesfile], success_codes=[1]) > > def generate_build_command(self, target=None): > cmd = ['./build.sh'] > if target is not None: sanity check: so if target == "", it will make it past this point. In which case, we will always append '-j{0}'. that's what we want right?
Attachment #8641647 -
Flags: review?(jlund) → review+
Assignee | ||
Comment 56•9 years ago
|
||
(In reply to Jordan Lund (:jlund) from comment #54) > Comment on attachment 8641646 [details] [diff] [review] > part 1: Copy blobfree zip to public upload dir. > > Review of attachment 8641646 [details] [diff] [review]: > ----------------------------------------------------------------- > > sweet. ship it > > ::: testing/mozharness/scripts/b2g_build.py > @@ +710,5 @@ > > files.append(f) > > if base_pattern in public_upload_patterns: > > public_files.append(f) > > > > + device_name = self.config['target'].split('-')[0] > > this seems fragile. should we have another config item for this hard coded > or do you think it's safe? Even if it is an empty string, the code will work (of course the build will fail, but not because this code).
Assignee | ||
Comment 57•9 years ago
|
||
(In reply to Jordan Lund (:jlund) from comment #55) > Comment on attachment 8641647 [details] [diff] [review] > part 2: Support blobfree images. > > Review of attachment 8641647 [details] [diff] [review]: > ----------------------------------------------------------------- > > mozharness pieces look sane > > ::: testing/mozharness/scripts/b2g_build.py > @@ +521,5 @@ > > self.run_command(["diff", "-u", sourcesfile_orig, sourcesfile], success_codes=[1]) > > > > def generate_build_command(self, target=None): > > cmd = ['./build.sh'] > > if target is not None: > > sanity check: so if target == "", it will make it past this point. In which > case, we will always append '-j{0}'. that's what we want right? Yes, it is.
Assignee | ||
Comment 58•9 years ago
|
||
url: https://hg.mozilla.org/integration/b2g-inbound/rev/5f4839ec1475be192508f3df6af815acd21ddae3 changeset: 5f4839ec1475be192508f3df6af815acd21ddae3 user: Wander Lairson Costa <wcosta@mozilla.com> date: Fri Jul 31 17:24:46 2015 -0300 description: Bug 1175934 part 1: Copy blobfree zip to public upload dir. If we ./build.sh blobfree, move the zip file to public upload dir. Also, we don't need a blobfree specific config file anymore. url: https://hg.mozilla.org/integration/b2g-inbound/rev/e6f3a48c462bf2604260800aeb03d4e4b335c303 changeset: e6f3a48c462bf2604260800aeb03d4e4b335c303 user: Wander Lairson Costa <wcosta@mozilla.com> date: Fri Jul 31 17:24:46 2015 -0300 description: Bug 1175934 part 2: Support blobfree images. When we ./build.sh blobfree, we have a new zip file called <target>.blobfree-dist.zip which contains a blobfree image. We copy this file to public artifact. We also refactor the build scripts a little bit to deduplicate the post build code. url: https://hg.mozilla.org/integration/b2g-inbound/rev/6fc853368a5aae767ba52daa232bc52ba91cc7e8 changeset: 6fc853368a5aae767ba52daa232bc52ba91cc7e8 user: Wander Lairson Costa <wcosta@mozilla.com> date: Fri Jul 31 17:24:46 2015 -0300 description: Bug 1175934 part 3: Add more build flags to phone builds. All phone builds must have B2G_UPDATER, MOZILLA_OFFICIAL and ENABLE_DEFAULT_BOOTANIMATION build flags.
Updated•9 years ago
|
Flags: needinfo?(lissyx+mozillians)
Comment 59•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/5f4839ec1475 https://hg.mozilla.org/mozilla-central/rev/e6f3a48c462b https://hg.mozilla.org/mozilla-central/rev/6fc853368a5a
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•