Closed Bug 1175361 Opened 6 years ago Closed 6 years ago

add partition fix to the flame-kk builds

Categories

(Taskcluster :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: nhirata, Unassigned)

References

Details

see https://bugzilla.mozilla.org/show_bug.cgi?id=1085230#c12

This fixes issues with OTA.

I am not sure if Taskcluster is capable of adding additional partitions and having a change to the flash.sh script.

I am willing to help out to make the changes for the pvtbuilds.
[Blocking Requested - why for this release]:
This blocks a smoke test case and creates a blocker.
Blocks: 1085230
blocking-b2g: --- → 3.0?
Keywords: smoketest
Wander, is this something that's needed on the build script side? This isn't taskcluster specific, but I'm not sure how the phone images get created with the partitions
Flags: needinfo?(wcosta)
(In reply to Greg Arndt [:garndt] from comment #2)
> Wander, is this something that's needed on the build script side? This isn't
> taskcluster specific, but I'm not sure how the phone images get created with
> the partitions

/me <--- deadlock
I have no idea how this work, I guess this is done inside mozharness or in the B2G build system. I will investigate and let you know. I am going to keep NI on me to not forget this.
garndt, wcosta:
1) We need the flame.zip file to include gpt_both0_big.bin from bug 1085230.
2) Then we need to change the flash script to include : 
fastboot flash partition gpt_both0_big.bin
Flags: needinfo?(garndt)
Naoki,

Any clue where the flashboot command fits inside flash.sh script?
Flags: needinfo?(wcosta) → needinfo?(nhirata.bugzilla)
Removing ni? , Wander has the best context on this and this is not a taskcluster platform specific issue I don't believe.
Flags: needinfo?(garndt)
Keywords: smoketest
I already have patches for this (somewhere), I only need to know the context of the adb command inside flash.sh, this is the last piece.
blocking-b2g: 2.5? → ---
Component: TaskCluster → General
Product: Testing → Taskcluster
It's done by fastboot not adb:
fastboot flash partition gpt_both0_big.bin
The flash script that's associated with the zip file needs to be updated to have that line.
Flags: needinfo?(nhirata.bugzilla)
I think we need the flash blob to contain the partition fix as well.  I'm not sure how the blob images are put together for the zip file.  Pinging jonasjf for that support/info.
Flags: needinfo?(jopsen)
:nhirata,
I'm sorry to say I have little clue what this is about. I have no idea about the build system.
This seems like something that happens inside the task, so you'll probably have to look at the
docker image (I suspect quay.io/mozilla/phone-builder) or whatever scripts it runs from the tree.
Flags: needinfo?(jopsen)
I think the docker image has to include the partition resize file, an edit to the flash script and a recoding of the zipping to include the partition resize file...

- OR - 

we replace the partition resize file in the docker image.  ie mv gpt_both0_big.bin gpt_both0.bin

Otherwise we would be relying on the base image to fix the issue, and we would have to make sure that whoever flashes these images use the base build image first and then flash a build.
Since we're doing FOTA now, it appears that we don't need this change to occur.
The base build should include the fix.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.