The answer is: we get a lot more events. It looks like this was intentional, to not behave this way.
On Wed, Jul 7, 2010 at 10:36 AM, Jesse Wolfe <[email protected]> wrote: > If "ensure" changes, then no other changes are logged for that resource. > I'm not sure why this special case is here. If I remove it, like so, > then what happens? > > Signed-off-by: Jesse Wolfe <[email protected]> > --- > lib/puppet/transaction/resource_harness.rb | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/lib/puppet/transaction/resource_harness.rb > b/lib/puppet/transaction/resource_harness.rb > index ae38bcb..7c1d019 100644 > --- a/lib/puppet/transaction/resource_harness.rb > +++ b/lib/puppet/transaction/resource_harness.rb > @@ -48,11 +48,10 @@ class Puppet::Transaction::ResourceHarness > > if param = resource.parameter(:ensure) > return [] if absent_and_not_being_created?(current, param) > - return [Puppet::Transaction::Change.new(param, > current[:ensure])] unless ensure_is_insync?(current, param) > return [] if ensure_should_be_absent?(current, param) > end > > - resource.properties.reject { |p| p.name == :ensure }.reject do > |param| > + resource.properties.reject do |param| > param.should.nil? > end.reject do |param| > param_is_insync?(current, param) > -- > 1.7.0.4 > > -- > You received this message because you are subscribed to the Google Groups > "Puppet Developers" 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-dev?hl=en. > > -- You received this message because you are subscribed to the Google Groups "Puppet Developers" 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-dev?hl=en.
