Rename any builder with prefix 'WebKit Linux' to 'WebKit Linux Precise'

Rename 'WebKit Linux' to 'WebKit Linux Precise'

Note: this is 1 of 3 CLs in different repos:

1. build CLs split up:
  - master: https://codereview.chromium.org/2382863002/
  - slaves: https://codereview.chromium.org/2382723004/
2. https://codereview.chromium.org/2347293003 (chromium) [this one]

(Considering not landing this one)
3. https://chromium-review.googlesource.com/386326 (infra)

BUG=613305
TBR=dpranke@chromium.org

Committed: https://crrev.com/520e37530ee98b49d0b58573212e67d4f9ea67bb
Cr-Commit-Position: refs/heads/master@{#422581}

[qyearsley, 2016-09-16 22:16:10]

qyearsley@chromium.org changed reviewers:
+ dpranke@chromium.org, qyearsley@chromium.org

[qyearsley, 2016-09-16 22:16:11]

Can't see any problems; I assume you searched for all instances of "WebKit
Linux" being used, these are the instances in src?

[Dirk Pranke, 2016-09-16 22:39:57]

Almost good, but see my important comments on
https://codereview.chromium.org/2338293005/ .

https://codereview.chromium.org/2347293003/diff/1/third_party/WebKit/Tools/Sc...
File
third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/update_test_expectations_unittest.py
(right):

https://codereview.chromium.org/2347293003/diff/1/third_party/WebKit/Tools/Sc...
third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/update_test_expectations_unittest.py:38:
'WebKit Linux Precise' : {
Ideally we'd change these test cases to refer to a totally fake builder, so that
we didn't have to change the tests when we change real revisions.

[jeffcarp, 2016-09-21 18:34:17]

Description was changed from

==========
Rename 'WebKit Linux' to 'WebKit Linux Precise'

BUG=613305
==========

to

==========
Rename 'WebKit Linux' to 'WebKit Linux Precise'

Note: this is 1 of 3 CLs in different repos:

1. https://codereview.chromium.org/2338293005 (build)
2. https://codereview.chromium.org/2347293003 (chromium) [this one]
3. https://chromium-review.googlesource.com/386326 (infra)

BUG=613305
==========

[jeffcarp, 2016-09-21 19:21:50]

https://codereview.chromium.org/2347293003/diff/1/third_party/WebKit/Tools/Sc...
File
third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/update_test_expectations_unittest.py
(right):

https://codereview.chromium.org/2347293003/diff/1/third_party/WebKit/Tools/Sc...
third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/update_test_expectations_unittest.py:38:
'WebKit Linux Precise' : {
On 2016/09/16 at 22:39:57, Dirk Pranke wrote:
> Ideally we'd change these test cases to refer to a totally fake builder, so
that we didn't have to change the tests when we change real revisions.

That sounds good but sounds like it should be in a separate CL. Should I revert
this file or leave it as-is so it matches the new WebKit Precise bots?

[Dirk Pranke, 2016-09-21 21:17:14]

On 2016/09/21 19:21:50, jeffcarp wrote:
>
https://codereview.chromium.org/2347293003/diff/1/third_party/WebKit/Tools/Sc...
> File
>
third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/update_test_expectations_unittest.py
> (right):
> 
>
https://codereview.chromium.org/2347293003/diff/1/third_party/WebKit/Tools/Sc...
>
third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/update_test_expectations_unittest.py:38:
> 'WebKit Linux Precise' : {
> On 2016/09/16 at 22:39:57, Dirk Pranke wrote:
> > Ideally we'd change these test cases to refer to a totally fake builder, so
> that we didn't have to change the tests when we change real revisions.
> 
> That sounds good but sounds like it should be in a separate CL. Should I
revert
> this file or leave it as-is so it matches the new WebKit Precise bots?

Matching the new stuff is probably better.

[qyearsley, 2016-09-29 20:31:46]

The CL description here could be updated now that CL # 1 is closed in favor of
these two:
  https://codereview.chromium.org/2382863002 (build/masters/...)
  https://codereview.chromium.org/2382723004 (build/scripts/slave/...)

https://codereview.chromium.org/2347293003/diff/1/tools/mb/mb_config.pyl
File tools/mb/mb_config.pyl (right):

https://codereview.chromium.org/2347293003/diff/1/tools/mb/mb_config.pyl#newc...
tools/mb/mb_config.pyl:263: 'WebKit Linux Precise - WPTServe':
'swarming_gn_release_bot',
Now these two lines can be reverted to how they were.

[qyearsley, 2016-09-29 20:32:57]

On 2016/09/29 at 20:31:46, qyearsley wrote:
> The CL description here could be updated now that CL # 1 is closed in favor of
these two:
>   https://codereview.chromium.org/2382863002 (build/masters/...)
>   https://codereview.chromium.org/2382723004 (build/scripts/slave/...)
> 

Also: the CL title/description could note that (a) multiple builders are being
renamed (b) this CL should include all changes in the src repo.

[jeffcarp, 2016-09-29 22:32:13]

Description was changed from

==========
Rename 'WebKit Linux' to 'WebKit Linux Precise'

Note: this is 1 of 3 CLs in different repos:

1. https://codereview.chromium.org/2338293005 (build)
2. https://codereview.chromium.org/2347293003 (chromium) [this one]
3. https://chromium-review.googlesource.com/386326 (infra)

BUG=613305
==========

to

==========
Rename 'WebKit Linux' to 'WebKit Linux Precise'

Note: this is 1 of 3 CLs in different repos:

1. build CLs split up:
  - master: https://codereview.chromium.org/2382863002/
  - slaves: https://codereview.chromium.org/2382723004/
2. https://codereview.chromium.org/2347293003 (chromium) [this one]

(Considering not landing this one)
3. https://chromium-review.googlesource.com/386326 (infra)

BUG=613305
==========

[qyearsley, 2016-09-30 17:42:34]

The CQ bit was checked by qyearsley@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-30 17:42:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-30 17:46:08]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-30 17:46:08]

Dry run: Try jobs failed on following builders:
  android_n5x_swarming_rel on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[qyearsley, 2016-09-30 17:48:25]

On 2016/09/29 at 20:32:57, qyearsley wrote:
> On 2016/09/29 at 20:31:46, qyearsley wrote:
> > The CL description here could be updated now that CL # 1 is closed in favor
of these two:
> >   https://codereview.chromium.org/2382863002 (build/masters/...)
> >   https://codereview.chromium.org/2382723004 (build/scripts/slave/...)
> > 
> 
> Also: the CL title/description could note that (a) multiple builders are being
renamed (b) this CL should include all changes in the src repo.

LGTM -- Note, I believe that the WebKit Linux Precise builders are currently red
because mb_config.pyl needs to be updated; I expect that they should be green
after landing this. This CL needs LGTM from Dirk as well before passing the
OWNERS check.

[jeffcarp, 2016-09-30 17:54:35]

The CQ bit was checked by jeffcarp@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-30 17:55:13]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-30 17:58:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-30 17:59:00]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[jeffcarp, 2016-09-30 18:06:08]

