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

Problème de compatibilité GeoNature/Taxhub #29

Open
lpofredc opened this issue Feb 26, 2024 · 2 comments
Open

Problème de compatibilité GeoNature/Taxhub #29

lpofredc opened this issue Feb 26, 2024 · 2 comments

Comments

@lpofredc
Copy link

lpofredc commented Feb 26, 2024

Testé sur deux instances Docker récemment installées via le dépot PnX-SI/GeoNature-Docker-services.

La route taxhub/api/taxref/allnamebylist/<id_liste>?search_name=<search_string>&limit=20 renvoie une erreur 500 et les logs indiquent l'absence du champ group3_inpn dans la vue vm_taxref_list_forautocomplete.

En fouillant les statuts de la bdd, je ne trouve aucune trace de cette migration > https://github.com/PnX-SI/TaxHub/blob/master/apptax/migrations/versions/8f3256f60915_group3_inpn_autocomplete.py

J'imagine que c'est dû au problème de compatibilité de TaxHub 1.13+ avec un GeoNature v2.14 pas encore publié?

Les dépendances de GeoNature font référence à TaxHub 2.12.1...

$ docker compose exec -it geonature-backend pip freeze  | grep taxhub
taxhub @ file:///dist/geonature/taxhub-1.12.1-py3-none-any.whl#sha256=c590e557a8d52bdfe6b82c48d0f518179953d035e8cf8621a3e72660a5387e85

Le problème peut-être corrigé en fixant (provisoirement) la version de taxhub à la v1.12.1 dans le .env

TAXHUB_IMAGE="ghcr.io/pnx-si/taxhub:1.12.1"
@camillemonchicourt
Copy link
Member

Ah bien vu, j'avais pas pensé à ce cas...

On prend toujours les versions "latest" de TH et UH, comme on voit ici - https://github.com/PnX-SI/GeoNature-Docker-services/blob/main/.env.sample#L37

Et donc si on sort de nouvelles versions de TH ou UH, ça pose soucis.

OK on va voir pour désormais fixer ces versions à chaque release de GeoNature-Docker-services, pour éviter ce genre de problème...

@jacquesfize
Copy link
Contributor

J'ai fait la PR : #30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants