Produits Entr'ouvert » Hobo

Pas validable pour l'instant Ozwillo dans les choux #23893.

Le cron pour executer la commande ozwillo_worker n'est pas dans ce patch? ou sera posé à la main?

À la main (ozwillo est optionnel dans hobo, je ne pense pas que ce soit nécessaire de déployer partout un cron.d partout même commenté) mais je vais mettre à jour le README

Exécuté en recette ça a échoué ainsi :

ozwillo: destroying entrouvert from CRON failed
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/hobo/contrib/ozwillo/management/commands/ozwillo_worker.py", line 63, in handle
    instance.destroy()
  File "/usr/lib/python2.7/dist-packages/hobo/contrib/ozwillo/models.py", line 241, in destroy
    wcs = services['wcs-au-quotidien']
KeyError: 'wcs-au-quotidien'

Contrairement à hobo/contrib/ozwillo/README.rst, sur la recette, le service wcs est déclaré dans une clé "wcs", pas "wcs-au-quotidien".

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

Exécuté en recette ça a échoué ainsi :

[...]

Contrairement à hobo/contrib/ozwillo/README.rst, sur la recette, le service wcs est déclaré dans une clé "wcs", pas "wcs-au-quotidien".

Oui, j'ai corrigé coté code, la clé ayant pour principe d'être l'utilisateur du service (même si ici ça n'a pas trop d'importance parce que le code est spécialisé pour w.c.s. à cause du paramètre obligatoire -f /etc/wcs/wcs-au-quotidien.cfg qui diffère des commandes pour les autres services), j'ai corrigé aussi /etc/hobo/settings.py où il n'y avait de toute façon pas de clé w.c.s.; je vais regarder ce qu'il en est en prod.

C'est visiblement encore plus faux en prod:

    "wcs-au-quotidien": [
        "",
        "wcsctl" 
    ]

la clé est bonne, par contre le préfixe est mauvais, je vais corriger le préfixe et je corrigerai la clé quand je mettrai à jour hobo.

Je dois aussi corriger la présence de la clé 'createdb-connect-params' dans config.pck['postgresql'] pour chaque instance, sinon la destruction (DROP DB pour w.c.s.) ne peut se faire jusqu'au bout. C'est fait en preprod (squelette et toutes instances corrigées).

J'aurai aimé qu'on puisse se passer complètement de ce bout de code mais il faudrait avancer sur la destructions depuis hobo pour cela.

Patchs posés par Benj dans la branche wip/23885-ozwillo-conserver-l-etat-des-demandes-de-deploiement .

Les tests plantent (juste rebaser sur master j'imagine, le tox.ini a bougé) :

pluggy.PluginValidationError: Plugin 'django' could not be loaded: (pytest 3.3.2 (/tmp/tox-cazino/hobo/coverage-django18-passerelle/lib/python2.7/site-packages), Requirement.parse('pytest>=3.6'))!

Dans le REAMDE :

• peut-être pointer /etc/hobo/settings.d/whatever.py plutot que /etc/hobo/settings.py
  pour les variables OZWILLO_*

• tout en bas un "wcs-au-quotidien" qui traine (devrait être "wcs" si j'ai bien compris)

Dans ozwillo.models.py une redondance dans les manipulations d'état, par exemple la methode to_destroy passe l'état à STATE_DESTROY juste après avoir appelé la méthode destroy qui fait de même (idem pour les méthodes to_deploy, deploy)

Dans ozwillo.views.py

    try:
        instance_id = data['instance_id']
    except Exception:
        logger.warning(u'ozwillo: no instance id in destroy request (%r)', data)
        return HttpResponseBadRequest('no instance id')

KeyError plutot que Exception

return HttpResponseNotFound('owillo providing is not active here.')

ozwillo

Tu touches à hobo/multitenant/management/commands/runscript.py dans c1a31ebbb3d44ac9aa3f4a0b98e00650d2259968 qui n'a rien à voir il me semble.

Benj, tu assembles dans un commit et envoie dans le dépôt ?