diff options
| author | Austin S. Hemmelgarn <austin@netdata.cloud> | 2024-06-13 13:25:43 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-06-13 13:25:43 -0400 |
| commit | a2f0b951d6a20fb1a92f3376348ccda2f34ec1a8 (patch) | |
| tree | bbde9961d6651f064901d1ec941981ee672683e9 /system | |
| parent | 9ec45735e08ed7ba7ac8e5b1c8f375d14fa89642 (diff) | |
Make our LSB init script _actually_ LSB compliant. (#17824)
We seem to have been relying on some Debian-specific stuff instead of
actually following LSB requirements.
Diffstat (limited to 'system')
| -rw-r--r-- | system/lsb/init.d/netdata.in | 16 |
1 files changed, 11 insertions, 5 deletions
diff --git a/system/lsb/init.d/netdata.in b/system/lsb/init.d/netdata.in index e429ad1c93..2c6ac7f435 100644 --- a/system/lsb/init.d/netdata.in +++ b/system/lsb/init.d/netdata.in @@ -47,18 +47,24 @@ service_start() { chown @netdata_user_POST@:@netdata_user_POST@ $PIDFILE_PATH - log_daemon_msg "Starting real-time performance monitoring" "netdata" + log_success_msg "Starting real-time performance monitoring" start_daemon -p $PIDFILE $DAEMON_PATH/$DAEMON $DAEMONOPTS RETVAL=$? - log_end_msg $RETVAL + case "${RETVAL}" in + 0) log_success_msg "Started real-time performance monitoring" ;; + *) log_error_msg "Failed to start real-time performance monitoring" ;; + esac return $RETVAL } service_stop() { - log_daemon_msg "Stopping real-time performance monitoring" "netdata" + log_success_msg "Stopping real-time performance monitoring" killproc -p ${PIDFILE} $DAEMON_PATH/$DAEMON RETVAL=$? - log_end_msg $RETVAL + case "${RETVAL}" in + 0) log_success_msg "Stopped real-time performance monitoring" ;; + *) log_error_msg "Failed to stop real-time performance monitoring" ;; + esac if [ $RETVAL -eq 0 ]; then rm -f ${PIDFILE} @@ -77,7 +83,7 @@ condrestart() { } service_status() { - status_of_proc -p $PIDFILE $DAEMON_PATH/$DAEMON netdata + pidofproc -p $PIDFILE $DAEMON_PATH/$DAEMON } |