Skip to content

Commit

Permalink
lib, watchfrr: Add some additional status messages to systemd
Browse files Browse the repository at this point in the history
Allow systemd to be informed about operational state so operators can
infer a bit about what is going on with FRR from the systemd status
cli.

sharpd@robot ~/frr4> systemctl status frr
● frr.service - FRRouting
   Loaded: loaded (/usr/lib/systemd/system/frr.service; enabled; vendor preset: disabled)
   Active: active (running) since Thu 2019-10-03 21:09:04 EDT; 7s ago
     Docs: https://frrouting.readthedocs.io/en/latest/setup.html
  Process: 32455 ExecStart=/usr/lib/frr/frrinit.sh start (code=exited, status=0/SUCCESS)
   Status: "FRR Operational"
    Tasks: 12 (limit: 4915)
   Memory: 76.5M
   CGroup: /system.slice/frr.service
           ├─32468 /usr/lib/frr/watchfrr -d zebra bgpd staticd
           ├─32487 /usr/lib/frr/zebra -d -A 127.0.0.1 -s 90000000
           ├─32492 /usr/lib/frr/bgpd -d -A 127.0.0.1
           └─32500 /usr/lib/frr/staticd -d -A 127.0.0.1

Please note the `Status: ...` line above.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
  • Loading branch information
donaldsharp committed Oct 4, 2019
1 parent bf9846b commit b3ee8bc
Show file tree
Hide file tree
Showing 3 changed files with 26 additions and 0 deletions.
9 changes: 9 additions & 0 deletions lib/systemd.c
Original file line number Diff line number Diff line change
Expand Up @@ -86,6 +86,7 @@ static int systemd_get_watchdog_time(int the_process)

void systemd_send_stopping(void)
{
systemd_send_information("STATUS=");
systemd_send_information("STOPPING=1");
}

Expand Down Expand Up @@ -116,3 +117,11 @@ void systemd_send_started(struct thread_master *m, int the_process)
if (wsecs != 0)
thread_add_timer(m, systemd_send_watchdog, m, wsecs, NULL);
}

void systemd_send_status(const char *status)
{
char buffer[1024];

snprintf(buffer, sizeof(buffer), "STATUS=%s", status);
systemd_send_information(buffer);
}
5 changes: 5 additions & 0 deletions lib/systemd.h
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,11 @@ void systemd_send_stopping(void);
*/
void systemd_send_started(struct thread_master *master, int the_process);

/*
* status - A status string to send to systemd
*/
void systemd_send_status(const char *status);

#ifdef __cplusplus
}
#endif
12 changes: 12 additions & 0 deletions watchfrr/watchfrr.c
Original file line number Diff line number Diff line change
Expand Up @@ -460,12 +460,20 @@ static int run_job(struct restart_info *restart, const char *cmdtype,
return -1;
}

#if defined HAVE_SYSTEMD
char buffer[512];

snprintf(buffer, sizeof(buffer), "restarting %s", restart->name);
systemd_send_status(buffer);
#endif

/* Note: time_elapsed test must come before the force test, since we
need
to make sure that delay is initialized for use below in updating the
restart interval. */
if ((time_elapsed(&delay, &restart->time)->tv_sec < restart->interval)
&& !force) {

if (gs.loglevel > LOG_DEBUG + 1)
zlog_debug(
"postponing %s %s: "
Expand All @@ -490,6 +498,9 @@ static int run_job(struct restart_info *restart, const char *cmdtype,
restart->pid = 0;
}

#if defined HAVE_SYSTEMD
systemd_send_status("FRR Operational");
#endif
/* Calculate the new restart interval. */
if (update_interval) {
if (delay.tv_sec > 2 * gs.max_restart_interval)
Expand Down Expand Up @@ -707,6 +718,7 @@ static void daemon_send_ready(int exitcode)
fclose(fp);
#if defined HAVE_SYSTEMD
systemd_send_started(master, 0);
systemd_send_status("FRR Operational");
#endif
sent = 1;
}
Expand Down

0 comments on commit b3ee8bc

Please sign in to comment.