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 [email protected].
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.
