1371776 - Create new build platform for OSX 10.10

Status: RESOLVED WONTFIX

(Release Engineering :: General, enhancement, P3)

Description

[Chris AtLee [:catlee]]

We need to support building both on 10.7 and 10.10 for some amount of time. This means we need a separate platform in buildbot for 10.10.

Initially we'll have builds available only on try, off by default.

Would 'macosx64-1010' work as a base platform name?

Comment 1

[Chris AtLee [:catlee]]

Attachment: Add 'macosx64-yosemite' to try

Comment 2

[Chris AtLee [:catlee]]

Attachment: patch for buildslaves.py on build masters

Comment 3

[Rail Aliiev [:rail]]

Comment on attachment 8876765 [details] [diff] [review]
Add 'macosx64-yosemite' to try

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

Probably you should skip one of the hunks, see below.

::: mozilla/BuildSlaves.py.template
@@ +7,5 @@
>          'win64-rev2': 'pass',
>          'try-win32': 'pass',
>          'macosx64': 'pass',
>          'macosx64-lion': 'pass',
> +        'macosx64-yosemite': 'pass',

Probably it's worth adding the new platform to https://hg.mozilla.org/build/buildbotcustom/file/tip/common.py#l56 as well

::: mozilla/config.py
@@ -1882,5 @@
> -PLATFORM_VARS["macosx64-lion-debug"] = deepcopy(PLATFORM_VARS["macosx64-debug"])
> -PLATFORM_VARS["macosx64-lion"]["base_name"] = 'OS X 10.7 %(branch)s'
> -PLATFORM_VARS["macosx64-lion-debug"]["base_name"] = 'OS X 10.7 64-bit %(branch)s leak test'
> -PLATFORM_VARS["macosx64-lion"]["slaves"] = SLAVES['macosx64-lion']
> -PLATFORM_VARS["macosx64-lion-debug"]["slaves"] = SLAVES['macosx64-lion']

I think we still need this for fuzzing. Not 100% sure, but it'd be safer to skip this hunk.

@@ -3006,5 @@
>          PROJECTS[branch_project_name] = project
>  
> -# ant test on try
> -## ant rides the trains (Bug 971841)
> -# for name, branch in items_before(BRANCHES, 'gecko_version', 30):

DIAF!

Comment 4

[Chris AtLee [:catlee]]

https://hg.mozilla.org/build/puppet/rev/2519e9e2f159c2ef3cc52320b7ae001869e4da9a
Bug 1371776: Add macosx64-yosemite to BuildSlaves.py r=rail

Comment 5

[Chris AtLee [:catlee]]

Attachment: Add 'macosx64-yosemite' to try

I just fixed up the naming scheme.

Confirmed that we don't need those hunks for fuzzing.

Comment 6

[Chris AtLee [:catlee]]

Comment on attachment 8876815 [details] [diff] [review]
Add 'macosx64-yosemite' to try

going to remove the debug builders

Comment 7

[Chris AtLee [:catlee]]

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

Comment 8

[Chris AtLee [:catlee]]

Attachment: Bug 1371776: Get OSX 10.10 builds working on try

Review commit: https://reviewboard.mozilla.org/r/148168/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/148168/

Comment 9

[Chris AtLee [:catlee]]

Attachment: Add 'macosx64-yosemite' to try

Comment 10

[Wander Lairson Costa]

Comment on attachment 8876840 [details]
Bug 1371776: Get OSX 10.10 builds working on try

https://reviewboard.mozilla.org/r/148168/#review152930

Comment 11

[Chris AtLee [:catlee]]

Back burner for now; we're hoping the TC cross-compiled builds work out.

Comment 12

[Chris AtLee [:catlee]]

We're going ahead with the cross-compiled builds.