Closed Bug 1094288 Opened 7 years ago Closed 7 years ago

Remove references to Mavericks/10.9 from releng code

Categories

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

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: coop, Assigned: coop)

References

Details

(Whiteboard: [10.9][mac][cleanup])

Attachments

(3 files)

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
Blocks: 1079468
Whiteboard: [10.9][mac][cleanup]
Assignee: nobody → coop
Status: NEW → ASSIGNED
Priority: -- → P2
A Pivotal Tracker story has been created for this Bug: https://www.pivotaltracker.com/story/show/82319574
Duplicate of this bug: 1098269
Attachment #8524930 - Attachment description: bug1094288_replace_mavericks_configs.diff → [buildbot-configs] Replace mavericks with yosemite
Machine entries exist in graphserver (staging and prod) and in slavealloc.

Puppet work was covered by bug 1078882.
See Also: → 1094293
(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 on attachment 8524931 [details] [diff] [review]
[tools] Replace mavericks with yosemite

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

lgtm :)
Attachment #8524931 - Flags: review?(jlund) → review+
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
Attachment #8524930 - Flags: review?(jlund) → review+
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
Attachment #8524931 - Flags: checked-in+
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
Attachment #8524930 - Flags: checked-in+
Attachment #8526166 - Flags: review?(jlund) → review+
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
Attachment #8526166 - Flags: checked-in+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Component: Platform Support → Buildduty
Product: Release Engineering → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.