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
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
roles/deploy-tenants/templates/cook-recipe.j2
View File

@ -32,37 +32,37 @@
"template_name": "{{wcs_skeleton_filename}}",
"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
Review
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
Review
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.
{"create-passerelle": {
"url": "https://{{tenants_conf['dev-passerelle']}}/",
"title": "Passerelle"
}},
{% endif %}
{% if tenants_conf['dev-fargo'] %}
{% if 'dev-fargo' in tenants_conf and tenants_conf['dev-fargo'] %}
{"create-fargo": {
"url": "https://{{tenants_conf['dev-fargo']}}/",
"title": "Porte-documents"
}},
{% endif %}
{% if tenants_conf['dev-chrono'] %}
{% if 'dev-chrono' in tenants_conf and tenants_conf['dev-chrono'] %}
{"create-chrono": {
"url": "https://{{tenants_conf['dev-chrono']}}/",
"title": "Agendas"
}},
{% endif %}
{% if tenants_conf['dev-bijoe'] %}
{% if 'dev-bijoe' in tenants_conf and tenants_conf['dev-bijoe'] %}
{"create-bijoe": {
"url": "https://{{tenants_conf['dev-bijoe']}}/",
"title": "Statistiques"
}},
{% endif %}
{% if tenants_conf['dev-welco'] %}
{% if 'dev-welco' in tenants_conf and tenants_conf['dev-welco'] %}
{"create-welco": {
"url": "https://{{tenants_conf['dev-welco']}}/",
"title": "Courrier"
}},
{% endif %}
{% if tenants_conf['dev-lingo'] %}
{% if 'dev-lingo' in tenants_conf and tenants_conf['dev-lingo'] %}
{"create-lingo": {
"url": "https://{{tenants_conf['dev-lingo']}}/",
"title": "Paiements"