Puppet: System Administration Automated

Support

Ticket #1016 (new defect)

Opened 10 months ago

Last modified 7 months ago

Mailalias type throws non-sane error when target contains an invalid record prior to managing it

Reported by: Fujin Assigned to: community
Priority: normal Milestone: unplanned
Component: mailalias Version: 0.24.1
Severity: trivial Keywords: mailalias syntax error split fileparsing
Cc: Triage Stage: Accepted
Attached Patches: None Complexity: Unknown

Description

When trying to instantiate a mailalias on an alias file that has a bogus record 

cat /etc/aliases:
root duncan.stewart@redzed.co.nz

err: Could not prefetch mailalias provider 'aliases': private method `split' called for :absent:Symbol
err: //Node[vps-redzed]/vps-systems/postfix::satellite/Mailalias[puppet]: Failed to retrieve current state of resource: private method `split' called for :absent:Symbol

have tracked this down to util/fileparsing.rb and/or type/mailalias.rb - I feel we should sanity check the 'target' for existing records, and fail cleanly (warning) with a sane error message if there are errors in the syntax - poking around with it now.

Change History

01/20/08 23:16:40 changed by Fujin

  • severity changed from normal to trivial.

02/05/08 07:20:31 changed by luke

  • stage changed from Unreviewed to Accepted.
  • milestone set to 0.24.2.

02/29/08 00:32:25 changed by luke

  • milestone changed from 0.24.2 to 0.24.3.

03/13/08 20:00:56 changed by luke

  • milestone changed from 0.24.3 to unplanned.

This is getting bumped indefinitely, probably until someone else provides a patch. I agree it's a problem, but the workaround is straightforward (fix your file so it's valid).

I just can't fix this problem in time for 0.24.3, and I'm more and more pushing this kind of problem to the community.

04/24/08 07:11:15 changed by luke

  • component changed from library to mailalias.