[rrd-developers] [PATCH] fixed bug with flush_old_values(-1) and -z <jitter>

kevin brintnall kbrint at rufus.net
Fri Sep 26 18:02:25 CEST 2008


When -z <jitter> is specified, some updates may be timestamped up to
<jitter> seconcds in the future.  Therefore, a timeout of now+1 may not be
sufficient.  Set abs_timeout past the point where any updates are
currently specified.
---
 src/rrd_daemon.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/rrd_daemon.c b/src/rrd_daemon.c
index 82d3bad..f746a35 100644
--- a/src/rrd_daemon.c
+++ b/src/rrd_daemon.c
@@ -501,7 +501,7 @@ static int flush_old_values (int max_age)
   if (max_age > 0)
     cfd.abs_timeout = cfd.now - max_age;
   else
-    cfd.abs_timeout = cfd.now + 1;
+    cfd.abs_timeout = cfd.now + 2*config_write_jitter + 1;
 
   /* `tree_callback_flush' will return the keys of all values that haven't
    * been touched in the last `config_flush_interval' seconds in `cfd'.
-- 
1.6.0.2



More information about the rrd-developers mailing list