No, if you remove the defined() call and just include it will fail to find
the class. Defined() is there simply to ensure that we can do the include.
For a bit more context I've written an abstraction layer that assigns
apps/roles to nodes and will pull in profiles if they exist, that is where
this snipit is from.
So changing this to:
if ($profile == 'existantclass') {
include "profile::${separator}::${profile}"
}
results in Error 400 on SERVER: Could not find class
profile::app::existantclass for node1.vnet on node node1.vnet.
Me and defined() have had a rocky but ultimately agreeable relationship.
On Tuesday, January 6, 2015 4:49:04 PM UTC-7, Felix.Frank wrote:
>
> On 12/30/2014 09:05 PM, Matthew Kennedy wrote:
>
>
> if (defined("profile::${separator}::${profile}")) {
> include "profile::${separator}::${profile}"
> }
>
>
> Hi,
>
> Is the if defined() construct significant? Can you only reproduce when it
> is in place? Why is it there and what does it do?
>
> Note that, regardless of this issue, using this function can be hazardous.
>
> Thanks,
> Felix
>
--
You received this message because you are subscribed to the Google Groups
"Puppet Users" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to puppet-users+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/puppet-users/efefacf2-5313-4020-bbe7-f91fc86ee694%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
