[Buildbot] #3223: detect builder and buildslave names that are not identifiers during configtest (was: raise exception early if name is not compatible to be an identifier)

[Buildbot trac]

#3223: detect builder and buildslave names that are not identifiers during
configtest
-----------------------+--------------------
Reporter:  yarikoptic  |       Owner:
    Type:  defect      |      Status:  new
Priority:  major       |   Milestone:  0.9.0
 Version:  0.8.10      |  Resolution:
Keywords:  simple      |
-----------------------+--------------------
Changes (by dustin):

 * keywords:   => simple
 * type:  undecided => defect
 * milestone:  undecided => 0.9.0


Old description:

> exceptions.ValueError: Buildslave name u'nd14.04' is not a 50-character
>              identifier
>
> because the name contained .  which is apparently illegit.  checkconfig
> was silent because name is not checked to be a valid identifier, and then
> only at run time puke happens

New description:

 > exceptions.ValueError: Buildslave name u'nd14.04' is not a 50-character
 identifier

 because the name contained `.`  which is apparently illegit.  checkconfig
 was silent because name is not checked to be a valid identifier, and then
 only at run time puke happens

--

--
Ticket URL: <http://trac.buildbot.net/ticket/3223#comment:1>
Buildbot <http://buildbot.net/>
Buildbot: build/test automation