Projet

Général

Profil

Development #8894

Ajouter une option --redeploy à check_hobos

Ajouté par Benjamin Dauvergne il y a plus de 8 ans. Mis à jour il y a environ 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
Début:
06 novembre 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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

Lié à Publik - Development #9405: Mieux gérer les mises à jour de hoboFermé18 décembre 2015

Actions
Bloqué par w.c.s. - Bug #7272: hobo_deploy: conserver le hobo.json (pour mémoire, débogue, etc)Fermé18 mai 2015

Actions

Révisions associées

Révision 0257dee4 (diff)
Ajouté par Benjamin Dauvergne il y a environ 8 ans

check_hobos: add --redeploy option (#8894)

Révision 20815297 (diff)
Ajouté par Benjamin Dauvergne il y a environ 8 ans

debian: add postinst and triggers files for hobo-redeploy (#8894)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a plus de 8 ans

#2

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é
#5

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.

#6

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

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.

#9472

#7

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
#9

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'])

#10

Mis à jour par Frédéric Péters il y a environ 8 ans

Oops, non, ça doit être $WCS_USER, pas $(WCS_USER).

#11

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).

#12

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

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)

#13

Mis à jour par Frédéric Péters il y a environ 8 ans

ok, go.

#14

Mis à jour par Frédéric Péters il y a environ 8 ans

  • Version cible mis à v1.26
#15

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)
#16

Mis à jour par Thomas Noël il y a environ 8 ans

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

Formats disponibles : Atom PDF