[B2G] Add support to build blobfree images

RESOLVED FIXED

Status

Release Engineering
Mozharness
P2
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: wcosta, Assigned: wcosta)

Tracking

unspecified
ARM
Gonk (Firefox OS)
Dependency tree / graph

Firefox Tracking Flags

(firefox42 fixed)

Details

MozReview Requests

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(7 attachments, 7 obsolete attachments)

40 bytes, text/x-review-board-request
garndt
: review+
Details | Review
40 bytes, text/x-review-board-request
jlund
: review+
Details | Review
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
(Assignee)

Description

2 years ago
B2G can create blob free images, through the command "./build.sh blobfree"
(Assignee)

Updated

2 years ago
OS: Unspecified → Gonk (Firefox OS)
Priority: -- → P2
Hardware: Unspecified → ARM
Flags: needinfo?(nhirata.bugzilla)
Blocks: 1166276
(Assignee)

Updated

2 years ago
Assignee: nobody → wcosta
(Assignee)

Updated

2 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

2 years ago
Depends on: 1178388
(Assignee)

Comment 1

2 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

2 years ago
Created attachment 8627521 [details]
MozReview Request: Bug 1175934 part 1: Copy blobfree zip to public upload dir. r?garndt

Bug 1175934: Add blob free build for flame and spark devices. r=garndt
Attachment #8627521 - Flags: review?(garndt)
(Assignee)

Comment 3

2 years ago
I cannot run task-graph to completely test this patch due to Bug 1177183, however.

Comment 4

2 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

2 years ago
Created attachment 8628506 [details]
MozReview Request: Bug 1175934 part 1: Allow build target configuration from config file. r=jlund

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

2 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

2 years ago
Created attachment 8628526 [details]
MozReview Request: Bug 1175934 part 2: Support blobfree images. r?garndt

Bug 1175934 part 2: Allow blob free images go on public. r=garndt

Die, blobs! Die!
Attachment #8628526 - Flags: review?(garndt)
(Assignee)

Comment 8

2 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/
I don't see the |build.sh blobfree| statement in the logs of those runs :(
(Assignee)

Comment 10

2 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 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 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+
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 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

2 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

2 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

2 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

2 years ago
Created attachment 8629078 [details]
MozReview Request: Bug 1175934 part 2: Add blob free build configs.

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 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 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

2 years ago
https://hg.mozilla.org/build/mozharness/rev/1d9d27ca8b31
https://hg.mozilla.org/build/mozharness/rev/0e613ece6d93
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/production
mozharness production tag moved to: https://hg.mozilla.org/build/mozharness/rev/production
(Assignee)

Comment 23

2 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

2 years ago
Created attachment 8632768 [details] [diff] [review]
phone-builder-version.patch

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

2 years ago
Attachment #8632768 - Flags: review?(garndt) → review+
(Assignee)

Comment 25

2 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
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox42: --- → fixed
Resolution: --- → FIXED
Flags: needinfo?(nhirata.bugzilla)
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

2 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)
(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

2 years ago
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 30

2 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)

Updated

2 years ago
Depends on: 1185643
(Assignee)

Comment 31

2 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/
(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.
Created attachment 8639544 [details]
blobfree.log

That is the expected output.
(Assignee)

Comment 34

2 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

2 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

2 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

2 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

2 years ago
Attachment #8628526 - Attachment is obsolete: true
(Assignee)

Updated

2 years ago
Attachment #8629078 - Attachment is obsolete: true
(Assignee)

Comment 37

2 years ago
Created attachment 8640226 [details] [diff] [review]
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 38

2 years ago
Created attachment 8640227 [details] [diff] [review]
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.
(Assignee)

Updated

2 years ago
Attachment #8640226 - Flags: review?(garndt)
(Assignee)

Updated

2 years ago
Attachment #8640227 - Flags: review?(garndt)
(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

2 years ago
Attachment #8640226 - Flags: review?(garndt) → review?(jlund)
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

2 years ago
Attachment #8640226 - Attachment is obsolete: true
Attachment #8640226 - Flags: review?(jlund)
(Assignee)

Updated

2 years ago
Attachment #8640227 - Attachment is obsolete: true
Attachment #8640227 - Flags: review?(garndt)
(Assignee)

Comment 41

2 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

2 years ago
Created attachment 8640473 [details] [diff] [review]
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.
(Assignee)

Comment 43

2 years ago
Created attachment 8640474 [details] [diff] [review]
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.
(Assignee)

Comment 44

2 years ago
Created attachment 8640475 [details] [diff] [review]
part 3: Add more build flags to phone builds.

All phone builds must have MOZILLA_OFFICIAL and
ENABLE_DEFAULT_BOOTANIMATION build flags.
Checking https://tools.taskcluster.net/task-inspector/#LEa8U2YcTMWOFrxRGbVCCw/0 ...
Flags: needinfo?(lissyx+mozillians)
(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

2 years ago
Attachment #8640473 - Attachment is obsolete: true
(Assignee)

Updated

2 years ago
Attachment #8640474 - Attachment is obsolete: true
(Assignee)

Updated

2 years ago
Attachment #8640475 - Attachment is obsolete: true
(Assignee)

Comment 47

2 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

2 years ago
Alexandre, could you give a check on those?
Flags: needinfo?(lissyx+mozillians)
(Assignee)

Comment 49

2 years ago
Created attachment 8641646 [details] [diff] [review]
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.
(Assignee)

Comment 50

2 years ago
Created attachment 8641647 [details] [diff] [review]
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.
(Assignee)

Comment 51

2 years ago
Created attachment 8641648 [details] [diff] [review]
part 3: Add more build flags to phone builds.

All phone builds must have B2G_UPDATER, MOZILLA_OFFICIAL and
ENABLE_DEFAULT_BOOTANIMATION build flags.
(Assignee)

Updated

2 years ago
Attachment #8641646 - Flags: review?(jlund)
(Assignee)

Updated

2 years ago
Attachment #8641647 - Flags: review?(jlund)
Attachment #8641647 - Flags: review?(garndt)
(Assignee)

Updated

2 years ago
Attachment #8641648 - Flags: review?(garndt)
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 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 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 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

2 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

2 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

2 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.
Flags: needinfo?(lissyx+mozillians)
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
Last Resolved: 2 years ago2 years ago
Resolution: --- → FIXED
https://hg.mozilla.org/mozilla-central/rev/e810d4d7ad99
You need to log in before you can comment on or make changes to this bug.