In my discussion with @adrienthebo on my pull request:
https://github.com/puppetlabs/puppet/pull/2309 I raised the question of
what should be the allowable or suggested values for the ensure property?
My pull request separated out the hold state into a separate property:
before:
package { 'foo':
ensure => held
}
after:
package { 'foo':
ensure => '0.4',
hold => false
}
@adrienthebo suggested I separate out the version instead:
package { 'foo':
ensure => held,
version => '0.4',
}
@adrienthebo's suggestion is certianly doable, my question is whether the
puppet community has come to a consensus on what values the ensure property
should have?
In general my experience has been that adding additional values to ensure
for resources makes using a resource less intuitive. For instance I find
the file resource has a confusing interface because of the way it abuses
the ensure property:
file {'/foo':
ensure => file,
}
file {'/foo':
ensure => directory,
}
I would rather have explicit resources:
directory {'/foo':
ensure => present
}
What are the communities thoughts?
-Jesse Hathaway
--
You received this message because you are subscribed to the Google Groups
"Puppet Developers" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to puppet-dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/puppet-dev/CANSNSoVSqQTwFk8ZxxVPW-Ffs%3DKcDSnotD%2B5gUz%3Du0MstH54Xg%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
