Selena Deckelmann :selenamarie :selena Reporter
	Description • 8 years ago

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.

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 1 • 8 years ago

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=nhirata@mozilla.com tasks/builds/b2g_aries_spark_dogfood.yml | sed -e 's/0\.0\.24/0.99.22/'

	Wander Lairson Costa
	Comment 2 • 8 years ago

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.

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 3 • 8 years ago

Patch fixed based on comment.

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 4 • 8 years ago

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

	Wander Lairson Costa
	Comment 5 • 8 years ago

Could you please run a try push?

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 6 • 8 years ago

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.

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 7 • 8 years ago

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

	Wander Lairson Costa
	Comment 8 • 8 years ago

(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?

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 9 • 8 years ago

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?

	Wander Lairson Costa
	Comment 10 • 8 years ago

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.

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 11 • 8 years ago

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

	Wander Lairson Costa
	Comment 12 • 8 years ago

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

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 13 • 8 years ago

(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

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 14 • 8 years ago

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

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

	Wander Lairson Costa
	Comment 15 • 8 years ago

(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...

	Wander Lairson Costa
	Comment 16 • 8 years ago

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

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 17 • 8 years ago

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

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

	Pulsebot
	Comment 18 • 8 years ago

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

	Carsten Book [:Tomcat]
	Comment 19 • 8 years ago
bugherder

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

	Carsten Book [:Tomcat]
	Comment 20 • 8 years ago

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

	Greg Arndt [:garndt]
	Comment 21 • 8 years ago

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.

	Pulsebot
	Comment 22 • 8 years ago

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

	Wander Lairson Costa
	Comment 23 • 8 years ago

(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?

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 24 • 8 years ago

Removed empty scope.

	Wander Lairson Costa
	Comment 25 • 8 years ago

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?

	Naoki Hirata :nhirata (please use needinfo instead of cc) Assignee
	Comment 26 • 8 years ago

ya.  you're right.

	Wander Lairson Costa
	Comment 27 • 8 years ago

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

	Wander Lairson Costa
	Comment 29 • 8 years ago

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

	Pulsebot
	Comment 30 • 8 years ago

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

	Carsten Book [:Tomcat]
	Comment 31 • 8 years ago
bugherder

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