819342 - Update b2g snapshot to capture test packaging changes

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Armen [:armenzg]]

This was resolved in bug 817040.

(In reply to Jonathan Griffin (:jgriffin) from comment #10)
> https://github.com/mozilla-b2g/gaia/commit/
> f3973d3d9834ed851b9987efb2a8ef9dbd7b32a4
> https://github.com/mozilla-b2g/gonk-misc/commit/
> 871cbba1eb71a54720515179130f79ac6107351b
> 
> You'll need to update your gonk-misc repo to use this.  After that, the test
> package can be generated using:
> 
> ./build.sh package-tests
> 
> The package will be placed in
> out/target/product/{productname}/tests/gaia-tests.zip

Comment 1

[Armen [:armenzg]]

Attachment: update gonk-misc

Created snapshot following private instructions in:
https://wiki.mozilla.org/ReleaseEngineering/How_To/Update_a_B2G_snapshot

The changes that we want are this:
https://github.com/mozilla-b2g/gonk-misc/compare/9947f535c...871cbba1
-  <project name="gonk-misc" path="gonk-misc" remote="b2g" revision="9947f535cbe34fe583d122322b077cb0fc0a9ff7"/>
+  <project name="gonk-misc" path="gonk-misc" remote="b2g" revision="871cbba1eb71a54720515179130f79ac6107351b"/>

Triggered job in:
https://tbpl.mozilla.org/?tree=Try&rev=b91238306407

Comment 2

[Armen [:armenzg]]

It build successfully and it seems to use the right revision:
https://tbpl.mozilla.org/php/getParsedLog.php?id=17722528&tree=Try&full=1
12:22:15     INFO -    <project name="gonk-misc" path="gonk-misc" remote="b2g" revision="871cbba1eb71a54720515179130f79ac6107351b"/>

I nevertheless cannot create the test packages. Anything that I did wrong?:
[cltbld@bld-centos6-hp-027 build]$ ./build.sh package-tests
including device/ti/panda/vendorsetup.sh
build/envsetup.sh: line 44: make: command not found

** Don't have a product spec for: 'full_panda'
** Do you have the right repo manifest?


> Build failed! <

Build with |./build.sh -j1| for better messages
If all else fails, use |rm -rf objdir-gecko| to clobber gecko and |rm -rf out| to clobber everything else.

Comment 3

[Chris AtLee [:catlee]]

That probably needs to be run from inside mock

Comment 4

[Jonathan Griffin (:jgriffin)]

(In reply to Chris AtLee [:catlee] from comment #3)
> That probably needs to be run from inside mock

That makes sense; I don't know why else it wouldn't understand 'full_panda'.

Comment 5

[Armen [:armenzg]]

Comment on attachment 689848 [details] [diff] [review]
update gonk-misc

We can grab a slave on Monday to try it.
Feel free to r- if you think we should check that first.

Comment 6

[Aki Sasaki (not active)]

Comment on attachment 689848 [details] [diff] [review]
update gonk-misc

We should probably check on Monday.

I'm ok with you landing this on Cedar though, even without r?, til we know more.

Comment 7

[Armen [:armenzg]]

Comment on attachment 689848 [details] [diff] [review]
update gonk-misc

It worked on my local testing.

Comment 8

[Armen [:armenzg]]

Attachment: enable packaging for panda builds and upload the test package

Comment 9

[Aki Sasaki (not active)]

Comment on attachment 690433 [details] [diff] [review]
enable packaging for panda builds and upload the test package

:)
Could you update b2g/config/panda-gaia-central/config.json as well?
Looks like everything else in there softlinks to b2g/config/panda/ except config.json.

Comment 10

[Armen [:armenzg]]

Comment on attachment 689848 [details] [diff] [review]
update gonk-misc

http://hg.mozilla.org/integration/mozilla-inbound/rev/0e23cc4a25b7
http://hg.mozilla.org/projects/cedar/rev/0b818d5411c9

Comment 11

[Armen [:armenzg]]

Comment on attachment 690433 [details] [diff] [review]
enable packaging for panda builds and upload the test package

http://hg.mozilla.org/integration/mozilla-inbound/rev/a19bcc4fd493
http://hg.mozilla.org/projects/cedar/rev/142d1baef990

Comment 12

[Armen [:armenzg]]

We will have to uplift the b2g snapshot once the sendchanges point to the gaia-test.zip.

Comment 13

[Armen [:armenzg]]

(In reply to Armen Zambrano G. [:armenzg] from comment #12)
> We will have to uplift the b2g snapshot once the sendchanges point to the
> gaia-test.zip.

We might not need to uplift this if bug 819979 gets uplifted first.

I will audit the patches on the various bugs and determine which patches need uplifting.

This has been fixed but it will need to be uplifted directly or via other bugs.

Comment 14

[Armen [:armenzg]]

(In reply to Armen Zambrano G. [:armenzg] from comment #13)
> (In reply to Armen Zambrano G. [:armenzg] from comment #12)
> > We will have to uplift the b2g snapshot once the sendchanges point to the
> > gaia-test.zip.
> 
> We might not need to uplift this if bug 819979 gets uplifted first.
> 
> I will audit the patches on the various bugs and determine which patches
> need uplifting.
> 
> This has been fixed but it will need to be uplifted directly or via other
> bugs.

I've confirmed with bhearsum that he will uplift his changes which will include this one.

Comment 15

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/0e23cc4a25b7
https://hg.mozilla.org/mozilla-central/rev/a19bcc4fd493