dolphin full image size is incorrect in pvt build



Release Engineering
General Automation
3 years ago
3 years ago


(Reporter: Peipei Cheng (needinfo if you need my action), Assigned: nthomas)



Firefox Tracking Flags

(b2g-v1.4 fixed)



(1 attachment)

I found that the size for dolphin full image( is only 9.7K. Please help check. Thanks!

Comment 1

3 years ago
Hi, Chris,

Could I have your help?
The 1)"boot.img", 2)"system.img", 3)"recovery.img", and 4)"userdata.img" don't exist on the "b2g-distro" folder.
We seemed to forget to package the correct images.

Flags: needinfo?(catlee)
Looks like this is because the images are under 


In this case, the "zip_files" section of needs to be updated to refer to the proper local directory.
Flags: needinfo?(catlee)

Comment 3

3 years ago
Could someone help on this problem?
Sometimes, we need these images to do automation test and manual test


3 years ago
blocking-b2g: --- → 1.4?

Comment 4

3 years ago
Created attachment 8450602 [details] [diff] [review]
[mozilla-b2g30_v1_4]  Use real value of DEVICE when packaging

I've confirmed catlee's comment about the img files being in 
and infer we should put them in 
in the zip. This if from .config having |DEVICE=scx15_sp7715ga|, and $DEVICE being used in 

The line for {workdir}/boot.img is dropped because the file doesn't exist there, it's in {workdir}/out/target/product/scx15_sp7715ga/ so the glob catches it.
Assignee: nobody → nthomas
Attachment #8450602 - Flags: review?(jlund)


3 years ago
Blocks: 1032066

Comment 5

3 years ago
This won't block the release.
blocking-b2g: 1.4? → ---
Flags: needinfo?(nthomas)

Comment 6

3 years ago
What is the needinfo asking for ? Current status is review pending.
Flags: needinfo?(nthomas) → needinfo?(itsay)

Comment 7

3 years ago
did it by accident.
Flags: needinfo?(itsay)
Comment on attachment 8450602 [details] [diff] [review]
[mozilla-b2g30_v1_4]  Use real value of DEVICE when packaging

logic seems sound. I could not track down a recent slave (all spots) to probe and confirm the locations of these.

I wonder if we need a verification (path.exists) check here:

I guess it's hard since we don't know what to expect with a '*.img' glob
Attachment #8450602 - Flags: review?(jlund) → review+

Comment 9

3 years ago
Maybe RyanVM can land this before I acquire a 1.4 tree.
Severity: normal → critical
Keywords: checkin-needed

Comment 10

3 years ago
Clearing checkin-needed, I need to ask for approval before landing.
Severity: critical → normal
Keywords: checkin-needed

Comment 11

3 years ago
Comment on attachment 8450602 [details] [diff] [review]
[mozilla-b2g30_v1_4]  Use real value of DEVICE when packaging

[Approval Request Comment]
Bug caused by (feature/regressing bug #): Incomplete build setup in bug 1011129
User impact if declined: None
Testing completed: None
Risk to taking this patch (and alternatives if risky): None, affects pvtbuilds only
String or UUID changes made by this patch: None
Attachment #8450602 - Flags: approval-mozilla-b2g30?
I think this is quite critical. Without this fix, pvtbuilds for dolphin was built but nothing is packed into the In other words, we spent machine hours to create build but build is not delivered correctly.
Comment on attachment 8450602 [details] [diff] [review]
[mozilla-b2g30_v1_4]  Use real value of DEVICE when packaging

Attachment #8450602 - Flags: approval-mozilla-b2g30? → approval-mozilla-b2g30+

Comment 14

3 years ago
status-b2g-v1.4: --- → fixed

Comment 15

3 years ago
Forced some dolphin builds on that rev for verification, but they won't show up at
until they've finished (user error!).

Comment 16

3 years ago
In the non-eng build:

16:04:08     INFO - Copy/paste: zip -r -9 -u /builds/slave/b2g_m-b30_14_dolphin_dep-00000/build/ b2g-distro
16:04:08     INFO -  	zip warning: /builds/slave/b2g_m-b30_14_dolphin_dep-00000/build/ not found or empty
16:04:08     INFO -    adding: b2g-distro/ (stored 0%)
16:04:08     INFO -    adding: b2g-distro/ (deflated 41%)
16:04:08     INFO -    adding: b2g-distro/.config (deflated 19%)
16:04:08     INFO -    adding: b2g-distro/sources.xml (deflated 68%)
16:04:08     INFO -    adding: b2g-distro/out/ (stored 0%)
16:04:08     INFO -    adding: b2g-distro/out/target/ (stored 0%)
16:04:08     INFO -    adding: b2g-distro/out/target/product/ (stored 0%)
16:04:08     INFO -    adding: b2g-distro/out/target/product/scx15_sp7715ga/ (stored 0%)
16:04:08     INFO -    adding: b2g-distro/out/target/product/scx15_sp7715ga/ramdisk.img (deflated 0%)
16:04:09     INFO -    adding: b2g-distro/out/target/product/scx15_sp7715ga/recovery.img (deflated 47%)
16:04:11     INFO -    adding: b2g-distro/out/target/product/scx15_sp7715ga/boot.img (deflated 50%)
16:04:11     INFO -    adding: b2g-distro/out/target/product/scx15_sp7715ga/userdata.img (deflated 100%)
16:04:16     INFO -    adding: b2g-distro/out/target/product/scx15_sp7715ga/system.img (deflated 14%)
16:04:16     INFO -    adding: b2g-distro/out/target/product/scx15_sp7715ga/cache.img (deflated 100%)
16:04:16     INFO -    adding: b2g-distro/out/target/product/scx15_sp7715ga/ramdisk-recovery.img (deflated 0%)
16:04:16     INFO -    adding: b2g-distro/ (deflated 61%)
16:04:16     INFO - Return code: 0 is 107MB. We get 112MB of for the eng build.

pcheng, you could verify with (or later).
Last Resolved: 3 years ago
Resolution: --- → FIXED


3 years ago
Duplicate of this bug: 1032066

Comment 18

3 years ago
Cool! Thanks everyone! Dolphin build is back.
I have successfully flashed these images into dolphin devices.
But, the ENG build still doesn't have test apps, it may relate to build configuration.
I would like to trace it on the other bug.

I am grateful for your help.
Thanks! Have a nice day!
You need to log in before you can comment on or make changes to this bug.