1094288 - Remove references to Mavericks/10.9 from releng code

Status: RESOLVED FIXED

(Infrastructure & Operations Graveyard :: CIDuty, task, P2)

Description

[Chris Cooper [:coop] (he/him)]

We've decided to stop testing efforts on 10.9 in favor of 10.10. We should rip out the references to 10.9 everywhere.

* buildbot-configs
* buildbotcustom
* mozharness
* tools
* puppet
* slaveapi
* relengapi
* slave heath

Comment 1

[:kanban-engops]

A Pivotal Tracker story has been created for this Bug: https://www.pivotaltracker.com/story/show/82319574

Comment 3

[Chris Cooper [:coop] (he/him)]

Attachment: [buildbot-configs] Replace mavericks with yosemite

Comment 4

[Chris Cooper [:coop] (he/him)]

Attachment: [tools] Replace mavericks with yosemite

Comment 5

[Chris Cooper [:coop] (he/him)]

Machine entries exist in graphserver (staging and prod) and in slavealloc.

Puppet work was covered by bug 1078882.

Comment 6

[Jordan Lund (:jlund)]

(In reply to Chris Cooper [:coop] from comment #0)
> We've decided to stop testing efforts on 10.9 in favor of 10.10. We should
> rip out the references to 10.9 everywhere.
> 

> * mozharness
IIRC unittests/mac_unittest.py and talos/mac_config.py is pretty mac version agnostic. Mozharness should just work out of the box without change.

Comment 7

[Jordan Lund (:jlund)]

Comment on attachment 8524931 [details] [diff] [review]
[tools] Replace mavericks with yosemite

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

lgtm :)

Comment 8

[Jordan Lund (:jlund)]

Comment on attachment 8524930 [details] [diff] [review]
[buildbot-configs] Replace mavericks with yosemite

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

lgtm :)

::: mozilla-tests/BuildSlaves.py.template
@@ +22,5 @@
>      "tiger": "pass",
>      "snowleopard": "pass",
>      "mountainlion": "pass",
>      "mountainlion-b2gdt": "pass",
> +    "yosemite": "pass",

should we ping releng about dev-master BuildSlaves.py change once this lands?

::: mozilla-tests/config.py
@@ +1785,5 @@
>  
>  ######## cedar
>  BRANCHES['cedar']['platforms']['linux64-asan']['ubuntu64-asan_vm']['opt_unittest_suites'] += MARIONETTE[:]
> +BRANCHES['cedar']['platforms']['macosx64']['yosemite']['opt_unittest_suites'] = UNITTEST_SUITES['opt_unittest_suites'][:]
> +BRANCHES['cedar']['platforms']['macosx64']['yosemite']['debug_unittest_suites'] = UNITTEST_SUITES['debug_unittest_suites'][:]

I guess this is fine since IIUC, we have disabled mavericks everywhere. so this will create yosemite builders but like mavericks state now, they won't be scheduled ever?

@@ +1837,5 @@
>              slave_p['opt_unittest_suites'] += MOCHITEST_CSB
>              slave_p['debug_unittest_suites'] += MOCHITEST_CSB
>  
> +# Enable Yosemite testing on select branches only
> +delete_slave_platform(BRANCHES, PLATFORMS, {'macosx64': 'yosemite'}, branch_exclusions=['cedar'])

this is fine to leave as well for same reasons mentioned above?

::: mozilla-tests/production_config.py
@@ +30,5 @@
>  for i in range(1,87) + range(88,101):
>      SLAVES['mountainlion']['talos-mtnlion-r5-%03i' % i] = {}
>  
>  for i in range(1,4):
> +    SLAVES['yosemite']['t-yosemite-r5-%04i' % i] = {}

hehe so you didn't just %s/mavericks/yosemite/g ... nice 04 catch

Comment 9

[Chris Cooper [:coop] (he/him)]

Comment on attachment 8524931 [details] [diff] [review]
[tools] Replace mavericks with yosemite

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

https://hg.mozilla.org/build/tools/rev/ffa7da66a33a

Comment 10

[Chris Cooper [:coop] (he/him)]

Comment on attachment 8524930 [details] [diff] [review]
[buildbot-configs] Replace mavericks with yosemite

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

https://hg.mozilla.org/build/buildbot-configs/rev/307c650da946

Comment 11

[Chris Cooper [:coop] (he/him)]

Attachment: [puppet] Replace mavericks with yosemite

Comment 12

[Chris Cooper [:coop] (he/him)]

Comment on attachment 8526166 [details] [diff] [review]
[puppet] Replace mavericks with yosemite

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

https://hg.mozilla.org/build/puppet/rev/47e997baec07

Comment 13

[Chris Cooper [:coop] (he/him)]

In production: https://hg.mozilla.org/build/buildbot-configs/rev/307c650da946