Tech: Ajout d'un mode de maintenance

[tonial]

🤔 Pourquoi ?

Pour permettre de facilement faire des petites coupures de service pour certains déploiements

• Finir de nettoyer le template (retirer les boutons de login/signup, recherche, etc)
  
• Ajouter les tests
• Peut être permettre l'affichage d'un message personnalisé (utile dans le cadre d'une maintenance plus longue et programmée) ?

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

• Mettre à jour le CHANGELOG_breaking_changes.md ?
• Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

[rsebille]

Et est-ce qu'en mode maintenant on ne voudrais pas rediriger vers une URL ou vue spécifique ?
Principalement pour ne pas foutre la merde dans tout ce qui est suivi par URL (APDEX, sentry, datadog, matomo, etc)

[tonial]

Notons que si la DB est en carafe on risque d'avoir du mal à déployer pour à cause du public_health_check
Peut être qu'il faudrait renvoyer systématiquement OK dans le cas où MAINTENANCE_MODE est vrai ?

[tonial]

@hellodeloo si tu veux modifier un peu le template de maintenance, n'hésites pas

[hellodeloo]

@hellodeloo si tu veux modifier un peu le template de maintenance, n'hésites pas

@tonial Yes, je ferais un petit commit UI demain

Par contre, on avait ça avant. C'est plus utile ? Dans ce cas, il faudrait peut être archiver le repo ?

[tonial]

Merci !!

L'ancien repo c'était dans le cas où clever est en carafe, donc c'est encore pertinent.
Ici le but c'est de pouvoir "éteindre" le service le temps d'une migration de base de donnée (donc limité aux emplois)

[hellodeloo]

Merci !!

L'ancien repo c'était dans le cas où clever est en carafe, donc c'est encore pertinent. Ici le but c'est de pouvoir "éteindre" le service le temps d'une migration de base de donnée (donc limité aux emplois)

Dak 👍
Question, si tu charges quand même django. Pourquoi pas simplement décliner un des templates 404/500 ? (parcque je vais faire un truc dans le genre du coup)

[tonial]

Le truc c'est qu'une partie des éléments du site ne marcheront pas.
Aucun lien interne ne va marcher donc je voulais éviter d'afficher "Rechercher un emplois / prescripteur" 'se connecter" etc
Mais c'est vrai que ça complique le template 🤔

[tonial]

@hellodeloo j'ai repris le template 400 pour que ce soit plus simple.
Les liens ne marcheront pas, mais en même temps on écrit que le site est en maintenance, donc l'utilisateur qui essaye n'a qu'à lire ce qu'il y a sur la page

[hellodeloo]

@hellodeloo j'ai repris le template 400 pour que ce soit plus simple. Les liens ne marcheront pas, mais en même temps on écrit que le site est en maintenance, donc l'utilisateur qui essaye n'a qu'à lire ce qu'il y a sur la page

Oui, je pense aussi que ça le fait comme ça et que c'est plus "propre" niveau template 👍
Si l'utilisateur clique sur un lien, il va être redirigé vers cette même page ? Il devrait donc vite comprendre les liens ne fonctionnent pas

[rsebille]

Le truc c'est qu'une partie des éléments du site ne marcheront pas. Aucun lien interne ne va marcher donc je voulais éviter d'afficher "Rechercher un emplois / prescripteur" 'se connecter" etc Mais c'est vrai que ça complique le template 🤔

De toute manière tout les liens renverrons la page de maintenance, donc ils peuvent bien cliquer ça changera pas grand chose :).

---

Bien vu pour la JsonResponse() 👍