On 09/10/2012 04:59 PM, jcbollinger wrote:
On Friday, September 7, 2012 8:04:57 AM UTC-5, Jakov Sosic wrote:
On 09/05/2012 08:07 PM, Jakov Sosic wrote:
> newparam(:destdir) do
> desc "The link of the distro ISO image."
> validate do |value|
> unless Pathname.new(value).absolute?
> raise ArgumentError, "Full pathname must be set"
> end
> end
> end
> cobblerdistro {'CentOS-6.3-x86_64':
> ensure => absent,
> destdir => '/distro',
> }
Maybe I didn't make my self clear enough.... On a puppet run, if the
resource doesn't have destdir defined, i get this error:
err:
/Stage[main]//Node[sunce.srce.hr
<http://sunce.srce.hr>]/Cobbler::Add_distro[CentOS-6.3-x86_64]/Cobblerdistro[CentOS-6.3-x86_64]:
Could not evaluate: undefined method `+' for nil:NilClass
I would like to inform user what the problem is rather then print out
some obscure message like this one.
So, how can I do this from type definition ruby code? Thank you.
You cannot use the parameter validation hook for this because it is
invoked only when a value is set. The case you want to trap is exactly
the case to which that hook does not apply.
Hmm, now that explains it!
As far as I know or can tell, there is no hook for what you want --
basically, whole-resource validation. You can put the validation code
into your provider(s), at the point where you (first) use the value.
That's not ideal, but it can get you a better error message (albeit
issued on the client side, not the master).
This is how I circuvmented it in my provider, by providing custom method
check_params and calling it from create/destroy/exits? methods.
def check_params
if @resource[:destdir].nil?
raise ArgumentError, "destdir must be specified in cobblerdistro
resource!"
end
end
def create
check_params
# other create code
...
...
end
"exists?" and "destroy" look the same as "create".
Now, if I don't have destdir specified in resource definition, here is
what the puppet clients poops out:
err: /Stage[main]//Node[somenode]/Cobblerdistro[CentOS-6.3-x86_64]:
Could not evaluate: destdir must be specified in cobblerdistro resource!
Now, I think this is much better. Hope you like it :) And thank you for
assistance.
--
Jakov Sosic
www.srce.unizg.hr
--
You received this message because you are subscribed to the Google Groups "Puppet
Users" group.
To post to this group, send email to puppet-users@googlegroups.com.
To unsubscribe from this group, send email to
puppet-users+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/puppet-users?hl=en.