The CQ bit was checked by jeffcarp@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-30 18:06:55]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jeffcarp, 2016-09-30 18:19:24]

Description was changed from

==========
Rename 'WebKit Linux' to 'WebKit Linux Precise'

Note: this is 1 of 3 CLs in different repos:

1. build CLs split up:
  - master: https://codereview.chromium.org/2382863002/
  - slaves: https://codereview.chromium.org/2382723004/
2. https://codereview.chromium.org/2347293003 (chromium) [this one]

(Considering not landing this one)
3. https://chromium-review.googlesource.com/386326 (infra)

BUG=613305
==========

to

==========
Rename 'WebKit Linux' to 'WebKit Linux Precise'

Note: this is 1 of 3 CLs in different repos:

1. build CLs split up:
  - master: https://codereview.chromium.org/2382863002/
  - slaves: https://codereview.chromium.org/2382723004/
2. https://codereview.chromium.org/2347293003 (chromium) [this one]

(Considering not landing this one)
3. https://chromium-review.googlesource.com/386326 (infra)

BUG=613305
TBR=dpranke@chromium.org
==========

[jeffcarp, 2016-09-30 18:20:27]

The CQ bit was unchecked by jeffcarp@chromium.org

[jeffcarp, 2016-09-30 18:20:30]

The CQ bit was checked by jeffcarp@chromium.org

[jeffcarp, 2016-09-30 18:20:30]

The patchset sent to the CQ was uploaded after l-g-t-m from
qyearsley@chromium.org
Link to the patchset: https://codereview.chromium.org/2347293003/#ps20001
(title: "Rename 'WebKit Linux' to 'WebKit Linux Precise'")

[commit-bot: I haz the power, 2016-09-30 18:21:08]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-30 19:26:05]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-30 19:26:07]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jeffcarp, 2016-09-30 19:47:16]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-09-30 19:48:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jeffcarp, 2016-09-30 19:49:08]

The CQ bit was unchecked by jeffcarp@chromium.org

