Projet

Général

Profil

Bug #9507

ignorer les tenants qui n'ont pas été déployés par hobo lors du hobo_notify

Ajouté par Frédéric Péters il y a plus de 8 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Catégorie:
-
Version cible:
-
Début:
05 janvier 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers

0001-agent-notify-skip-tenants-that-were-not-deployed-wit.patch (1,18 ko) 0001-agent-notify-skip-tenants-that-were-not-deployed-wit.patch Frédéric Péters, 05 janvier 2016 11:18

Révisions associées

Révision bad50162 (diff)
Ajouté par Frédéric Péters il y a plus de 8 ans

agent/notify: skip tenants that were not deployed with hobo (#9507)

Historique

#1

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

#2

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

mmmh... c'est vraiment génant ? (parce qu'on peut imaginer un déploiement "manuel", pour une raison x ou y, ensuite intégré à un publik)

(et parce que, aussi, le hobo.json, on l'a mis en copie dans le tenant parce que c'est très pratique, mais en principe on devrait éviter, y'a des secrets que le tenant n'a pas à connaître)

bon voilà j'ai fini de râler, ack.

#3

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

Normalement dans les messages hobo_notify il y a une notion d'audience, donc de toute façon seul les tenants concernés reçoivent les messages, ou alors il y a un bug.

#4

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

Hier lors d'un hobo_notify, trace de l'agent :

RuntimeError('command "/home/fred/src/eo/venv/bin/python /home/fred/src/eo/combo/manage.py hobo_notify -" failed: '' 'Traceback (most recent call last):
  File "/home/fred/src/eo/combo/manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/home/fred/src/eo/venv/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 385, in execute_from_command_line
    utility.execute()
  File "/home/fred/src/eo/venv/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 377, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/home/fred/src/eo/venv/local/lib/python2.7/site-packages/django/core/management/base.py", line 288, in run_from_argv
    self.execute(*args, **options.__dict__)
  File "/home/fred/src/eo/venv/local/lib/python2.7/site-packages/django/core/management/base.py", line 338, in execute
    output = self.handle(*args, **options)
  File "/home/fred/src/eo/hobo/hobo/agent/common/management/commands/hobo_notify.py", line 46, in handle
    self.process_notification(tenant, notification)
  File "/home/fred/src/eo/hobo/hobo/agent/common/management/commands/hobo_notify.py", line 149, in process_notification
    service = tenant.get_service()
  File "/home/fred/src/eo/hobo/hobo/multitenant/models.py", line 38, in get_service
    for service in self.get_hobo_json()['services']:
  File "/home/fred/src/eo/hobo/hobo/multitenant/models.py", line 34, in get_hobo_json
    'hobo.json')))
IOError: [Errno 2] No such file or directory: '/var/tmp/combo/tenants/ideo.127.0.0.1.xip.io/hobo.json'
#5

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

Ok ack car en l'état actuel du code un service déployé hors hobo ne peut pas accepter une notification.

#6

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

  • Statut changé de En cours à Résolu (à déployer)
commit bad50162cd442ad13cecb2bff28af4b58162119f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Jan 5 11:17:42 2016 +0100

    agent/notify: skip tenants that were not deployed with hobo (#9507)
#7

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF