Bug #54614
hobo agent, prendre en compte la possibilité pour un tenant wcs d'être sous tenants/
Début:
07 juin 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
hobo fait une vérification rapide pour ne pas inutilement forker la commande hobo_notify mais ne vérifie pas le nouveau tenants/.
Fichiers
Révisions associées
agent: look for wcs tenant in both old and new directories (#54614)
Historique
Mis à jour par Frédéric Péters il y a presque 3 ans
- Fichier 0001-agent-look-for-wcs-tenant-in-both-old-and-new-direct.patch 0001-agent-look-for-wcs-tenant-in-both-old-and-new-direct.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Nicolas Roche il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3252785cd0204803288970c32220ec9f5dba06d0
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Mon Jun 7 10:48:46 2021 +0200
agent: look for wcs tenant in both old and new directories (#54614)
À noter un premier build avec un test en erreur, corrigé via :
@@ -137,7 +137,7 @@ def test_deploy_host_with_agent_patterns(mocked_subprocess, mocked_exists, local
@mock.patch('hobo.agent.worker.services.subprocess')
def test_notify(mocked_subprocess, mocked_listdir, mocked_exists):
def listdir(path):
- return '%s.dev.publik.love' % path.split('/')[3]
+ return ['%s.dev.publik.love' % path.split('/')[3]]
mocked_listdir.side_effect = listdir
mocked_communicate = mock.Mock(return_value=('', ''))
ce qui me laisse dire que le test n'était pas bien correct.
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
agent: look for wcs tenant in both old and new directories (#54614)