[jeffcarp, 2016-09-30 20:10:07]

The CQ bit was checked by jeffcarp@chromium.org

[jeffcarp, 2016-09-30 20:10:07]

The patchset sent to the CQ was uploaded after l-g-t-m from
qyearsley@chromium.org
Link to the patchset: https://codereview.chromium.org/2347293003/#ps40001
(title: "Update more recent addition to WebKit Precise")

[commit-bot: I haz the power, 2016-09-30 20:10:28]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-30 23:16:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-30 23:16:49]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jeffcarp, 2016-09-30 23:17:56]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-09-30 23:18:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-01 01:27:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-01 01:27:58]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jeffcarp, 2016-10-01 01:30:01]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-10-01 01:30:25]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Dirk Pranke, 2016-10-01 02:26:18]

lgtm

[commit-bot: I haz the power, 2016-10-01 03:07:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-01 03:07:23]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jeffcarp, 2016-10-01 03:08:16]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-10-01 03:08:34]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-01 04:35:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-01 04:35:53]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jeffcarp, 2016-10-01 04:37:02]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-10-01 04:37:10]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-01 06:12:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-01 06:12:51]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jeffcarp, 2016-10-01 06:17:46]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-10-01 06:17:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-01 09:28:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-01 09:28:41]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jeffcarp, 2016-10-01 19:45:44]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-10-01 19:45:55]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-01 21:55:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-01 21:55:55]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[jeffcarp, 2016-10-01 22:01:45]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-10-01 22:01:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-01 22:23:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-01 22:23:26]

Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jeffcarp, 2016-10-01 23:03:13]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-10-01 23:03:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-02 00:06:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-02 00:06:49]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jeffcarp, 2016-10-02 00:17:28]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-10-02 00:17:40]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-02 01:12:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-02 01:12:59]

Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[jeffcarp, 2016-10-02 01:13:22]

The CQ bit was checked by jeffcarp@chromium.org

[commit-bot: I haz the power, 2016-10-02 01:13:37]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-02 01:56:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-02 01:56:28]

Exceeded global retry quota

[qyearsley, 2016-10-02 17:27:28]

CQ is failing because some tests are failing at ToT; this is blocked on
http://crrev.com/652014.

[qyearsley, 2016-10-03 15:51:58]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-10-03 15:52:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-03 16:59:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-03 16:59:28]

Exceeded global retry quota

[qyearsley, 2016-10-03 21:16:46]

On 2016/10/03 at 16:59:28, commit-bot wrote:
> Exceeded global retry quota

Last remaining CQ failure (mac_chromium_rel_ng) *still* looks unrelated to this
CL.

[qyearsley, 2016-10-03 22:51:34]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2016-10-03 22:52:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-03 22:58:49]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-03 23:03:19]

Description was changed from

==========
Rename 'WebKit Linux' to 'WebKit Linux Precise'

Note: this is 1 of 3 CLs in different repos:

1. build CLs split up:
  - master: https://codereview.chromium.org/2382863002/
  - slaves: https://codereview.chromium.org/2382723004/
2. https://codereview.chromium.org/2347293003 (chromium) [this one]

(Considering not landing this one)
3. https://chromium-review.googlesource.com/386326 (infra)

BUG=613305
TBR=dpranke@chromium.org
==========

to

==========
Rename 'WebKit Linux' to 'WebKit Linux Precise'

Note: this is 1 of 3 CLs in different repos:

1. build CLs split up:
  - master: https://codereview.chromium.org/2382863002/
  - slaves: https://codereview.chromium.org/2382723004/
2. https://codereview.chromium.org/2347293003 (chromium) [this one]

(Considering not landing this one)
3. https://chromium-review.googlesource.com/386326 (infra)

BUG=613305
TBR=dpranke@chromium.org

Committed: https://crrev.com/520e37530ee98b49d0b58573212e67d4f9ea67bb
Cr-Commit-Position: refs/heads/master@{#422581}
==========

[commit-bot: I haz the power, 2016-10-03 23:03:20]

Patchset 3 (id:??) landed as
https://crrev.com/520e37530ee98b49d0b58573212e67d4f9ea67bb
Cr-Commit-Position: refs/heads/master@{#422581}

[jeffcarp, 2016-10-03 23:32:00]

On 2016/10/03 at 23:03:20, commit-bot wrote:
> Patchset 3 (id:??) landed as
https://crrev.com/520e37530ee98b49d0b58573212e67d4f9ea67bb
> Cr-Commit-Position: refs/heads/master@{#422581}

YEEEEEEEESSSSSSSSSSSSSSS