Issue #4411 has been updated by Daniel Pittman.

Everyone, I added Randall, our UX ninja to the discussion.

Randall, I am inclined to agree with these views: being able to run constantly 
seems like something of an exciting way to fail to me, and relatively unlikely 
to be of common value.

IMO, compatible alternatives would be to use a string "always" where zero now 
works, or to have a separate option for this.
----------------------------------------
Bug #4411: runinterval=0 does not behave as expected
https://projects.puppetlabs.com/issues/4411

Author: Oliver Hookins
Status: Re-opened
Priority: Normal
Assignee: 
Category: 
Target version: 
Keywords: 
Branch: 
Affected URL: 


Arguably "most" applications that can be configured to run periodically take a 
period of "0" to mean "don't run periodically". In Puppet's case, the value of 
runinterval is passed directly to the agent timer loop.

Evidently, EventLoop::Timer is happy to fire off an event after 0 seconds have 
passed and thus runinterval=0 signifies "run constantly" which is not the 
expected nor desired behaviour (would *anyone* want this behaviour???).

So I propose that a value of 0 should be taken to mean "disable the timed run 
loop".


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