Affiliations united : one directory for all nodejs ws #3

Merged thouveni merged 4 commits into tdm:master from tdm:affiliations-united on 24 Dec
@thouveni thouveni commented on 23 Dec

No description provided.

I don't understand how libpostal works: the .ini references @ezs/local but I don't see any local Javascript file.

@parmentf je propose de rapatrier les scripts du ws dédié à libpostal dans le répertoire affiliations-tools

ensuite on pourrait envisager d'avoir une instance unique à laplace des 2 instances actuelles affiliation-tools & affiliations-libpostal
pour cela il faudra utiliser une image lodex-worker contenant les binaires libpostal (ezmaster-libpostal)

Cette image est basée sur lodex-worker et elle y ajoute juste les binaires libpostal.
Du coup ezmaster-libpostal fonctionne comme lodex-worker

Qu'en penses-tu ?

affiliations-tools/examples.http

Example URL was fixed

Excellente idée.

Je n'avais pas compris que affiliations-libpostal utilisait une autre image que le classique lodex-workers.

Par contre, je ne sais pas exactement comment fonctionne @ezs/local.

Oups, pardon, je viens de voir que le code est dans un répertoire local, mais sur ezmaster-libpostal.

Je me demande si c'est bien de le localiser là. On ne voit pas tout le code ezs nécessaire au service.
Que la bibliothèque libPostal soit sur l'image ne me choque pas, ce n'est qu'une dépendance...

idéalement, je voulais mettre @ezs/local dans le mono repo ezs avec une peerDependencies pour libpostal

mais je ne sais pas encore si ça peut marcher comme ça (sans modification de lodex-worker)

à priori, il faut lancer un npm rebuild même après un npm install

Par comparaison avec le service rnsr: le code est dans le paquet @ezs/conditor, mais il est vrai qu'il ne dépend d'aucune bibliothèque binaire.

C'est le npm rebuild qui pose problème, si j'ai bien compris ?

oui, après installation , il faut un rebuild
je vais créer un paquet dans le mono repo ezs, et après je verrai comment gérer ça

@thouveni thouveni referenced the pull request on 24 Dec

Affiliations united : one directory for all nodejs ws

@thouveni thouveni merged commit 721e5bd into master from affiliations-united on 24 Dec
@thouveni thouveni closed this pull request on 24 Dec
Labels

Priority
default
Milestone
No milestone
Assignee
No one
2 participants
@thouveni @parmentf