782826 - Make TBPL recognise OS X 10.8

Status: RESOLVED FIXED

(Tree Management Graveyard :: TBPL, defect)

Description

[Ed Morley [:emorley]]

Attachment: Patch v1

Bug 777759 has added OS X Mountain Lion (10.8) support. The builds are currently showing up under 'OS X' instead of what we would prefer: 'OS X 10.8'

Patch adds support for 10.8, including the extra regex for the way Jetpack names the jobs.

Works locally (though wasn't able to test for Jetpack, since none scheduled yet).

Comment 1

[Phil Ringnalda (:philor)]

Comment on attachment 651924 [details] [diff] [review]
Patch v1

If we ever had to type OSNames, I'd vote for mtnlion, but I think after these patches, we never type or touch them again.

Comment 2

[Ed Morley [:emorley]]

(Switched to osxmtnlion)

http://hg.mozilla.org/users/mstange_themasta.com/tinderboxpushlog/rev/e3bc3562965f

Comment 3

[Ed Morley [:emorley]]

Kim, adding you so this step can be included in the 'how to add a new platform' document mentioned in bug 777759 comment 17 :-)

Comment 4

[Kim Moir [:kmoir] ET]

Regarding comment #3, the documentation has been updated.

https://wiki.mozilla.org/ReferencePlatforms/HowToSetupNewPlatform

Comment 5

[Ed Morley [:emorley]]

Thanks Kim! :-)

Comment 6

[Kim Moir [:kmoir] ET]

Should tbpl show 10.8 tests running now, I don't seem them.

Comment 7

[Ed Morley [:emorley]]

It does, but most/all will be hidden due to being busted, unless you use &noignore=1. Also, given the shortage of 10.8 machines, not many trees have been able to run tests on 10.8 (priority is roughly beta, aurora, m-c, inbound; give or take):
https://tbpl.mozilla.org/?noignore=1&tree=Mozilla-Aurora&jobname=10.8

Comment 8

[Phil Ringnalda (:philor)]

Actually, the shortage and priority thing was my wrong guess - the problem is that your landing raced with the patch that shut off 10.5, so on any branch where we don't do 10.5 tests we're currently clobbering the mountainlion out of slave_platforms, until the latest patch on bug 777759 lands. Sort of curious how aurora is stacking them up despite not having slave_platforms, and trunk isn't, though.