Bug #9985
AttributeError: 'Settings' object has no attribute 'SITE_APP' lors d'un déploiement via hobo
100%
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Josué Kouka il y a environ 8 ans
- Fichier 0001-fix-AttributeError-Settings.SITE_APP-9985.patch 0001-fix-AttributeError-Settings.SITE_APP-9985.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Je ne comprends pas bien comment cela peut arriver, le champ SITE_APP coté hobo n'est pas obligatoire ?
Mis à jour par Josué Kouka il y a environ 8 ans
Benjamin Dauvergne a écrit :
Je ne comprends pas bien comment cela peut arriver, le champ SITE_APP coté hobo n'est pas obligatoire ?
Euh non il ne l'est pas. Ce champ est sensé etre défini dans le settings.json
Mis à jour par Josué Kouka il y a environ 8 ans
On ne peut malheureusement pas déduire le SITE_APP à partir du nom du tenant
Mis à jour par Frédéric Péters il y a environ 8 ans
Si Mandaye dépend d'un paramètre dans les settings, celui-ci devrait être défini dans le mandayejs/settings.py. (ça peut être à None, et la valeur peut être vérifiée à cet endroit du code).
Mis à jour par Josué Kouka il y a environ 8 ans
Frédéric Péters a écrit :
Si Mandaye dépend d'un paramètre dans les settings, celui-ci devrait être défini dans le mandayejs/settings.py. (ça peut être à None, et la valeur peut être vérifiée à cet endroit du code).
Oui, n'empeche que mandayejs.applications.get_app_settings() sera toujours exécuté par django lors de la recherche d'url. Voila pourquoi j'ai préféré faire la vérification dans cette fonction
Mis à jour par Frédéric Péters il y a environ 8 ans
Je suggère quand même de mettre SITE_APP = None dans le settings.py, ça permet de le faire précéder d'un commentaire explicitant ce qui est attendu dans ce paramétrage, ça constitue ainsi dans mandayejs même un minimum de configuration sur le paramétrage. (plutôt que devoir faire des grep dans des settings.json posés sur les serveurs).
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Je pensais qu'on s'était entendu pour que le SITE_APP soit un paramètre qu'on choisit lors du déploiement dans hobo, ce qu'il faudrait c'est qu'on ait un champ site_app dans le modèle hobo.environment.models.MandayeJS
, avec un attribut choices qui suivrait ce qui est dispo dans mandayejs (ça aurait été bien que l'un alimente l'autre mais ça ne me semble pas trop possible pour l'instant).
Mis à jour par Josué Kouka il y a environ 8 ans
- Fichier 0001-fix-AttributeError-Settings.SITE_APP-9985.patch 0001-fix-AttributeError-Settings.SITE_APP-9985.patch ajouté
Benjamin Dauvergne a écrit :
Je pensais qu'on s'était entendu pour que le SITE_APP soit un paramètre qu'on choisit lors du déploiement dans hobo, ce qu'il faudrait c'est qu'on ait un champ site_app dans le modèle
hobo.environment.models.MandayeJS
, avec un attribut choices qui suivrait ce qui est dispo dans mandayejs (ça aurait été bien que l'un alimente l'autre mais ça ne me semble pas trop possible pour l'instant).
J'arrive un peu visualiser ce dont tu parles. Pour l'instant la chose que je ne vois pas encore comment faire, c'est intégrer un ChoicesField en settings.SITE_APP. En regardant le code de hobo, je devrais pourvoir trouver normalement.
Mis à jour par Josué Kouka il y a environ 8 ans
- Lié à Bug #9991: Ajouter un champs site_app dans le models MandayeJS ajouté
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Le plus simple est à mon avis d'en faire une variable du service, et ensuite il faudrait un MandayeSettingsLoader qui récupère la variable dans le hobo.json et la pose dans les settings, tu peux demander à Thomas pour les détails.
Mis à jour par Josué Kouka il y a environ 8 ans
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
Le getattr ne sert plus à rien puisqu'il y a une valeur par défaut dans settings.
Mis à jour par Josué Kouka il y a environ 8 ans
Mis à jour par Josué Kouka il y a environ 8 ans
- Statut changé de Nouveau à Résolu (à déployer)
- % réalisé changé de 0 à 100
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
- Statut changé de Résolu (à déployer) à Fermé
fix AttributeError: 'Settings.SITE_APP' (#9985)