summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--CHANGES1
-rw-r--r--doc/smokeping_upgrade.pod6
-rw-r--r--lib/Smokeping/probes/Curl.pm3
-rw-r--r--lib/Smokeping/probes/EchoPingHttp.pm3
-rw-r--r--lib/Smokeping/probes/EchoPingHttps.pm4
5 files changed, 17 insertions, 0 deletions
diff --git a/CHANGES b/CHANGES
index bc804df..2903d33 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,3 +1,4 @@
+* 3 pings is the official minimum now -- niko
* graph fixes for small numbers of pings -- Chris Wilson <chris *aidworld.org>
* fix the LDAP probe killing the CGI with perl <5.6 -- Peter Farmer <pfarmer *hashbang.org.uk>
* make the LDAP probe work withouth IO::Socket::SSL -- niko
diff --git a/doc/smokeping_upgrade.pod b/doc/smokeping_upgrade.pod
index cbc324a..ac393aa 100644
--- a/doc/smokeping_upgrade.pod
+++ b/doc/smokeping_upgrade.pod
@@ -95,6 +95,12 @@ The matcher modules have been renamed to start with a capital letter,
to differentiate the actual modules from the base classes. You have to
capitalize the matcher name in the pattern definition accordingly.
+=item Minimum number of pings
+
+The C<pings> variable now has an enforced minimum value of 3, as the
+whole design of Smokeping is based on the idea of sending several probes
+and measuring and visualizing the variation between them.
+
=back
In addition to this, some probes have had minor incompatible changes to
diff --git a/lib/Smokeping/probes/Curl.pm b/lib/Smokeping/probes/Curl.pm
index 8e40067..fdce3d6 100644
--- a/lib/Smokeping/probes/Curl.pm
+++ b/lib/Smokeping/probes/Curl.pm
@@ -30,6 +30,9 @@ sub pod_hash {
Niko Tyni <ntyni@iki.fi>
DOC
notes => <<DOC,
+You should consider setting a lower value for the C<pings> variable than the
+default 20, as repetitive URL fetching may be quite heavy on the server.
+
The URL to be tested used to be specified by the variable 'url' in earlier
versions of Smokeping, and the 'host' setting did not influence it in any
way. The variable name has now been changed to 'urlformat', and it can
diff --git a/lib/Smokeping/probes/EchoPingHttp.pm b/lib/Smokeping/probes/EchoPingHttp.pm
index 464dd12..570ab66 100644
--- a/lib/Smokeping/probes/EchoPingHttp.pm
+++ b/lib/Smokeping/probes/EchoPingHttp.pm
@@ -27,6 +27,9 @@ DOC
Measures HTTP roundtrip times (web servers and caches) for SmokePing.
DOC
notes => <<DOC,
+You should consider setting a lower value for the C<pings> variable than the
+default 20, as repetitive URL fetching may be quite heavy on the server.
+
The I<fill>, I<size> and I<udp> EchoPing variables are not valid for EchoPingHttp.
DOC
authors => <<'DOC',
diff --git a/lib/Smokeping/probes/EchoPingHttps.pm b/lib/Smokeping/probes/EchoPingHttps.pm
index 13cd7f5..629400e 100644
--- a/lib/Smokeping/probes/EchoPingHttps.pm
+++ b/lib/Smokeping/probes/EchoPingHttps.pm
@@ -30,6 +30,10 @@ DOC
description => <<DOC,
As EchoPingHttp(3pm), but SSL-enabled.
DOC
+notes => <<DOC,
+You should consider setting a lower value for the C<pings> variable than the
+default 20, as repetitive URL fetching may be quite heavy on the server.
+DOC
authors => <<'DOC',
Niko Tyni <ntyni@iki.fi>
DOC