Issue #6678 has been updated by Ben Hughes.

so running 'puppet resource zone' which fails in a similar place points back to 
this commit 
<https://github.com/puppetlabs/puppet/commit/a166d50c3c555a38ae13c1658b9afaefd583cfc9#L1R404>.
 Will investigate more, but didn't want to lose this info.
----------------------------------------
Bug #6678: zone type fails with internal error when :ip is not specified (type 
lacks appropriate parameter validation)
https://projects.puppetlabs.com/issues/6678#change-61534

Author: Daniel Pittman
Status: Code Insufficient
Priority: Normal
Assignee: 
Category: usability
Target version: 
Affected Puppet version: development
Keywords: 
Branch: 


We fail trying to split 'nil' internally, because of insufficient validation 
done in the type.

    zone { "foo": }
    private method `split' called for nil:NilClass
    # lib/puppet/type/zone.rb:441:in `validate'
    # lib/puppet/type.rb:1751:in `initialize'

Extracting this behaviour requires either Solaris, or stubbing enough of the 
provider code to pretend we are.

This seems to be a pattern in these type, and presumably providers, where they 
simply don't do enough validation early enough on their inputs.  We also don't 
actually test those, which means we don't detect this sort of problem earlier 
in the picture.


-- 
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