Puppet: System Administration Automated

Support

Ticket #1192 (closed enhancement: wontfix)

Opened 6 months ago

Last modified 6 months ago

"notify" should allow for "undef"

Reported by: tim Assigned to: community
Priority: normal Milestone:
Component: library Version: 0.24.4
Severity: normal Keywords: notify undef
Cc: Triage Stage: Unreviewed
Attached Patches: None Complexity: Unknown

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?

Change History

04/16/08 13:17:28 changed by immerda

  • status changed from new to closed.
  • resolution set to wontfix.