Oui c’est simple, à noter que ça demanderait une dépendance en plus. La lib pour parser le json resterait de toute façon quoi qu’il arrive car j’en ai besoin pour gérer les chunk de blocs en json pour la sync locale.
Bon c’est un argument faible, mais j’essaye de n’ajouter une dépendance qui n’est pas déjà dans le projet que si la plue-value me semble valoir le coup par rapport à ce que ça rajoute en temps de compilation et en taille du binaire
Je ne le savais même pas, et je n’ai pas touché à la portion de code qui fait ça, donc c’est très probablement toujours le cas
Si tu prends le temps de la testée convenablement et qu’il semble ne pas y avoir de régression, oui je mergerai. Je dis juste que je ne le ferai pas moi-même
Je viens de basculer mon nœud sur l’image docker de dev.
Ça fonctionne bien, mais j’ai des blocs invalides qui m’empêche de suivre les leaders (dont je faisais fièrement partie !) . Pas grave, je ferais un reset sync…
J’ai poussé plus loin et ajouté la config GVA dans le dossier adéquate via Ansible et je crois que ça fonctionne aussi :
Bon c’est surtout @poka qui voulait plein de nœuds GVA pour pouvoir tester son algo de découverte du réseau dans ğecko.
Mais le problème c’est que là je suis dans une phase de dev qui rend le déploiement de nœuds bien trop compliqués. Il serait préférable d’attendre quelques semaines que je puisse avoir le temps de :
trouver le schéma de db gva qui va bien pour pouvoir fournir les chunk compressés
migrer la db des fiches de peer
coder dans gva les requetes qu’il faut pour pouvoir se sync
coder le client gva de sync
Comptez 2 à 3 semaines.
Enfin faites comme vous voulez, mais vous êtes prévenu, c’est pas le bon moment pour moi pour que plein de gens lances des nœuds de dev vu sur quoi je travaille ces jours-ci.
J’ai une erreur concernant le port BMA avec la dernière version de duniter:dev via docker-compose.
J’ai fait un reset hard avec rm -rf /var/lib/docker/volumes/duniter-prod_data/_data/duniter_default/data/*
Au redémarrage du container j’ai cette erreur:
2021-06-30T00:33:11+00:00 - info: GVA server listen on http://0.0.0.0:30901/gva
2021-06-30T00:33:11+00:00 - info: Server::run; addr=0.0.0.0:30901
2021-06-30T00:33:11+00:00 - info: listening on http://0.0.0.0:30901
2021-06-30T00:33:11+00:00 - info: Block resolution: 0 potential blocks for root block...
2021-06-30T00:33:11+00:00 - error: Error: BMA: no port for remote contact.
at Object.input (/duniter/app/modules/bma/index.js:206:31)
at Stack.processCommand (/duniter/index.js:348:65)
at process._tickCallback (internal/process/next_tick.js:68:7)
Pour info il y a une MR en cours qui permet de lancer un noeud duniter 1.9 sous docker.
Pour finaliser cette MR, j’ai besoin de mettre à jour les dépendances nodejs, je suis bloqué à cette étape car je ne connais pas assez npm, j’aurais bien besoin d’un petit coup de main
Oui, on peut les ignorer, mais j’aimerais m’assurer qu’il n’y a pas de faille critique avant de sortir une release 1.9 officielle.
Vu l’oxidation du code, je ne sais pas si tout le code js et les tests associés sont encore utilisés.
Peut-être que @cgeek pourrait nous en dire plus ?
Ça dépend quel est l’objectif, hein. Mais là il faut bien que vous ayez conscience qu’il y a exactement les mêmes warnings sur le build de la v1.8 et de la v1.9.
Et le fait de corriger ces warnings est AMHA orthogonal à la génération d’une image docker. C’est dans ce sens que je répondais.
