Issue #3377 has been updated by James Turnbull.

Category set to provider
Status changed from Unreviewed to Accepted


----------------------------------------
Feature #3377: parsedfile stops processing all entries if one resource is 
missing a required field.
http://projects.puppetlabs.com/issues/3377

Author: Dan Bode
Status: Accepted
Priority: Normal
Assigned to: 
Category: provider
Target version: 
Affected version: 0.25.4
Keywords: parsedfile
Branch: 


consider the following puppet code:

<pre>
sshkey {"blah":
  name => $fqdn,
  type => dsa,
  key => 'dfd'
}
sshkey {"blah2":
  name => 'blah2',
  type => dsa,
  key => 'fgf',
}

sshkey {"blah3":
  name => 'blah3',
  type => dsa,
}
sshkey {"blah4":
  name => 'blah4',
  type => dsa,
  key => 'dfd',
}
</pre>
<pre>
[r...@collector manifests]# puppet sshtest.pp 
notice: //Sshkey[blah4]/ensure: created
notice: //Sshkey[blah3]/ensure: created
err: Got an uncaught exception of type ArgumentError: Field 'key' is required
</pre>

Puppet stops processing all resources related to the target when it encounters 
an error. The final output is 

<pre>
# HEADER: This file was autogenerated at Tue Mar 16 02:45:51 -0500 2010
# HEADER: by puppet.  While it can still be managed manually, it
# HEADER: is definitely not recommended.
blah4 ssh-dss dfd
</pre>

It would be better if 'blah', and 'blah2' were inserted into the target file. 
parsedfile should still flush out all valid records after the failure.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to