From mboxrd@z Thu Jan 1 00:00:00 1970 From: chrisw@rice.edu (Christopher Warrington) Date: Thu, 17 Apr 2008 17:24:35 -0500 Subject: [sup-talk] [PATCH] polling is now done per source In-Reply-To: <1208465750-sup-5733@cabinet> References: <1206784680-sup-3052@chris-tablet> <1206786725-5456-1-git-send-email-chrisw@rice.edu> <1208465750-sup-5733@cabinet> Message-ID: <1208470560-sup-9902@chris-tablet> 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