1207770 - Add no ril device on TaskCluster

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, task)

Description

[Ronald Claveau [:sousmangoosta]]

We don't have a NO RIL TaskCluster image builded on try and other branches.
Can be source of bug on tablet builds where used of disable-b2g-ril.

Comment 1

[Ronald Claveau [:sousmangoosta]]

Attachment: Use aries existant image with an env variables to simulate no ril device

Comment 2

[Ronald Claveau [:sousmangoosta]]

I pushed a try job here https://treeherder.mozilla.org/#/jobs?repo=try&revision=75e2d68df396

Comment 3

[Greg Arndt [:garndt]]

Comment on attachment 8665089 [details] [diff] [review]
Use aries existant image with an env variables to simulate no ril device

Review of attachment 8665089 [details] [diff] [review]:
-----------------------------------------------------------------

Looks like this is orange on TH.  Also, we should ask the sheriffs if this should be reporting to TH production.  I'm not sure what visibility should be set for a job like this.  Also, this has the same symbol as the regular build, so we might want to make it unique so there aren't more than one 'B' symbol.

Thanks for working on this!

Comment 4

[Ronald Claveau [:sousmangoosta]]

It should be orange until Bug 1207266 be landed.
Will try do better tomorrow for the symbol.

Comment 5

[Ronald Claveau [:sousmangoosta]]

The try with additionnaly the patch of Bug 1207266 looks good : https://treeherder.mozilla.org/#/jobs?repo=try&revision=2613cebacfc8

Comment 6

[Ronald Claveau [:sousmangoosta]]

Attachment: 9bb346625a8e.patch

This build passed try : https://treeherder.mozilla.org/#/jobs?repo=try&revision=096835397c3b&selectedJob=14227718

Comment 7

[Carsten Book [:Tomcat]]

(In reply to Greg Arndt [:garndt] from comment #3)
> Comment on attachment 8665089 [details] [diff] [review]
> Use aries existant image with an env variables to simulate no ril device
> 
> Review of attachment 8665089 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> Looks like this is orange on TH.  Also, we should ask the sheriffs if this
> should be reporting to TH production.  I'm not sure what visibility should
> be set for a job like this.  Also, this has the same symbol as the regular
> build, so we might want to make it unique so there aren't more than one 'B'
> symbol.
> 
> Thanks for working on this!

Hi Roland, we have visibility requirements here https://wiki.mozilla.org/Sheriffing/Job_Visibility_Policy#Requirements_for_jobs_shown_in_the_default_Treeherder_view

that requirements must be passed before getting to default treeherder

Comment 8

[Ronald Claveau [:sousmangoosta]]

Attachment: [Tier-2] on treeherder

This patch adds Tier-2 flag on Treeherder

Comment 9

[Greg Arndt [:garndt]]

Great! This looks good to me.

Comment 10

[Ronald Claveau [:sousmangoosta]]

Comment on attachment 8697008 [details] [diff] [review]
[Tier-2] on treeherder

Build is ok on try : https://treeherder.mozilla.org/#/jobs?repo=try&revision=381c03dc1c17

Comment 11

[Greg Arndt [:garndt]]

Comment on attachment 8697008 [details] [diff] [review]
[Tier-2] on treeherder

Review of attachment 8697008 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me.

Comment 12

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/49efc3161093

Comment 13

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/49efc3161093