[sup-talk] [PATCH] polling is now done per source
Christopher Warrington
chrisw at rice.edu
Thu Apr 17 18:24:35 EDT 2008
Excerpts from Marc Hartstein's message of Thu Apr 17 15:58:29 -0500 2008:
>> Each source has a poll_interval property. This property is used to
>> determine whether the source should be polled. The user can still
>> for a poll of all sources.
> Thanks for the patch. Seems to work great. Only one problem.
Well, there is another. The before-poll hook doesn't get the correct
arguments and before-acct-poll is run twice. I've fixed this locally
and will post a patch to the patch (or a unified path) once I figure
out how to do that in git.
> When this patch is applied and then sup is run against an old
> sources.yaml (which doesn't provide the poll_interval property), sup
> will crash on startup.
Hmmm... I don't know enough about the yaml infrastructure to fix
this. Does anyone else know if there is a way to specify a default
value when one if not found in the yaml file. This would allow for
graceful upgrading and (in theory) keep the sources.yaml file backward
compatible.
--
Christopher Warrington <chrisw at rice.edu>
More information about the sup-talk
mailing list