Issue #6711 has been updated by Nick Fagerlund.

Pending action on #8002, I'm abandoning this for now.
----------------------------------------
Bug #6711: exec type refresh cmd should do tries and log
https://projects.puppetlabs.com/issues/6711

Author: Matt Robinson
Status: Accepted
Priority: Low
Assignee: Nick Fagerlund
Category: documentation
Target version: 
Affected Puppet version: development
Keywords: 
Branch: 


    # Run the command, or optionally run a separately-specified command.
    def refresh
      if self.check_all_attributes(true)
        if cmd = self[:refresh]
          provider.run(cmd)
        else
          self.property(:returns).sync
        end
      end
    end

If a refresh parameter is given to an exec type with tries set

    exec { '/some/cmd' :
      refresh => '/some/othercmd',
      tries => 10,
    }

tries will only apply to the /some/cmd, and not to the /some/othercmd when the 
exec resource is triggered by a refresh.  This is probably unintuitive and 
should be fixed or at least documented.  Also, by having different code paths 
for running the command regularly and through a refresh some logging gets lost 
when refreshing.


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