Skip to content
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

Ajoute des statistiques Munin concernant les membres #6639

Merged
merged 1 commit into from
Sep 7, 2024

Conversation

philippemilink
Copy link
Member

  • le nombre de membres bannis (dans zds/munin au lieu de django_munin pour ne pas avoir à importer zds.member.models.Profile et garder django_munin indépendant de zds)
  • le nombre de membres qui ont l'attribut is_active=True
  • le nombre de membres qui se sont connectés au moins une fois

Contrôle qualité

Vérifier que la CI passe, le patch est déployé sur la bêta, le résultat est observable sur Munin.

@coveralls
Copy link

coveralls commented Sep 3, 2024

Coverage Status

coverage: 88.879% (-0.002%) from 88.881%
when pulling 6c2f1a8 on philippemilink:munin-users
into f3a93ab on zestedesavoir:dev.

Copy link
Contributor

@Arnaud-D Arnaud-D left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

Je n'ai pas testé les graphes. Tu sais comment faut faire ? Il faudrait une instance munin en local ?

Sinon on peut fusionner.

- le nombre de membres bannis
- le nombre de membres qui ont l'attribut `is_active=True`
- le nombre de membres qui se sont connectés au moins une fois
@philippemilink
Copy link
Member Author

philippemilink commented Sep 7, 2024

Je n'ai pas testé les graphes. Tu sais comment faut faire ? Il faudrait une instance munin en local ?

J'ai testé en local avec une VM construite avec Vagrant et provisionnée avec nos scripts Ansible. J'ai ensuite appliqué le patch sur le serveur de bêta pour voir le résultat avec de vraies données.

@philippemilink philippemilink enabled auto-merge (squash) September 7, 2024 17:46
@philippemilink philippemilink merged commit cefa2cb into zestedesavoir:dev Sep 7, 2024
8 checks passed
@philippemilink philippemilink deleted the munin-users branch September 7, 2024 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants