Feature #1192

"notify" should allow for "undef"

Added by Tim Stoop about 4 years ago. Updated about 4 years ago.

Status:Closed Start date:
Priority:Normal Due date:
Assignee:Puppet Community % Done:

0%
Category:-
Target version:-
Affected Puppet version:0.25.4 Branch:
Keywords:
Votes: 0

Description

I tried a definition like the following (leaving out the non-important bits):

define configfile(. . ., $notify = undef) {
        file { $name:
                . . .
                notify => $notify,
                . . .
        }
}

However, when I haven’t given a notify in my usage of this definition, I get the error:

err: Could not create /etc/phppgadmin/config.inc.php: Parameter notify failed: Relationships must be resource references at /srv/puppet/prod/common/manifests/functions.pp:24

warning: Not using cache on failed catalog

warning: Configuration could not be instantiated: Parameter notify failed: Relationships must be resource references at /srv/puppet/prod/common/manifests/functions.pp:24

Any tips on how to do this correctly? Or is this something that might need some enhancement?

History

#1
Updated by Peter Meier about 4 years ago

  • Status changed from 1 to Closed
  • 7 set to wontfix

http://groups.google.com/group/puppet-users/browse_thread/thread/c28aa647e774cc84/0fd4387d029e7e38

Please use the mailinglist for the discussion.

Also available in: Atom PDF