CrOS: Builder names == target config names.

CrOS: Builder names == target config names.

BUG=chromium:591117
TEST=None

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299039

[dgarrett, 2016-03-01 21:41:35]

dgarrett@chromium.org changed reviewers:
+ dgarrett@chromium.org

[dgarrett, 2016-03-01 21:41:35]

LGTM, with one important question.

Is there any way this can affect master/slave communication?

Is there any reason that this is hard to revert (other than the waterfall
restart) if it causes problems?

[dnj, 2016-03-01 21:43:46]

On 2016/03/01 21:41:35, dgarrett wrote:
> LGTM, with one important question.
> 
> Is there any way this can affect master/slave communication?

I doubt it. The master/slave communication already uses the config name.

> Is there any reason that this is hard to revert (other than the waterfall
> restart) if it causes problems?

None that I can think of.

[dnj, 2016-03-01 21:43:59]

> > LGTM, with one important question.

(Oh that's two questions BTW) :)

[dgarrett, 2016-03-01 21:45:01]

On 2016/03/01 21:43:59, dnj wrote:
> > > LGTM, with one important question.
> 
> (Oh that's two questions BTW) :)

Hold off until I get a Chromite name removal CL ready.

[dgarrett, 2016-03-02 00:24:12]

On 2016/03/01 21:45:01, dgarrett wrote:
> On 2016/03/01 21:43:59, dnj wrote:
> > > > LGTM, with one important question.
> > 
> > (Oh that's two questions BTW) :)
> 
> Hold off until I get a Chromite name removal CL ready.

Which doesn't appear to be needed.

[dnj, 2016-03-02 00:57:10]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2016-03-02 00:57:15]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1757553002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1757553002/1

[commit-bot: I haz the power, 2016-03-02 01:01:16]

Description was changed from

==========
CrOS: Builder names == target config names.

BUG=chromium:591117
TEST=None
==========

to

==========
CrOS: Builder names == target config names.

BUG=chromium:591117
TEST=None

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299039
==========

[commit-bot: I haz the power, 2016-03-02 01:01:17]

Committed patchset #1 (id:1) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=299039