Development #8894
Ajouter une option --redeploy à check_hobos
0%
Description
Idem que #8893 mais pour w.c.s. Il faudrait d'abord stocker le hobo.json dans le app_dir avant comme pour les hobo_deploy w.c.s.
Fichiers
Demandes liées
Révisions associées
debian: add postinst and triggers files for hobo-redeploy (#8894)
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Lié à Development #9405: Mieux gérer les mises à jour de hobo ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Bloqué par Bug #7272: hobo_deploy: conserver le hobo.json (pour mémoire, débogue, etc) ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Fichier 0001-check_hobos-add-redeploy-option-8894.patch 0001-check_hobos-add-redeploy-option-8894.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
Mis à jour par Frédéric Péters il y a environ 8 ans
Dans le os.listdir(), ignorer les répertoires se terminant par ".invalid" ?
Le "www-data" devrait pouvoir être tiré de /dec/default/wcs (quand il existe), comme c'est le cas pour le script init.d. (et là il y aura à revoir des déploiements où le /etc/default/wcs contient un "exit 0").
Je mettrais "interest-noawait" (parce que "This should be used when the functionality provided by the trigger is not crucial.").
Et pour que ce soit réellement utile, il faudrait pareil dans le postinst de wcs-au-quotidien.
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
- Fichier 0001-check_hobos-add-redeploy-option-8894.patch 0001-check_hobos-add-redeploy-option-8894.patch ajouté
- Fichier 0002-debian-add-postinst-and-triggers-files-for-hobo-rede.patch 0002-debian-add-postinst-and-triggers-files-for-hobo-rede.patch ajouté
Frédéric Péters a écrit :
Dans le os.listdir(), ignorer les répertoires se terminant par ".invalid" ?
Ok.
Le "www-data" devrait pouvoir être tiré de /dec/default/wcs (quand il existe), comme c'est le cas pour le script init.d. (et là il y aura à revoir des déploiements où le /etc/default/wcs contient un "exit 0").
Ok, j'ai aussi ajouté la lecture des fichiers "CONFIG_FILE" comme dans le init.d au cas où ça pourrait modifier le comportement.
Je mettrais "interest-noawait" (parce que "This should be used when the functionality provided by the trigger is not crucial.").
Ok.
Et pour que ce soit réellement utile, il faudrait pareil dans le postinst de wcs-au-quotidien.
Mis à jour par Frédéric Péters il y a environ 8 ans
www-data reste mentionné lors de l'appel :
su -c "$COMMAND check_hobos --redeploy" www-data
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Mis à jour par Frédéric Péters il y a environ 8 ans
Par précaution je mettrais me = [service for service in hobo_json['services'] if service['this'] is True][0]
entre try/except IndexError, tu peux modifier dans ce sens (si ça te va) et directement pousser ?
(je pense à imio qui a pu déployer en copiant le même hobo.json partout, qui n'aura pas de service['this'])
Mis à jour par Frédéric Péters il y a environ 8 ans
Oops, non, ça doit être $WCS_USER, pas $(WCS_USER).
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Frédéric Péters a écrit :
Par précaution je mettrais
me = [service for service in hobo_json['services'] if service['this'] is True][0]
entre try/except IndexError, tu peux modifier dans ce sens (si ça te va) et directement pousser ?(je pense à imio qui a pu déployer en copiant le même hobo.json partout, qui n'aura pas de service['this'])
Ils n'utilisent même pas hobo_deploy ? Parce que c'est lui qui pose la clé 'this' il me semble (avant d'écrire le hobo.json dans le répertoire du tenant).
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
- Fichier 0001-check_hobos-add-redeploy-option-8894.patch 0001-check_hobos-add-redeploy-option-8894.patch ajouté
- Fichier 0002-debian-add-postinst-and-triggers-files-for-hobo-rede.patch 0002-debian-add-postinst-and-triggers-files-for-hobo-rede.patch ajouté
diff --git a/wcs/ctl/check_hobos.py b/wcs/ctl/check_hobos.py index 696b00d..a02fe30 100644 --- a/wcs/ctl/check_hobos.py +++ b/wcs/ctl/check_hobos.py @@ -55,8 +55,12 @@ class CmdCheckHobos(Command): if not os.path.exists(hobo_json_path): continue hobo_json = json.load(open(hobo_json_path)) - me = [service for service in hobo_json['services'] if service['this'] is True][0] - self.deploy(base_options, sub_options, [me['base_url'], hobo_json_path]) + try: + me = [service for service in hobo_json['services'] if service['this'] is True][0] + except IndexError: + pass + else: + self.deploy(base_options, sub_options, [me['base_url'], hobo_json_path]) else: self.deploy(base_options, sub_options, args)
Mis à jour par Frédéric Péters il y a environ 8 ans
- Statut changé de Nouveau à Résolu (à déployer)
commit 208152976b0f129c60e5da043f827e895d5f9a94 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Fri Dec 18 23:21:07 2015 +0100 debian: add postinst and triggers files for hobo-redeploy (#8894) commit 0257dee49f1ff57a00b7985393eb9389dc2dd791 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Fri Dec 18 23:15:19 2015 +0100 check_hobos: add --redeploy option (#8894)
check_hobos: add --redeploy option (#8894)