drivers/main.{c,h}, docs/man/ups.conf.txt, conf/ups.conf.sample, NEWS.adoc: introduce reconnect_max_tries setting for NUT drivers [#3540]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>

Author: jimklimov

NEWS.adoc

@@ -156,6 +156,10 @@ https://github.com/networkupstools/nut/milestone/13
       chunks populated only partially with bytes we intended. These buffers are
       now more diligently pre-zeroed as their siblings were in other code paths,
       to avoid sending host stack garbage to a device. [#3529]
+    * Introduced a `reconnect_max_tries` setting for NUT drivers, so they can
+      be configured to exit after the specified number of reconnection attempts
+      (so that the OS service management facility can completely recycle the
+      driver program). [#3540]
 
  - NUT client libraries:
     * Complete support for actions documented in `docs/net-protocol.txt`

conf/ups.conf.sample

@@ -112,6 +112,18 @@ maxretry = 3
 #
 #                The default is 75 seconds.
 #
+# reconnect_max_tries: OPTIONAL. This can be set as a global variable above
+#              your first UPS definition and it can also be set in a
+#              UPS section. If a driver loses connection to the device,
+#              try reconnecting for the specified number of iterations and
+#              exit (so some OS service management facility or an alerted
+#              sysadmin can restart the driver program "from scratch").
+#              A zero value (unreasonable but possible) means to exit as
+#              soon as the device connection was lost.
+#              A negative value (default) means to try reconnecting
+#              indefinitely (e.g. data cable fell out, or the data path
+#              to a networked UPS was lost for a significant time).
+#
 #   debug_min: OPTIONAL.  Specify a minimum debug level for all driver daemons
 #              and for the upsdrvctl tool (when specified at global level),
 #              or for this driver daemon (when specified in a driver section),

docs/man/ups.conf.txt

@@ -100,6 +100,19 @@ its own in `upsd.conf`.
 +
 Environment variable `NUT_STATEPATH` set by caller can override this setting.
 
+*reconnect_max_tries*::
+
+Optional. If a driver loses connection to the device, try reconnecting for
+the specified number of iterations and exit (so some OS service management
+facility or an alerted sysadmin can restart the driver program "from scratch").
++
+A zero value (unreasonable but possible) means to exit as soon as the device
+connection was lost.
++
+A negative value (default) means to try reconnecting indefinitely (e.g. data
+cable fell out, or the data path to a networked UPS was lost for a significant
+time).
+
 *maxstartdelay*::
 
 Optional.  Same as the UPS field of the same name, but this is the
@@ -340,6 +353,11 @@ They would not be actively killed by `upsdrvctl` after this timeout expires.
 +
 The default is 75 seconds.
 
+*reconnect_max_tries*::
+Optional.  This can be set as a global variable above your first UPS
+definition and it can also be set in a UPS section.  See explanation
+above, in the global section.
+
 *maxretry*::
 Optional.  This can be set as a global variable above your first UPS
 definition and it can also be set in a UPS section.  See explanation

drivers/main.c

@@ -102,6 +102,8 @@ vartab_t	*vartab_h = NULL;
 time_t	poll_interval = 2;
 static char	*chroot_path = NULL, *user = NULL, *group = NULL;
 static int	user_from_cmdline = 0, group_from_cmdline = 0;
+/* visible to driver implementations */
+int	reconnect_max_tries = -1, reconnect_count = 0;
 
 /* signal handling */
 int	exit_flag = 0;
@@ -1397,6 +1399,18 @@ static int main_arg(char *var, char *val)
 		return 1;	/* handled */
 	}
 
+	/* Allow per-driver overrides of the global setting
+	 * and allow to reload this, why not. */
+	if (!strcmp(var, "reconnect_max_tries")) {
+		int	intval = -1;
+		if ( str_to_int (val, &intval, 10) ) {
+			reconnect_max_tries = intval;
+			reconnect_count = 0;
+		} else {
+			upslogx(LOG_INFO, "WARNING : Invalid reconnect_max_tries value found in ups.conf global settings");
+		}
+	}
+
 	/* only for upsdrvctl - ignored here */
 	if (!strcmp(var, "sdorder"))
 		return 1;	/* handled */
@@ -1571,6 +1585,18 @@ static void do_global_args(const char *var, const char *val)
 		return;
 	}
 
+	if (!strcmp(var, "reconnect_max_tries")) {
+		int	intval = -1;
+		if ( str_to_int (val, &intval, 10) ) {
+			reconnect_max_tries = intval;
+			reconnect_count = 0;
+		} else {
+			upslogx(LOG_INFO, "WARNING : Invalid reconnect_max_tries value found in ups.conf global settings");
+		}
+
+		return;
+	}
+
 	/* Allow to specify its minimal debugging level for all drivers -
 	 * admins can set more with command-line args, but can't set
 	 * less without changing config. Should help debug of services.

drivers/main.h

@@ -14,7 +14,8 @@
 extern const char	*upsname, *device_name;
 extern char		*device_path, *device_sdcommands;
 extern int		broken_driver, experimental_driver,
-			do_lock_port, exit_flag, handling_upsdrv_shutdown;
+			do_lock_port, exit_flag, handling_upsdrv_shutdown,
+			reconnect_max_tries, reconnect_count;
 extern TYPE_FD		upsfd, extrafd;
 extern time_t		poll_interval;