Merge branch 'no_alias_for_host' into 'master'

stop defining an alias by default for hosts

Since the "alias" attribute is special for puppet in that it needs to be
a string that's unique throughout all of the resources on a given host's
catalogue, we shouldn't always define an alias unless users really want
them.

This alias might (and did for some ppl) create a conflict when using a
class that is named the same as $::hostname.

We've just hit hit limitation on a Koumbit node. The alias attribute is legitimate in terms of nagios host attributes, but the name of the attribute clashes with the special-purpose attribute for puppet. So one might argue that the native puppet types are flawed, and they would be correct! but getting those types fixed would be a pain in the ass since they haven't been maintained for quite some time.

See merge request !10

0 files changed, 0 insertions, 0 deletions