Corrige le nombre de sessions rapporté par Munin #6636
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Bug introduit par 2b2215e (PR zestedesavoir#6021), où le backend pour gérer les sessions passe de django.contrib.sessions.backends.cached_db à zds.utils.custom_cached_db_backend, ce qui change la table utilisée pour stocker les sessions et le modèle à utiliser pour les manipuler. La vue qui compte le nombre de sessions pour Munin n'a pas été mise à jour utiliser le nouveau modèle. - Utilise les classes de sessions définies dans les paramètres, plutôt que forcément django.contrib.sessions - Ajoute un test pour vérifier que le nombre de sessions rapporté par Munin est correct - Renomme l'import DBStore en CachedDBStore pour éviter une confusion sur quel backend est utilisé pour stocker les sessions
philippemilink
force-pushed
the
munin-session-number
branch
from
7d6488c to
49752ab
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bug introduit par 2b2215e (PR #6021), où le backend pour gérer les sessions passe de
django.contrib.sessions.backends.cached_dbàzds.utils.custom_cached_db_backend, ce qui change la table utilisée pour stocker les sessions et le modèle à utiliser pour les manipuler. La vue qui compte le nombre de sessions pour Munin n'a pas été mise à jour utiliser le nouveau modèle.django.contrib.sessionsDBStoreenCachedDBStorepour éviter une confusion sur quel backend est utilisé pour stocker les sessionsLa correction du bug en elle-même se trouve dans
django_munin/munin/views.py, où on charge la classe deSessionqui est vraiment utilisée.Contrôle qualité