-
Notifications
You must be signed in to change notification settings - Fork 1
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
Certaines tâches s'exécutent systématiquement en env Symfony de prod #35
Comments
symfony_env
de ansistrano-symfony-deploy
?
@lucasmirloup pourrais-tu préciser pour ton déploiement, quelles sont les tâches qui s'executent en |
Les tâches Ansible provenant d' |
Effectivement à ce jour il faut penser à renseigner symfony_env et application_environment. Sur certains projets on fait déjà : |
Il faudrait que ce soit dans les defaults du role, @lucasmirloup chaud pr le faire ? |
Si ça vous va, yes 👍 |
Ok pour moi. |
Description du problème
Après installation d'un nouveau projet utilisant
le-phare/ansible-deploy
ainsi que les recipes Flex le-phare/flex-recipes, certaines tâches de déploiement enpreprod
s'effectuent avec un environnement Symfonyprod
.Hypothèse
cbrunnkvist/ansistrano-symfony-deploy utilise la variable
symfony_env
afin de déterminer quel env Symfony utiliser pour certaines commandes (via les variables d'envSYMFONY_ENV
etAPP_ENV
).Nous faisons également appel à cette variable Ansible dans ce projet : exemple, mais nous n'avons pas de valeur par défaut pour cette variable.
La valeur par défaut de ansistrano-symfony-deploy est alors choisie, qui est
prod
.La recette
le-phare/flex-recipes
deploy-pack
utilise également cette variable Ansible : voir code, mais ne définit pas non plus de valeur par défaut.Ce projet déclare une variable nommée
application_environment
que je trouve conceptuellement proche desymfony_env
.Proposition de modification
"Unifier" les variables Ansible
application_environment
etsymfony_env
comme ceci :dans
defaults/main.yaml
ou dans le fichieransible/_variables.yml
de la recettedeploy-pack
dele-phare/flex-recipes
(lien).The text was updated successfully, but these errors were encountered: