entrouvert
/
publik-devinst
16
0
Fork 0
Code Issues Pull Requests 2 Releases Activity

templates: do not assume optional services' presence in tenant conf (#75645) #5

Merged
pmarillonnet merged 1 commits from wip/75645-ansible-undefined-tenant-conf-variable-error into main 2023-03-21 12:36:09 +01:00
Conversation 0 Commits 1 Files Changed 1 +6 -6
pmarillonnet commented 2023-03-21 11:13:30 +01:00
Owner
Copy Link
No description provided.
pmarillonnet added 1 commit 2023-03-21 11:13:31 +01:00
smihai reviewed 2023-03-21 11:19:22 +01:00
roles/deploy-tenants/templates/cook-recipe.j2
@ -33,3 +33,3 @@
"title": "Démarches"
}},
{% if tenants_conf['dev-passerelle'] %}
{% if 'dev-passerelle' in tenants_conf and tenants_conf['dev-passerelle'] %}
smihai commented 2023-03-21 11:19:22 +01:00
Owner
Copy Link

Je suis d'avis de garder Passerelle quand meme. J'aime bien passerelle.

Je suis d'avis de garder Passerelle quand meme. J'aime bien passerelle.
pmarillonnet commented 2023-03-21 11:52:54 +01:00
Author
Owner
Copy Link

Je ne fais que reprendre telle quelle la liste des services considérés comme optionnels dans Publik, et fais en sorte que si l’entrée correspondante dans la config de tenant est absente, ça ne crashe pas lors du déploiement.

Pour moi, revoir la liste des services considérés comme optionnels, c’est autre autre débat autre ticket autre PR.

Je ne fais que reprendre telle quelle la liste des services considérés comme optionnels dans Publik, et fais en sorte que si l’entrée correspondante dans la config de tenant est absente, ça ne crashe pas lors du déploiement. Pour moi, revoir la liste des services considérés comme optionnels, c’est autre autre débat autre ticket autre PR.
ecazenave approved these changes 2023-03-21 12:24:15 +01:00
pmarillonnet merged commit 4a6dd19739 into main 2023-03-21 12:36:09 +01:00
pmarillonnet deleted branch wip/75645-ansible-undefined-tenant-conf-variable-error 2023-03-21 12:36:09 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
ecazenave
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/publik-devinst#5
No description provided.
Delete Branch "wip/75645-ansible-undefined-tenant-conf-variable-error"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?