1244893 - Disable F/OTA updates to balrog

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, task)

Description

[Selena Deckelmann :selenamarie :selena]

Per bug 1232008, this bug is to disable our F/OTA updates. 

We have identified a way supporting F/OTA updates going forward identified in bug 1244181.

Comment 1

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: 0001-bug-1244893-Disable-Balrog.patch

Here's a patch; I still need to test.  Wanting to get feedback to see if I'm on the right path or if I should make modifications to the approach.

Test build : https://tools.taskcluster.net/task-inspector/#L27Fpbz8RASBixxD_eUChA/

based on mach command : ./mach taskcluster-build --head-repo=https://github.com/nhirata/gecko-dev --head-rev=bug_1244893_disable_balrog --owner=[e-mail] tasks/builds/b2g_aries_spark_dogfood.yml | sed -e 's/0\.0\.24/0.99.22/'

Comment 2

[Wander Lairson Costa]

Comment on attachment 8714655 [details] [diff] [review]
0001-bug-1244893-Disable-Balrog.patch

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

lgtm. As a follow up, we will have to remove the credentials from S3 (we = I).

::: testing/taskcluster/scripts/phone-builder/build-phone-ota.sh
@@ -27,5 @@
>  
>  $WORKSPACE/gecko/testing/mozharness/scripts/b2g_build.py \
>    --config $MOZHARNESS_CONFIG \
> -  --config $BALROG_SERVER_CONFIG \
> -  "$debug_flag" \

I don't think debug flag needs to be removed.

Comment 3

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: 0001-bug-1244893-Disable-Balrog.patch

Patch fixed based on comment.

Comment 4

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Rerunning as a test : https://tools.taskcluster.net/task-inspector/#DbZaOShgQIK-2VuGvINtgA/

Comment 5

[Wander Lairson Costa]

Could you please run a try push?

Comment 6

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: 0001-bug-1244893-Disable-Balrog.patch

Didn't they remove b2g from the tree?
If they didn't running try won't test the builds will it?

Oh I just realized I went too overboard though with the removal.

Comment 7

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

push to try : https://treeherder.mozilla.org/#/jobs?repo=try&revision=209929a8aa6c

Comment 8

[Wander Lairson Costa]

(In reply to Naoki Hirata :nhirata (please use needinfo instead of cc) from comment #6)
> Created attachment 8715562 [details] [diff] [review]
> 0001-bug-1244893-Disable-Balrog.patch
> 
> Didn't they remove b2g from the tree?

afaik, no.

> If they didn't running try won't test the builds will it?
> 

Just use -p all -t none -u none.

> Oh I just realized I went too overboard though with the removal.

What did you change in your patch?

Comment 9

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

I removed the - from 2 references to : 
       MOZHARNESS_CONFIG: b2g/taskcluster-spark-ota-balrog.py 
from my original patch.  Without those, it doesn't have a script to build... which would bust the build from building.  I put them back in.

The push to try in comment 7 ( https://treeherder.mozilla.org/#/jobs?repo=try&revision=209929a8aa6c ) uses : try: -b o -e -p all -u all -t none
is that good enough?

Comment 10

[Wander Lairson Costa]

Comment on attachment 8715562 [details] [diff] [review]
0001-bug-1244893-Disable-Balrog.patch

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

lgtm, but notice the two tests with aries dogfood failed to build. Although the bustages don't relate to the patch, the balrog stuff was not tested. I recommend rerunning again.

Comment 11

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=065eb646e106

Comment 12

[Wander Lairson Costa]

It feels like no "-b do" flag was supplied.

Comment 13

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

(In reply to Wander Lairson Costa [:wcosta] from comment #12)
> It feels like no "-b do" flag was supplied.

Yes, no "-b do" based on based on comment 8.

resent with the -b do option : 
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9b4fc432c7c4

Comment 14

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

woops.  last one didn't have -b do either.

This one does:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=b2833855e9bc

Comment 15

[Wander Lairson Costa]

(In reply to Naoki Hirata :nhirata (please use needinfo instead of cc) from comment #13)
> (In reply to Wander Lairson Costa [:wcosta] from comment #12)
> > It feels like no "-b do" flag was supplied.
> 
> Yes, no "-b do" based on based on comment 8.
> 
> resent with the -b do option : 
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=9b4fc432c7c4

My bad, forgot to add the flag...

Comment 16

[Wander Lairson Costa]

Notice B2G builds live in staging now https://treeherder.allizom.org/#/jobs?repo=try&revision=b2833855e9bc&selectedJob=17059142

Comment 17

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Thanks, Wander!  Yes.  I realize I need that flag.

Posting checkin-needed and pinging a sheriff for checkin.  :)

Comment 18

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/88735739af7a

Comment 19

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/88735739af7a

Comment 20

[Carsten Book [:Tomcat]]

backed out for problems like https://treeherder.mozilla.org/logviewer.html#?job_id=3283060&repo=mozilla-central that happend after this got merged to m-c

Comment 21

[Greg Arndt [:garndt]]

this was backed out because of the empty "scope:" assignment in the debug and opt task definitions for spark aries ota.  I would remove that key entirely from the yml and it should be ok.

Comment 22

[Pulsebot]

Backout:
https://hg.mozilla.org/mozilla-central/rev/d719ac4bcbec

Comment 23

[Wander Lairson Costa]

(In reply to Greg Arndt [:garndt] from comment #21)
> this was backed out because of the empty "scope:" assignment in the debug
> and opt task definitions for spark aries ota.  I would remove that key
> entirely from the yml and it should be ok.

Naoki, could you please update the patch?

Comment 24

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: 0001-bug-1244893-disable-balrog.patch

Removed empty scope.

Comment 25

[Wander Lairson Costa]

Comment on attachment 8720547 [details] [diff] [review]
0001-bug-1244893-disable-balrog.patch

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

Just on nit before r+.

::: testing/taskcluster/tasks/builds/b2g_aries_spark_ota_balrog_opt.yml
@@ -6,4 @@
>    payload:
> -    features:
> -      balrogVPNProxy: true
> -    env:

I think we should not remove env:, right?

Comment 26

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Attachment: 0001-bug-1244893-disable-balrog.patch

ya.  you're right.

Comment 27

[Wander Lairson Costa]

Thanks for updating the patch, Naoki. I am going to test the patch next week.

Comment 29

[Wander Lairson Costa]

Comment on attachment 8721035 [details] [diff] [review]
0001-bug-1244893-disable-balrog.patch

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

https://treeherder.allizom.org/#/jobs?repo=try&revision=00bc02adacaf&selectedJob=17858598

Comment 30

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a3837ca1b1c9

Comment 31

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a3837ca1b1c9