Bug #6772
hobo_deploy : ne pas ajouter les variables des autres instances
Statut:
Fermé
Priorité:
Normal
Assigné à:
Jérôme Schneider
Version cible:
-
Début:
18 mars 2015
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Actuellement le hobo_deploy pour générer la section [variables] du site-options.cfg ajoute bien toutes les urls, les variables mais il ajoute toutes les variables des services au lieu de n'ajouter que les variable du service courant.
Fichiers
Révisions associées
Historique
Mis à jour par Jérôme Schneider il y a environ 9 ans
- Fichier 0001-hobo_deploy-only-add-variables-of-the-current-servic.patch 0001-hobo_deploy-only-add-variables-of-the-current-servic.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a environ 9 ans
Ça va ajouter un message pylint Invalid variable name "s", il faudrait un nom plus complet. Alternativement, en gardant le shadowing du paramètre par la variable "service" (ce qui est pas top), la boucle sur le service courant pourrait se placer avant la boucle ajoutant les %s_url.
Mis à jour par Jérôme Schneider il y a environ 9 ans
- Fichier 0001-hobo_deploy-only-add-variables-of-the-current-servic.patch 0001-hobo_deploy-only-add-variables-of-the-current-servic.patch ajouté
Nouveau patch qui utilise un nom d'argument de fonction plus explicite et laisse la variable 'service' disponible.
Mis à jour par Frédéric Péters il y a environ 9 ans
- Statut changé de En cours à Résolu (à déployer)
Celui-ci a été poussé.
commit f109d388fdf5095664c117efa1188f79aa32e86e
Author: Jérôme Schneider <jschneider@entrouvert.com>
Date: Wed Mar 18 18:39:48 2015 +0100
hobo_deploy: only add variables of the current service (#6772)
-- le concierge.
hobo_deploy: only add variables of the current service (#6772)