Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TAXHUB_SETTINGS absolute path default #20

Merged
merged 3 commits into from
Oct 18, 2023
Merged

Conversation

joelclems
Copy link
Contributor

No description provided.

@joelclems joelclems requested a review from bouttier October 16, 2023 13:51
@camillemonchicourt
Copy link
Member

camillemonchicourt commented Oct 18, 2023

OK donc il y avait un soucis et les modifications locales de la configuration de TaxHub n'étaient pas prises en compte :

  • Le chemin relatif faisait qu’on utilisait en fait le fichier de configuration fourni dans l’image Docker : https://github.com/PnX-SI/TaxHub/blob/master/apptax/docker_config.py
  • En passant à un chemin absolu, on utilise bien le fichier de configuration de GeoNature-Docker-services. Mais c’est un fichier qui contient uniquement la SECRET_KEY : https://github.com/PnX-SI/GeoNature-Docker-services/blob/main/init-config.sh
  • Pour ne pas avoir à définir également dans le fichier de configuration de TaxHub les autres paramètres obligatoires, ces paramètres ont été rajoutés via des variables d’environnement : 4bd37fd
  • Seuls ceux qui ont modifié localement le fichier docker-compose.yml doivent reporter ces modifications
  • À noter qu’il serait bien que TaxHub démarre correctement sans ces paramètres (et qu’il trouve tout seul des valeurs par défaut)

@camillemonchicourt camillemonchicourt changed the base branch from main to develop October 18, 2023 06:55
@camillemonchicourt camillemonchicourt merged commit c240825 into develop Oct 18, 2023
@camillemonchicourt camillemonchicourt deleted the fix/taxhub_config branch October 18, 2023 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants