[C++] Eigengemaakte linux daemon wil niet stoppen

[Gehakt]

Ik ben bezig met het bouwen van een red hat cluster op twee virtual machines.
Dit is gelukt en met apache als service zie ik netjes een fail over.

Nu is de volgende stap die ik wilde doen zelf een programma maken en deze als daemon starten en laten stoppen. De applicatie is zeer simpel. Het enige wat de applicatie moet doen is een zin met de hostname wegschrijven in een bestand.

Het resultaat wat ik wil bereiken is dat ik op een gedeelde opslag 1 logfile zie met iets van de volgende meldingen:

code:

Actief op node: 'node1'
Actief op node: 'node1'
Actief op node: 'node1'
Actief op node: 'node2'
Actief op node: 'node2'
Actief op node: 'node2'
Actief op node: 'node1'
Actief op node: 'node1'
Actief op node: 'node1'

De applicatie (simpele main) die ik heb geschreven bevat in feite het volgende:

C++:

// haal de hostname op
while(1)
{
   // open het logbestand
   // schrijf een regel inc hostname in het logbestand
   // sluit het logbestand
   // sleep 1 seconde
}

De linux cluster service maakt gebruik van een script in /etc/init.d voor het stopen/starten en de status van de daemon. Hiervoor heb ik een kopie gemaakt van /etc/init.d/skeleton
De enige aanpassingen die ik daarin heb gemaakt zijn het veranderen van de applicatie naam en de optie "--background" meegegeven voor het starten.

Het starten van de daemon in het cluster lijkt ook netjes te gaan. Het stoppen van de daemon duurt echter erg lang en mislukt vervolgens. Ik vermoed dus dat ik ergens een denkfout maak en dat het niet zo simpel is. Moet ik nog iets van signals in mijn applicatie implementeren?


Mijn init.d script is als volgt:

code:

#! /bin/sh
### BEGIN INIT INFO
# Provides:          clustertest
# Required-Start:    $remote_fs $syslog
# Required-Stop:     $remote_fs $syslog
# Default-Start:     2 3 4 5
# Default-Stop:      0 1 6
# Short-Description: Example initscript
# Description:       This file should be used to construct scripts to be
#                    placed in /etc/init.d.
### END INIT INFO

# Author: Foo Bar <[e-mail]>
#
# Please remove the "Author" lines above and replace them
# with your own name if you copy and modify this script.

# Do NOT "set -e"

# PATH should only include /usr/* if it runs after the mountnfs.sh script
PATH=/sbin:/usr/sbin:/bin:/usr/bin
DESC="Description of the service"
NAME=clustertest
DAEMON=/usr/bin/$NAME
DAEMON_ARGS="/home/joost/DaemonLogFile.txt"
PIDFILE=/var/run/$NAME.pid
SCRIPTNAME=/etc/init.d/$NAME

# Exit if the package is not installed
[ -x "$DAEMON" ] || exit 0

# Read configuration variable file if it is present
[ -r /etc/default/$NAME ] && . /etc/default/$NAME

# Load the VERBOSE setting and other rcS variables
. /lib/init/vars.sh

# Define LSB log_* functions.
# Depend on lsb-base (>= 3.0-6) to ensure that this file is present.
. /lib/lsb/init-functions

#
# Function that starts the daemon/service
#
do_start()
{
    # Return
    #   0 if daemon has been started
    #   1 if daemon was already running
    #   2 if daemon could not be started
    start-stop-daemon --start --quiet --background --pidfile $PIDFILE --exec $DAEMON --test > /dev/null \
        || return 1
    start-stop-daemon --start --quiet --background --pidfile $PIDFILE --exec $DAEMON -- \
        $DAEMON_ARGS \
        || return 2
    # Add code here, if necessary, that waits for the process to be ready
    # to handle requests from services started subsequently which depend
    # on this one.  As a last resort, sleep for some time.
}

#
# Function that stops the daemon/service
#
do_stop()
{
    # Return
    #   0 if daemon has been stopped
    #   1 if daemon was already stopped
    #   2 if daemon could not be stopped
    #   other if a failure occurred
    start-stop-daemon --stop --quiet --retry=TERM/30/KILL/5 --pidfile $PIDFILE --name $NAME
    RETVAL="$?"
    [ "$RETVAL" = 2 ] && return 2
    # Wait for children to finish too if this is a daemon that forks
    # and if the daemon is only ever run from this initscript.
    # If the above conditions are not satisfied then add some other code
    # that waits for the process to drop all resources that could be
    # needed by services started subsequently.  A last resort is to
    # sleep for some time.
    start-stop-daemon --stop --quiet --oknodo --retry=0/30/KILL/5 --exec $DAEMON
    [ "$?" = 2 ] && return 2
    # Many daemons don't delete their pidfiles when they exit.
    rm -f $PIDFILE
    return "$RETVAL"
}

#
# Function that sends a SIGHUP to the daemon/service
#
do_reload() {
    #
    # If the daemon can reload its configuration without
    # restarting (for example, when it is sent a SIGHUP),
    # then implement that here.
    #
    start-stop-daemon --stop --signal 1 --quiet --pidfile $PIDFILE --name $NAME
    return 0
}

case "$1" in
  start)
    [ "$VERBOSE" != no ] && log_daemon_msg "Starting $DESC" "$NAME"
    do_start
    case "$?" in
        0|1) [ "$VERBOSE" != no ] && log_end_msg 0 ;;
        2) [ "$VERBOSE" != no ] && log_end_msg 1 ;;
    esac
    ;;
  stop)
    [ "$VERBOSE" != no ] && log_daemon_msg "Stopping $DESC" "$NAME"
    do_stop
    case "$?" in
        0|1) [ "$VERBOSE" != no ] && log_end_msg 0 ;;
        2) [ "$VERBOSE" != no ] && log_end_msg 1 ;;
    esac
    ;;
  status)
       status_of_proc "$DAEMON" "$NAME" && exit 0 || exit $?
       ;;
  #reload|force-reload)
    #
    # If do_reload() is not implemented then leave this commented out
    # and leave 'force-reload' as an alias for 'restart'.
    #
    #log_daemon_msg "Reloading $DESC" "$NAME"
    #do_reload
    #log_end_msg $?
    #;;
  restart|force-reload)
    #
    # If the "reload" option is implemented then remove the
    # 'force-reload' alias
    #
    log_daemon_msg "Restarting $DESC" "$NAME"
    do_stop
    case "$?" in
      0|1)
        do_start
        case "$?" in
            0) log_end_msg 0 ;;
            1) log_end_msg 1 ;; # Old process is still running
            *) log_end_msg 1 ;; # Failed to start
        esac
        ;;
      *)
        # Failed to stop
        log_end_msg 1
        ;;
    esac
    ;;
  *)
    #echo "Usage: $SCRIPTNAME {start|stop|restart|reload|force-reload}" >&2
    echo "Usage: $SCRIPTNAME {start|stop|status|restart|force-reload}" >&2
    exit 3
    ;;
esac

:

Zoals eerder vermeld: alleen de naam van de applicatie is ingevuld en bij het starten wordt de optie --background extra opgegeven.

Iemand ideeen waarom mijn daemon niet netjes zou willen stoppen? Het lijkt er na lange tijd (30+ s) op dat de service toch gestopt maar het werkt niet in combinatie met het cluster.

[BoAC]

Kijk is naar de manpage van signal. Zet in de functie die je gaat maken een variable om in je prog bijvoorbeeld en break dan uit je eigen while loop.

[Gehakt]

Ik heb een signal handler ingebouwd maar het lijkt nog niet te werken. Ik ben er al wel achter dat het default signal wat --stop stuurt TERM is.

Dit is de code van mij applicatie. Het is de volledige code en het verdient aangezien het bijna niks is. Ik heb echt geen idee waarom hij niet netjes en snel gestopt kan worden door het init.d script.

C++:

#include <stdio.h>
#include <unistd.h>

#include <iostream>
#include <fstream>
#include <string.h>
#include <signal.h>

using namespace std;

// GLOBAL VARIABLES
const int HOST_NAME_MAX(64);
volatile bool run_program(true);

// FORWARD DECLARATION
void handler(int sig, siginfo_t* siginfo, void* context);

int main(int argc, char **argv)
{

    struct sigaction new_action;

    memset(&new_action, '\0', sizeof(new_action));

    new_action.sa_sigaction = &handler;
    new_action.sa_flags = SA_SIGINFO;

    sigaction(SIGTERM, &new_action, 0);

// declare buffer and get hostname string
    char hostName[HOST_NAME_MAX];
    hostName[HOST_NAME_MAX] = '\0';
    int error = gethostname(hostName, HOST_NAME_MAX);

// declare output stream (file)
    ofstream outputFile;

    while(true)
    {
        // Check if we should brake the loop because of a signal
        if (!run_program)
        {
            break;
        }

        string hostNameString;
        hostNameString += string("ClusterTest runs on node: '");
        hostNameString += string(hostName);
        hostNameString += string("'\n");

// Open file, write string, close file
        outputFile.open(argv[1], ios::app);
        outputFile << hostNameString;
        outputFile.close();

    }
    return 0;
}

void handler(int sig, siginfo_t* siginfo, void* context)
{
    printf ("Sending PID: %ld, UID: %ld\n",
        (long)siginfo->si_pid, 
        (long)siginfo->si_uid);
    run_program = false;
}

[ Voor 4% gewijzigd door Gehakt op 07-01-2012 00:06 ]

[farlane]

Stuur em eens handmatig een SIGTERM of SIGKILL, stopt ie dan wel?

[Gehakt]

Ja dan stop hij wel. Als ik kill -s TERM <processid> stuur stopt het geheel wel. Stukje uit de manpage van start-stop-daemon:
−s, −−signal signal

With −−stop, specifies the signal to send to processes being stopped (default TERM).

[ Voor 55% gewijzigd door Gehakt op 07-01-2012 01:06 ]

[Soultaker]

Volgens mij zit het probleem niet in je "daemon" maar in het feit dat er nooit een PID-file niet aangemaakt wordt. Vervolgens kan het rc-script je daemon logischerwijs ook niet signalen. Aangezien je applicatie zelf geen PID-file schrijft, kun je die door start-stop-daemon laten genereren door de --make-pidfile optie (of kortweg -m) mee te geven bij het starten van de daemon. Grote kans dat het dan gewoon werkt.

[Gehakt]

Soultaker schreef op zaterdag 07 januari 2012 @ 06:14:
Volgens mij zit het probleem niet in je "daemon" maar in het feit dat er nooit een PID-file niet aangemaakt wordt. Vervolgens kan het rc-script je daemon logischerwijs ook niet signalen. Aangezien je applicatie zelf geen PID-file schrijft, kun je die door start-stop-daemon laten genereren door de --make-pidfile optie (of kortweg -m) mee te geven bij het starten van de daemon. Grote kans dat het dan gewoon werkt.

Dit is inderdaad de oplossing voor het niet stoppen van de "daemon"! Op het cluster werkt het nu ook gewoon correct!

Dank aan allen die meegedacht hebben! Ik kan weer verder bouwen aan het geheel

[ Voor 13% gewijzigd door Gehakt op 07-01-2012 10:56 ]