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

Sortir du dépôt Git les données GeoLite #6384

Closed
4 tasks done
philippemilink opened this issue Aug 27, 2022 · 4 comments · Fixed by #6409
Closed
4 tasks done

Sortir du dépôt Git les données GeoLite #6384

philippemilink opened this issue Aug 27, 2022 · 4 comments · Fixed by #6409
Labels
S-Évolution Ajoute de nouvelles fonctionnalités

Comments

@philippemilink
Copy link
Member

philippemilink commented Aug 27, 2022

Les données GeoLite, utilisées pour la géolocalisation des adresses IP des membres, doivent être mises à jour (la version actuellement utilisée date de 2020 et une mise à jour est proposée chaque semaine).

Cependant, ce fichier de données a une taille de 60 Mo, donc ce n'est pas vraiment une idée de mettre à jour simplement ce fichier, faire un commit et une PR : les dépôts Git vont grossir très vite si on met régulièrement à jour le fichier.

Il faut donc sortir ce fichier du dépôt :

  • supprimer le fichier du dépôt
  • permettre l'utilisation du site si le fichier n'est pas présent (notamment pour l'environnement de dev)
  • mettre à jour les instructions d'installation pour indiquer comment récupérer le fichier et où le placer
  • mettre à jour https://github.com/zestedesavoir/ansible-zestedesavoir pour qu'il récupère et installe tout seul le fichier (il me semble que MaxMind met à disposition une API pour ce genre de choses)
@philippemilink philippemilink added the S-Évolution Ajoute de nouvelles fonctionnalités label Aug 27, 2022
@AmauryCarrade
Copy link
Member

Il est en effet possible d'utiliser leur API, avec une clef d'API, pour télécharger la dernière version de la base de données. La documentation est ici.

Peut-être faudrait-il stocker aussi quelque part la dernière date de mise à jour, afin de ne pas retélécharger le fichier à chaque fois, sauf si ce n'est pas un souci pour vous — et éventuellement prendre soin de ne pas le télécharger si exécuté en mode CI ?

@AmauryCarrade
Copy link
Member

Le fichier étant lourd, peut-être serait-il également pertinent de songer à nettoyer l'historique du dépôt de ce fichier ? En fonction des conséquences que ça aurait, bien sûr (à éviter si ça peut avoir des conséquences louches, mais si ça permet de réduire le clone du dépôt de quelques centaines de Mio, ça peut valoir le coup).

@AmauryCarrade
Copy link
Member

AmauryCarrade commented Aug 27, 2022

Autre solution que l'on peut considérer (à voir son coût et sa praticité, cependant) : utiliser GitLFS. Mais sachant que l'offre gratuite de GitHub est d'un Gio de stockage et d'autant de bande passante, ça me semble étroit (surtout la bande passante).

@Arnaud-D
Copy link
Contributor

J'ai lié un ticket sur le dépôt Ansible, comme ça, on a la tâche là-bas. Ça évite d'avoir des tickets concernant deux dépôts ici. Je pense que c'est plus simple comme ça. :)

@Arnaud-D Arnaud-D moved this to À trier in Suivi des tickets Dec 17, 2022
@philippemilink philippemilink moved this from À trier to Lié à une PR active in Suivi des tickets Dec 18, 2022
@github-project-automation github-project-automation bot moved this from Lié à une PR active to Clos in Suivi des tickets Jan 21, 2023
@philippemilink philippemilink moved this to Done in Objectif v30.7 Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-Évolution Ajoute de nouvelles fonctionnalités
Projects
Archived in project
Status: Clos
Development

Successfully merging a pull request may close this issue.

3 participants