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

Insertion des données stoppée à 10% #81

Closed
lpojgc opened this issue Aug 8, 2024 · 3 comments · Fixed by #82
Closed

Insertion des données stoppée à 10% #81

lpojgc opened this issue Aug 8, 2024 · 3 comments · Fixed by #82
Assignees
Labels
bug Something isn't working

Comments

@lpojgc
Copy link

lpojgc commented Aug 8, 2024

What is the bug or the crash?

Lors de l'exécution de la commande d'update, elle s'arrête à 10 %.
L'apparition du problème semble correlée aux montées en versions de nos instances (GN 2.13.3>2.14.2, Export 1.6>1.7).

Capture d’écran du 2024-08-08 08-23-00

Steps to reproduce the issue

gn2pg_cli --update conf.toml

Versions

GN 2.14.2 pour les 2 bdd
Export 1.7

Additional context

Problème probablement lié à la MaJ de SQLAlchemy du module Export.

@lpojgc lpojgc added the bug Something isn't working label Aug 8, 2024
@lpojgc
Copy link
Author

lpojgc commented Aug 14, 2024

même constat par @jpm-cbna sur une instance source 2.14 et export 1.7 et destination 2.10

@lpofredc
Copy link
Member

lpofredc commented Aug 19, 2024

Ces lignes sont à l'origine de ce problème (cf. PnX-SI/gn_module_export#198):

https://github.com/PnX-SI/gn_module_export/blob/e22ed1b72b8c2a4c477774fd1d5eda316434d5a2/backend/gn_module_export/blueprint.py#L276-L277

Deux choses seraient à faire:

  • monter la limite arbitraire de 1000 lignes sur le module d'export
  • modifier gn2pg pour que le calcul de l'avancement soit réalisé d'après la valeur limit renvoyée par l'API et non par celle paramétrée dans la config de l'import.

@lpofredc lpofredc pinned this issue Aug 19, 2024
@lpofredc lpofredc unpinned this issue Aug 19, 2024
lpofredc added a commit that referenced this issue Aug 20, 2024
fix pagination calculation (fix #81)
lpofredc added a commit that referenced this issue Aug 20, 2024
@lpofredc
Copy link
Member

* monter la limite arbitraire de 1000 lignes sur le module d'export

Proposition de PR sur gn_module_export > PnX-SI/gn_module_export#212

* **modifier gn2pg** pour que le calcul de l'avancement soit réalisé d'après la valeur `limit` renvoyée par l'API et non par celle paramétrée dans la config de l'import.

Modifié dans la version 1.6.6 (cf. #83)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants