Development #16804
debian : fournir un fichier de service systemd
0%
Description
Comme ailleurs.
(mais attention, pas comme ailleurs bijoe a une commande "update" à son script init.d qui fait les migrations et le collectstatic)
Fichiers
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a plus de 6 ans
Frédéric Péters a écrit :
(mais attention, pas comme ailleurs bijoe a une commande "update" à son script init.d qui fait les migrations et le collectstatic)
Avec systemd on perd cette possibilité, non ? Est-ce qu'à la place on ajoute la prise en charge du update dans la commande exécutée au démarrage du service ?
Mis à jour par Frédéric Péters il y a plus de 6 ans
Avec systemd on perd cette possibilité, non ? Est-ce qu'à la place on ajoute la prise en charge du update dans la commande exécutée au démarrage du service ?
Pour moi c'est en effet l'occasion de faire comme les autres modules.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Les restart sont vraiment coûteux avec un update au milieu, j'étais plus à l'aise sur un update sur le postinst et à la rigueur ajouter une commande update dans hobo.agent.common qui face migrate_schemas + collectstatic -l pour les cas où il faut reprendre tout ça à la main.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-WIP-debian-systemd-service-file-16804.patch 0001-WIP-debian-systemd-service-file-16804.patch ajouté
- Patch proposed changé de Non à Oui
Benj tu verrais un fichier systemd minimal (comme dans le patch WIP) et la commande update irait ailleurs ?
Et donc conserver la fonctionnalité update dans le postinst, sans recours au script sysv-init ?
Même si c'est coûteux, c'est pas le rôle du restart de prendre en charge ça ?
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Et bien pour moi non, mais si la majorité valide je ne m'y opposerai pas, quand t'es en stress attendre que 30 tenants se migre je trouve ça chiant, bon sur bijoe je reconnais qu'il n'y a jamais de stress, mais sur authentic ça me ferait un peu chier.
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Donc va pour le update dans le restart pour bijoe.
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-WIP-debian-systemd-service-file-16804.patch 0001-WIP-debian-systemd-service-file-16804.patch ajouté
Donc dans l'idée quelque chose comme ça ? (ajout de deux directives ExecStartPre dans le fichier de service)
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
- Statut changé de Nouveau à Information nécessaire
- Assigné à mis à Christophe Siraut
Christophe tu peux donner ton avis là dessus je ne m'y connais pas en systemd.
Mis à jour par Frédéric Péters il y a plus de 6 ans
Dans les autres modules, genre combo, c'est presque ça, on a comme différence qu'on ne passe pas par des liens symboliques :
ExecStartPre=/usr/bin/combo-manage migrate_schemas --noinput ExecStartPre=/usr/bin/combo-manage collectstatic --noinput ExecStart=/usr/bin/gunicorn \ ...
Mis à jour par Paul Marillonnet il y a plus de 6 ans
(Et ignorer les erreurs pour l'opération de collectstatic (préfixe '-') ?)
Mis à jour par Frédéric Péters il y a plus de 6 ans
(Et ignorer les erreurs pour l'opération de collectstatic (préfixe '-') ?)
Il y a des situations d'erreur connues et "légitimes" ? Comme ça je dirais que non et donc qu'il ne faut rien ignorer.
Mis à jour par Christophe Siraut il y a plus de 6 ans
je pense qu'il faut rajouter dh-systemd aux Build-depends
Mis à jour par Christophe Siraut il y a plus de 6 ans
Christophe Siraut a écrit :
je pense qu'il faut rajouter dh-systemd aux Build-depends
Mis à jour par Paul Marillonnet il y a plus de 6 ans
- Fichier 0001-WIP-debian-systemd-service-file-16804.patch 0001-WIP-debian-systemd-service-file-16804.patch ajouté
Christophe Siraut a écrit :
je pense qu'il faut rajouter dh-systemd aux Build-depends
Et donc ajouter --with systemd à la cible par défaut dans debian/rules, non ?
Nouveau patch, avec prise en compte de vos remarques Frédéric et Christophe.
Mis à jour par Christophe Siraut il y a plus de 6 ans
- Assigné à changé de Christophe Siraut à Paul Marillonnet
Ton patch me semble bien, mais je ne l'ai pas testé (et je ne suis pas encore un spécialiste de systemd).
Mis à jour par Thomas Noël il y a plus de 6 ans
Modifier le message de commit : « debian: add systemd service file (#16804) »
Puis ack.
Mis à jour par Paul Marillonnet il y a environ 6 ans
commit cd140d19cccae5f2ed6ae4ae75983cd18a6a5ac5 (HEAD -> master, origin/master, origin/HEAD)
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Mon Dec 18 17:40:09 2017 +0100
debian: add systemd service file (#16804)
Mis à jour par Benjamin Dauvergne il y a environ 6 ans
- Statut changé de Information nécessaire à Résolu (à déployer)
Mis à jour par Benjamin Dauvergne il y a presque 6 ans
- Statut changé de Résolu (à déployer) à Fermé
debian: add systemd service file (#16804)