1254549 - Disable b2g-ota automation

Status: RESOLVED FIXED

(Infrastructure & Operations Graveyard :: CIDuty, task)

Description

[Chris AtLee [:catlee]]

This includes all buildbot support, b2g bumper, and any other automation.

Comment 1

[Vlad Ciobancai [:vladC] [:vciobancai]]

Attachment: bug1254549_buildbot-configs.patch

Attached you can find the patch to disable b2g-ota from buildbot configs repository

Comment 2

[Vlad Ciobancai [:vladC] [:vciobancai]]

Attachment: bug1254549_output

Attached the differences

Comment 3

[Vlad Ciobancai [:vladC] [:vciobancai]]

Attachment: bug1254549_production-branches.patch

Attached you can find the patch that remove b2g-ota from production-branches

Comment 4

[Vlad Ciobancai [:vladC] [:vciobancai]]

Attachment: bug1254549_buildbotcustom_common.patch

Comment 5

[Vlad Ciobancai [:vladC] [:vciobancai]]

Attachment: bug1254549_test-master.output

Attached you can find the output for test-master.sh script 

Kim I would like to ask you help about the errors that I'm receiving

Comment 6

[Vlad Ciobancai [:vladC] [:vciobancai]]

(In reply to Vlad Ciobancai [:vladC] from comment #5)
> Created attachment 8729536 [details]
> bug1254549_test-master.output
> 
> Attached you can find the output for test-master.sh script 
> 
> Kim I would like to ask you help about the errors that I'm receiving

I run the script without my changes (like a buildbotmaster fresh) and I'm still receiving the same error.

Comment 7

[Kim Moir [:kmoir] ET]

It looks like you need to disable the builds for b2g-ota too, not just the tests.  There are still references in mozilla/config.py on your master.

catlee also mentions in the intial bug description the b2g bumper support needs to be removed too
https://wiki.mozilla.org/ReleaseEngineering/Applications/Bumper

Comment 8

[Kim Moir [:kmoir] ET]

not sure what is causing the problem on a fresh master, let me look

Comment 9

[Vlad Ciobancai [:vladC] [:vciobancai]]

(In reply to Kim Moir [:kmoir] from comment #7)
> It looks like you need to disable the builds for b2g-ota too, not just the
> tests.  There are still references in mozilla/config.py on your master.


I think when you checked my master the buildbot-configs have been reverted to test the test-master.sh with a fresh buildbotmaster. The attachment 8729518 [details] [diff] [review] contains the change

Comment 10

[Vlad Ciobancai [:vladC] [:vciobancai]]

Attachment: bug1254549_test-master.output

After I re-created the master and applied again the patches , the tests passed (attached the output from my master)

Comment 11

[Vlad Ciobancai [:vladC] [:vciobancai]]

Attachment: bug1254549_mozharness.patch

Attached you can find the patch where I removed b2g-ota from bumper

@Kim do you know if there;s anything else to remove. I searched for b2g-ota but I didn't found 

The patches have been tested and the output is attached as attachment 8732891 [details]

Comment 12

[Kim Moir [:kmoir] ET]

Comment on attachment 8732935 [details] [diff] [review]
bug1254549_mozharness.patch

I think this is okay but adding hwine as a reviewer since is more familiar with the bumper code

Comment 13

[hwine]

Comment on attachment 8732935 [details] [diff] [review]
bug1254549_mozharness.patch

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

Nothing in config/vcssync should be touched for this bug, so r- on beagle.py change (We won't undo that until the b2g-ota repositories are deleted.)

The b2g_bumper config removal is proper. The b2g_bumper driver script iterates over all *.py files in b2g_bumper/config, so deletion of the file will do what is intended (stop bumping b2g-ota changes). r+ on the b2g-ota.py change

Please resubmit with just the b2g-ota.py removal for a quick review.

Comment 14

[Vlad Ciobancai [:vladC] [:vciobancai]]

Attachment: bug1254549_mozharness.patch

Updated the patch using the suggestion that you provided

Comment 15

[hwine]

Comment on attachment 8733271 [details] [diff] [review]
bug1254549_mozharness.patch

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

lgtm - thanks!

Comment 16

[Vlad Ciobancai [:vladC] [:vciobancai]]

Attachment: bug1254549_buildbot-configs.patch

I re-created the patch in order to remove the conflicts

Comment 17

[Vlad Ciobancai [:vladC] [:vciobancai]]

Changes pushed in production