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

[OpenStack] nettoyage des variables d'environnement #6890

Closed
akarzim opened this issue Feb 2, 2022 · 0 comments · Fixed by #6932
Closed

[OpenStack] nettoyage des variables d'environnement #6890

akarzim opened this issue Feb 2, 2022 · 0 comments · Fixed by #6932

Comments

@akarzim
Copy link
Contributor

akarzim commented Feb 2, 2022

Résumé

Cette proposition concerne un petit nettoyage dans l'utilisation des variables d'environnement liées à OpenStack en systématisant l'usage de la variable FOG_OPENSTACK_URL en lieu et place de FOG_BASE_URL.

mots-clés : env, openstack, url

ticket lié : #6887

Actuellement

La variable FOG_BASE_URL semble n'être utilisée nulle part (et pas non plus dans des gems dépendantes).

La variable FOG_OPENSTACK_URL, bien que toujours présente, n'est plus utilisée depuis le 19 juin 2020 (commit 6a3811a), sa valeur ayant été inscrite en dur dans le code.

La variable DS_PROXY_URL a un temps partagé la même valeur que FOG_BASE_URL, c'est-à-dire https://static.demarches-simplifiees.fr/, mais semble ne plus être utilisée aujourd'hui.

Ça vient de la PR #5280, qui avait été un peu compliquée. En substance je pense que c'est une erreur.

— Pierre de La Morinerie, le 2 février 2022

Comportement attendu

Deux variables d'environnement devraient être mises à disposition avec les valeurs par défaut suivante :

FOG_OPENSTACK_AUTH_URL: https://auth.cloud.ovh.net
FOG_OPENSTACK_URL: https://static.demarches-simplifiees.fr

Proposition

Effectuer les changements nécessaires pour faire disparaitre toute trace de DS_PROXY_URL et FOG_BASE_URL. Et systématiser l'utilisation de FOG_OPENSTACK_AUTH_URL et FOG_OPENSTACK_URL en lieu et place des valeurs inscrites en dur dans le code.


voir : PR #6932 / @adullact & @synbioz

  • L'ADULLACT a mandaté le prestataire @synbioz pour développer plusieurs fonctionnalités (tickets et PR à venir).
  • C'est dans ce cadre que @synbioz nous propose certains correctifs annexes comme celui-ci.
  • Si c'est nécessaire, @akarzim et @jobygoude de @synbioz pourront interagir avec l'équipe @betagouv sur ce ticket et sur la PR (répondre aux commentaires, pousser des commits…).

trackingAdullactContrib

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 a pull request may close this issue.

1 participant