Projet

Général

Profil

Bug #14712

le script init.d n'échoue pas si le démarrage échoue

Ajouté par Frédéric Péters il y a plus de 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
20 janvier 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers

Révisions associées

Révision 7354867f (diff)
Ajouté par Frédéric Péters il y a plus de 7 ans

debian: make init.d script fail in case of any error (#14712)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 7 ans

#2

Mis à jour par Thomas Noël il y a plus de 7 ans

Mmmh.... ça marche vraiment ?

Parce que par exemple :

    do_stop
    case "$?" in
        0|1) log_end_msg 0 ;;
        2) log_end_msg 1 ;;
    esac

avec -e il me semble qu'on va même pas atteindre le "case", on va s'arrêter lors du "do_stop" en cas de pépin.

#3

Mis à jour par Frédéric Péters il y a plus de 7 ans

S'arrêter en cas de pépin c'est bien l'idée ici, je ne comprends sans doute pas ta remarque.

Avec le -e et une erreur dans le debian_config.py, j'obtiens bien ça :

 $ sudo service combo start
Job for combo.service failed. See 'systemctl status combo.service' and 'journalctl -xn' for details.

#4

Mis à jour par Thomas Noël il y a plus de 7 ans

Frédéric Péters a écrit :

je ne comprends sans doute pas ta remarque.

Elle n'était valable qu'en dehors de systemd, je suis vieux. Donc, ack.

Allez, je préférerais quand même un "set -e", moins discret.

#5

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Statut changé de En cours à Résolu (à déployer)

Avec "set -e", même si côté Passerelle ça a pas été fait ainsi.

commit 7354867f4e7da084fdcf406455243ae2e3ffac15
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Jan 20 14:10:30 2017 +0100

    debian: make init.d script fail in case of any error (#14712)
#6

Mis à jour par Frédéric Péters il y a plus de 7 ans

Pour mémoire, cette modification a cassé la construction des images Docker chez imio : dans les étapes de construction il y a genre un "apt install combo" et le démarrage de combo que ça implique échoue parce que combo n'arrive pas à joindre syslog, pas démarré à ce moment de la construciton.

#7

Mis à jour par Thomas Noël il y a plus de 7 ans

Le problème sera identique (et moins facile à corriger sans doute) si postgresql n'est pas disponible au moment de l'installation. C'est systématique quand postgresql est sur une machine distante, car combo ne le sait pas au moment de l'installation (par défaut il cherche un postgresql local).

#8

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF