Issue #6678 has been updated by Rahul Gopinath.

Branch changed from emptyzone to https://github.com/puppetlabs/puppet/pull/955


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

Author: Daniel Pittman
Status: In Topic Branch Pending Review
Priority: Normal
Assignee: Rahul Gopinath
Category: usability
Target version: 3.x
Affected Puppet version: development
Keywords: 
Branch: https://github.com/puppetlabs/puppet/pull/955


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