Bug 149605 - Bring El Capitan buildbot infrastructure online
Summary: Bring El Capitan buildbot infrastructure online
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P1 Normal
Assignee: Lucas Forschler
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2015-09-28 15:00 PDT by Lucas Forschler
Modified: 2015-09-30 09:36 PDT (History)
4 users (show)

See Also:


Attachments
Bring El Capitan buildbot infrastructure online (10.54 KB, patch)
2015-09-29 12:57 PDT, Lucas Forschler
no flags Details | Formatted Diff | Diff
Add El Capitan OpenSource buildbot configuration (10.54 KB, patch)
2015-09-29 13:22 PDT, Lucas Forschler
lforschler: commit-queue+
Details | Formatted Diff | Diff
El Capitan Buildbot Config plus unittest (15.65 KB, patch)
2015-09-29 16:49 PDT, Lucas Forschler
ap: review+
ap: commit-queue-
Details | Formatted Diff | Diff
El Capitan buildbot configuration (15.65 KB, patch)
2015-09-30 03:36 PDT, Lucas Forschler
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Lucas Forschler 2015-09-28 15:00:16 PDT
Prepare for the future by adding El Capitan infrastructure.
We should not commit and update the master until El Capitan is available.
Comment 1 Radar WebKit Bug Importer 2015-09-28 15:02:20 PDT
<rdar://problem/22886617>
Comment 2 Lucas Forschler 2015-09-29 12:57:16 PDT
Created attachment 262094 [details]
Bring El Capitan buildbot infrastructure online

We will update the buildbot master and restart it once Capitan is available to the world.
Comment 3 Lucas Forschler 2015-09-29 13:17:38 PDT
Comment on attachment 262094 [details]
Bring El Capitan buildbot infrastructure online

View in context: https://bugs.webkit.org/attachment.cgi?id=262094&action=review

notes from Alexey

> Tools/BuildSlaveSupport/build.webkit.org-config/config.json:87
> +                      "triggers": ["elcapitan-debug-tests-jsc", "elcapitan-debug-tests-wk1", "elcapitan-debug-tests-wk2", "elcapitan-leaks"],

ap: make this run on release instead of debug

> Tools/BuildSlaveSupport/build.webkit.org-config/config.json:109
> +                      "platform": "mac-elcapitan", "configuration": "debug", "architectures": ["x86_64"],

ap: should be 32 bit arch

> Tools/BuildSlaveSupport/build.webkit.org-config/config.json:137
> +                      "platform": "mac-elcapitan", "configuration": "debug", "architectures": ["x86_64"],

ap: make this release
Comment 4 Lucas Forschler 2015-09-29 13:22:27 PDT
Created attachment 262097 [details]
Add El Capitan OpenSource buildbot configuration

Updates from feedback
Comment 5 Csaba Osztrogonác 2015-09-29 14:20:42 PDT
(In reply to comment #4)
> Created attachment 262097 [details]
> Add El Capitan OpenSource buildbot configuration
> 
> Updates from feedback

Please add expected results for these new bots 
in mastercfg_unittest.py - expected_build_steps
Comment 6 Csaba Osztrogonác 2015-09-29 14:24:57 PDT
Comment on attachment 262097 [details]
Add El Capitan OpenSource buildbot configuration

View in context: https://bugs.webkit.org/attachment.cgi?id=262097&action=review

> Tools/BuildSlaveSupport/build.webkit.org-config/config.json:116
> +                    { "name": "Apple El Capitan Release (Build)", "type": "Build", "builddir": "elcapitan-release",
> +                      "platform": "mac-elcapitan", "configuration": "release", "architectures": ["x86_64"],
> +                      "triggers": ["elcapitan-release-tests-jsc", "elcapitan-release-tests-wk1", "elcapitan-release-tests-wk2", "elcapitan-release-perf", "elcapitan-leaks"],
> +                      "slavenames": ["bot417", "bot418"]
> +                    },

The leak bot should be triggered by the debug builder instead of the release one.
Comment 7 Alexey Proskuryakov 2015-09-29 14:32:35 PDT
> The leak bot should be triggered by the debug builder instead of the release one.

Can you elaborate? Leaks tests should work fine in release Mac builds, thanks to bmalloc disabling itself (although I haven't tried that).
Comment 8 Lucas Forschler 2015-09-29 16:49:30 PDT
Created attachment 262121 [details]
El Capitan Buildbot Config plus unittest

Added unit tests for El Capitan.
Comment 9 Alexey Proskuryakov 2015-09-29 17:17:02 PDT
Comment on attachment 262121 [details]
El Capitan Buildbot Config plus unittest

View in context: https://bugs.webkit.org/attachment.cgi?id=262121&action=review

> Tools/BuildSlaveSupport/build.webkit.org-config/config.json:137
> +                      "platform": "mac-elcapitan", "configuration": "debug", "architectures": ["x86_64"],

Should be configuration: release.
Comment 10 Lucas Forschler 2015-09-30 03:36:42 PDT
Created attachment 262150 [details]
El Capitan buildbot configuration

Fix debug->release for Leaks
Comment 11 Csaba Osztrogonác 2015-09-30 03:47:05 PDT
(In reply to comment #7)
> > The leak bot should be triggered by the debug builder instead of the release one.
> 
> Can you elaborate? Leaks tests should work fine in release Mac builds,
> thanks to bmalloc disabling itself (although I haven't tried that).

I just saw the Yosemite configuration of the bots and the leak bot is 
triggered by the debug builder. I thought we need the same configuration.
Comment 12 WebKit Commit Bot 2015-09-30 09:36:24 PDT
Comment on attachment 262150 [details]
El Capitan buildbot configuration

Clearing flags on attachment: 262150

Committed r190348: <http://trac.webkit.org/changeset/190348>
Comment 13 WebKit Commit Bot 2015-09-30 09:36:29 PDT
All reviewed patches have been landed.  Closing bug.