1265799 - Disable b2g-inbound

Status: RESOLVED FIXED

(Infrastructure & Operations Graveyard :: CIDuty, task)

Description

[Chris AtLee [:catlee]]

b2g-inbound is no longer needed for development. Let's disable all buildbot automation running there.

Comment 1

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

Attachment: bug1265799_buildbot-configs.patch

Comment 2

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

Attachment: bug1265799_mozharness.patch

Comment 3

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

Attachment: bug1265799_puppet.patch

Comment 4

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

Attachment: bug1265799_tools.patch

Comment 5

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

I tested the changes on dev-master2 by running the test-master.sh and script run without issues

Comment 6

[Kim Moir [:kmoir] ET]

Comment on attachment 8743324 [details] [diff] [review]
bug1265799_mozharness.patch

this patch looks good, however, you might want to check in-tree (mozilla-inbound) mozharness configs for references to b2g-inbound like you did for the other bug

Comment 7

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

Attachment: bug1265799_braindump.patch

Comment 8

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

Attachment: bug1265799_mozilla-inbound.patch

Comment 9

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8743740 [details] [diff] [review]
bug1265799_mozilla-inbound.patch

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

I am not familiar with this code, technically I don't see anything wrong with it, let me pass to review to callek

Comment 10

[Justin Wood (:Callek)]

Comment on attachment 8743740 [details] [diff] [review]
bug1265799_mozilla-inbound.patch

I've never touched either file, so had to brush up a bit, but looks good to me.

Comment 11

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

(In reply to Justin Wood (:Callek) from comment #10)
> Comment on attachment 8743740 [details] [diff] [review]
> bug1265799_mozilla-inbound.patch
> 
> I've never touched either file, so had to brush up a bit, but looks good to
> me.

I run the change on try https://treeherder.mozilla.org/#/jobs?repo=try&revision=6a72b94920a9d3b81423a488ebda28c6c5afde79

Comment 12

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

(In reply to Vlad Ciobancai [:vladC] from comment #11)
> (In reply to Justin Wood (:Callek) from comment #10)
> > Comment on attachment 8743740 [details] [diff] [review]
> > bug1265799_mozilla-inbound.patch
> > 
> > I've never touched either file, so had to brush up a bit, but looks good to
> > me.
> 
> I run the change on try
> https://treeherder.mozilla.org/#/
> jobs?repo=try&revision=6a72b94920a9d3b81423a488ebda28c6c5afde79

The attachment 8743740 [details] [diff] [review] have been tested on try and from what I can see everything run OK, can you please validate that I'm correct or not ? 

If the patch is OK can somebody help me to land it to m-i ?

Comment 13

[Pulsebot]

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

Comment 14

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

kmoir I think the rest of patches can be landed , right ?

Comment 15

[Kim Moir [:kmoir] ET]

Yes, they look good

Comment 16

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

jmaher thank you for your help also I land the rest of the patches.

Comment 17

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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

Comment 18

[hwine]

changes to mozharness were not passing tests

Comment 19

[hwine]

https://hg.mozilla.org/build/mozharness/rev/449a5bc0876875c7721218586cf9175a49a520a6
Bug 1265799 - Disable b2g-inbound broke mozharness; r=me

Comment 20

[hwine]

changes in comment 19 backed out the changes to beagle.py. As I noted in another bug, any changes to vcs-sync should occur in their own bug, as it is handled by a team-other-than-releng. As with the prior case, there is nothing to be done to vcs-sync until the repo is deleted.

Please make sure you can run the tests locally on mozharness repo, and watch for the messages in #releng after commit. Both methods will help ensure you don't leave failing tests committed. 

If all the work for this bug is still functional, you can close it again.

Comment 21

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

From what I can see yes, maybe kmoir can confirm

Comment 22

[Kim Moir [:kmoir] ET]

I don't understand what you are asking me to confirm.  Hal mentioned that vcs-sync shouldn't be changed because it's near end of life, so I think we are good.

Comment 23

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

Thank you Kim, I will close the bug