Project

General

Profile

Development #8894

Ajouter une option --redeploy à check_hobos

Added by Benjamin Dauvergne almost 7 years ago. Updated over 6 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
Start date:
06 November 2015
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
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.


Files


Related issues

Related to Publik - Development #9405: Mieux gérer les mises à jour de hoboFermé18 December 2015

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

Actions

Associated revisions

Revision 0257dee4 (diff)
Added by Benjamin Dauvergne over 6 years ago

check_hobos: add --redeploy option (#8894)

Revision 20815297 (diff)
Added by Benjamin Dauvergne over 6 years ago

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

History

#1

Updated by Benjamin Dauvergne almost 7 years ago

#2

Updated by Benjamin Dauvergne almost 7 years ago

  • Blocked by Bug #7272: hobo_deploy: conserver le hobo.json (pour mémoire, débogue, etc) added
#5

Updated by Frédéric Péters over 6 years ago

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

Updated by Benjamin Dauvergne over 6 years ago

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

Updated by Frédéric Péters over 6 years ago

www-data reste mentionné lors de l'appel :

        su -c "$COMMAND check_hobos --redeploy" www-data
#9

Updated by Frédéric Péters over 6 years ago

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

Updated by Frédéric Péters over 6 years ago

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

#11

Updated by Benjamin Dauvergne over 6 years ago

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

Updated by Benjamin Dauvergne over 6 years ago

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

Updated by Frédéric Péters over 6 years ago

ok, go.

#14

Updated by Frédéric Péters over 6 years ago

  • Target version set to v1.26
#15

Updated by Frédéric Péters over 6 years ago

  • Status changed from Nouveau to 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

Updated by Thomas Noël over 6 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF