791854 - Disable noion armv7 android builds/tests

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Justin Wood (:Callek)]

So, after a chat with dvander it appears we don't need the armv7-noion tests after we clarified a few facts:

* Armv6 builds are built without ionmonkey
* Armv6 tests are [currently] run on tegras, which are an armv7 chipset
* the JIT/JS code that is armv6 specific are determined at runtime, not build time.

Due to this, the specific armv6-noion builds are suitable for the coverage the B2G team needed out of noion-on-armv7.

We'll need armv7-noion builds *again* IFF we start testing armv6 builds on an armv6-chipset prior to having B2G specific testing up.

... This bug is to track that work, and will be primarily backing out the [rebased] changes from Bug 789373 - use the new --disable-ion flag for specific b2g-related builds

Comment 1

[Justin Wood (:Callek)]

Attachment: [m-c] remove noion config

This attachment removes the noion configs from m-c, and will be landed after the reconfig to disable noion builds

Comment 2

[Justin Wood (:Callek)]

Attachment: [puppet-manifests] Remove from PW templates

This patch as well would land after the reconfig.

Comment 3

[Justin Wood (:Callek)]

Attachment: [tools] remove from pm.json

Comment 4

[Justin Wood (:Callek)]

Attachment: [custom] remove refs to noion

Comment 5

[Aki Sasaki (not active)]

Comment on attachment 661957 [details] [diff] [review]
[custom] remove refs to noion

We don't need the backslashes in the |if self.OS in| check, but they don't hurt.

Comment 6

[Justin Wood (:Callek)]

Attachment: [config] Remove noion from production

Comment 7

[Justin Wood (:Callek)]

Attachment: [tbpl] remove refs

This should land ~24 hours after the reconfig, to give any longer-lasting jobs a chance to properly show.

Comment 8

[Aki Sasaki (not active)]

Comment on attachment 661960 [details] [diff] [review]
[config] Remove noion from production

passes test-masters.sh (with specified -j), and passes the grep -rl test.

Comment 9

[Justin Wood (:Callek)]

Comment on attachment 661956 [details] [diff] [review]
[tools] remove from pm.json

http://hg.mozilla.org/build/tools/rev/60b0b6d56411

Comment 10

[Justin Wood (:Callek)]

Comment on attachment 661957 [details] [diff] [review]
[custom] remove refs to noion

http://hg.mozilla.org/build/buildbotcustom/rev/3d29d0dcc378

Comment 11

[Justin Wood (:Callek)]

Comment on attachment 661960 [details] [diff] [review]
[config] Remove noion from production

http://hg.mozilla.org/build/buildbot-configs/rev/6e0589868327

Comment 12

[Justin Wood (:Callek)]

(In reply to Aki Sasaki [:aki] from comment #5)
> Comment on attachment 661957 [details] [diff] [review]
> [custom] remove refs to noion
> 
> We don't need the backslashes in the |if self.OS in| check, but they don't
> hurt.

Since I was effectively backing out your change in this hunk, I opted not to re-remove the \'s (for now)

Comment 13

[Phil Ringnalda (:philor)]

Comment on attachment 661961 [details] [diff] [review]
[tbpl] remove refs

Actually, it should land 31 days after the reconfig, after all the noion logs are gone from ftp.m.o and nobody can be looking at anything on tbpl and getting confused by the way that every Android row has two builds and two sets of most tests, but whatever - between the lag until the next push to production tbpl and the way that almost nobody looks at them while they are fresh, much less once they are stale, it won't make any practical difference.

Comment 15

[Justin Wood (:Callek)]

Resolving as "Fixed" without landing teh remaining patches, since we need to re-enable these builds in Bug 793994