Bug 62372 - [CSSRegions] Add build slave
Summary: [CSSRegions] Add build slave
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-09 06:39 PDT by Mihnea Ovidenie
Modified: 2011-07-01 07:51 PDT (History)
4 users (show)

See Also:


Attachments
Patch (2.87 KB, patch)
2011-06-09 07:29 PDT, Mihnea Ovidenie
aroben: review-
Details | Formatted Diff | Diff
Patch 2 (2.91 KB, patch)
2011-06-22 04:33 PDT, Mihnea Ovidenie
aroben: review-
Details | Formatted Diff | Diff
Patch 3 (2.90 KB, patch)
2011-06-22 07:15 PDT, Mihnea Ovidenie
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Mihnea Ovidenie 2011-06-09 06:39:50 PDT
Add a Mac build slave.
Comment 1 Mihnea Ovidenie 2011-06-09 06:43:02 PDT
Machine has the following specs:
CPU: 3.2 GHz Quad-Core Xeon
RAM: 8Gb
OS: Snow Leopard
Comment 2 Mihnea Ovidenie 2011-06-09 07:29:24 PDT
Created attachment 96585 [details]
Patch
Comment 3 Eric Seidel (no email) 2011-06-13 15:10:16 PDT
Comment on attachment 96585 [details]
Patch

Sounds fine to me.
Comment 4 Eric Seidel (no email) 2011-06-13 15:10:37 PDT
Comment on attachment 96585 [details]
Patch

I'll let aroben give the official r+
Comment 5 Adam Roben (:aroben) 2011-06-21 08:31:26 PDT
Comment on attachment 96585 [details]
Patch

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

> Tools/BuildSlaveSupport/build.webkit.org-config/config.json:242
> +                    { "name": "Adobe SnowLeopard Intel Release (Build)", "type": "Build", "builddir": "adobe-snowleopard-intel-release",

We don't currently have any builders with company names in them. The important thing is the flavor of WebKit that runs on the builder and what kinds of operations it performs, not who maintains the slaves used by that builder. (In fact, you could have multiple different companies that maintain slaves that are used by a single builder.) I'd recommend giving this builder a name that that indicates its purpose.
Comment 6 Mihnea Ovidenie 2011-06-22 04:33:32 PDT
Created attachment 98156 [details]
Patch 2
Comment 7 Mihnea Ovidenie 2011-06-22 04:45:12 PDT
(In reply to comment #5)
> (From update of attachment 96585 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=96585&action=review
> 
> > Tools/BuildSlaveSupport/build.webkit.org-config/config.json:242
> > +                    { "name": "Adobe SnowLeopard Intel Release (Build)", "type": "Build", "builddir": "adobe-snowleopard-intel-release",
> 
> We don't currently have any builders with company names in them. The important thing is the flavor of WebKit that runs on the builder and what kinds of operations it performs, not who maintains the slaves used by that builder. (In fact, you could have multiple different companies that maintain slaves that are used by a single builder.) I'd recommend giving this builder a name that that indicates its purpose.

Hi Adam,

Thanks for your suggestions. We are thinking to use the following steps for this machine for the near future:
Step1: configure it like a SnowLeopard slave build bot to do Mac WebKit Builds
Step2: configure it like a SnowLeopard slave build bot to do Mac WebKit Builds with css-regions/css-exclusions enabled(they are disabled by default)
Step3: configure it like a SnowLeopard slave build bot to do Mac WebKit Builds and Tests with css-regions/css-exclusions enabled. This bot would also run css-regions/css-exclusions specific tests, that are skipped in WebKit main trunk.

Does this sound ok to you?
Comment 8 Adam Roben (:aroben) 2011-06-22 06:43:46 PDT
Comment on attachment 98156 [details]
Patch 2

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

> Tools/BuildSlaveSupport/build.webkit.org-config/config.json:255
> +                                       "CSSRegions SnowLeopard Intel Release (Build)"]

You should add your new builder to the mac-snowleopard PlatformSpecificScheduler. That way it won't do a build when, e.g., Qt layout test results get updated and nothing else changes.
Comment 9 Mihnea Ovidenie 2011-06-22 07:15:16 PDT
Created attachment 98165 [details]
Patch 3
Comment 10 WebKit Review Bot 2011-07-01 07:51:43 PDT
Comment on attachment 98165 [details]
Patch 3

Clearing flags on attachment: 98165

Committed r90228: <http://trac.webkit.org/changeset/90228>
Comment 11 WebKit Review Bot 2011-07-01 07:51:48 PDT
All reviewed patches have been landed.  Closing bug.