Development #21605
hobo_deploy : le hobo.json stocké dans le répertoire du tenants devrait être identique à celui passé à la commande hobo_deploy
0%
Description
Dans wcs/ctl/check_hobos.py :
- l'objet
self.all_services
est créé à partir du hobo.json passé à la commande hobo_deploy - plus loin:
service['base_url'] = base_url[:-1]
ce qui altère l'objet self.all_services
- encore plus loin :
atomic_write(os.path.join(pub.app_dir, 'hobo.json'), json.dumps(self.all_services))
qui écrit dans le répertoire des tenants un hobo.json altéré par rapport à ce qu'il était avant d'être passé à la commande hobo_deploy.
Tombé là dessus en essayant de reproduire un hobo_deploy exécuté par un agent hobo, qui invoque la commande de la façon suivante:
'... hobo_deploy https://tenant-name/ < hobo.json'
Relancer la commande à la main comme suit: '... hobo_deploy https://tenant-name/ /var/lib/wcs/tenant-name/hobo.json' échoue ici
try: service = [x for x in self.all_services.get('services', []) if \ x.get('service-id') == 'wcs' and x.get('base_url') == base_url and not x.get('secondary')][0] except IndexError: return
parce que 'https://tenant-name/' n'est pas présent dans '/var/lib/wcs/tenant-name/hobo.json' où 'https://tenant-name' s'y trouve à la place.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a environ 6 ans
- Fichier 0001-check_hobos-handle-base_url-with-ou-without-trailing.patch 0001-check_hobos-handle-base_url-with-ou-without-trailing.patch ajouté
- Patch proposed changé de Non à Oui
une autre façon de voir les choses, c'est d'être plus souple sur la recherche du base_url, avec ou sans / final... (parce que modifier le hobo.json qu'on gère actuellement, je ne suis pas sûr à ce stade)
Mis à jour par Frédéric Péters il y a environ 6 ans
L'hobo.json n'est pas utilisé, juste stocké pour la forme (#7272), donc il me semble qu'on peut sans risque le stocker non-altéré (modulo le this qui est ajouté et utilisé lors du redeploy).
Mis à jour par Thomas Noël il y a environ 6 ans
- Fichier 0001-check_hobo-don-t-modify-wcs-base_url-in-stored-hobo..patch 0001-check_hobo-don-t-modify-wcs-base_url-in-stored-hobo..patch ajouté
Same player, shoot again. Nettoyage du / final uniquement dans les moments où on gère de la config d'un service wcs (site-options.ini et config.pck)
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/832
- Titre : WIP: misc: do not alter base_url in stored hobo.json (#21605)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/832/files
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Solution proposée à Solution validée
Nicolas Roche (nroche) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Solution validée à Résolu (à déployer)
Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/wcs/pulls/832
- Titre : misc: do not alter base_url in stored hobo.json (#21605)
- Modifications : https://git.entrouvert.org/entrouvert/wcs/pulls/832/files
Mis à jour par Transition automatique il y a 5 mois
- Statut changé de Résolu (à déployer) à Solution déployée
misc: do not alter base_url in stored hobo.json (#21605)