Ajout un champ dans les log

18 views
Skip to first unread message

chtiloft

unread,
Feb 3, 2019, 1:59:49 PM2/3/19
to Eibd / Linknx / KnxWeb
bonjour à tous,
on dirait que le projet est déserté :(.

Perso j’utilise toujours le trio sur un RPI et une VM, avec les log des 2 machines dans une database sur synology.
J'aimerai ajouter l'origine des logs (nom de la machine).

Et comme ca fait un moment que j'ai pas mis le nez dedans... QQ1 a peut être le savoir de comment faire ca :).

Merci.
Arnaud

Cyrille Defranoux

unread,
Feb 5, 2019, 7:24:51 AM2/5/19
to Eibd / Linknx / KnxWeb
Bonjour Arnaud,

Moi je suis toujours là :)

Pour répondre à ta question : puisque tu parles de log dans une DB, je suppose que tu parles des logs des objets et non des logs de l'appli.
Permets-moi de reformuler ta question pour être sûr d'avoir bien compris ton besoin. Tu as :
  • deux machines hébergeant chacune une instance de linknx
  • un synology hébergeant une DB
  • les deux linknx écrivent dans les mêmes tables de la même DB
Et tu veux pouvoir distinguer les entrées en DB venant de chaque instance.

Si j'ai bon, alors la solution la plus simple à mon avis serait d'utiliser des tables différentes. Mais cela n'est peut-être pas acceptable pour toi. Tu peux aussi préfixer les id d'objets différemment pour les deux instances. Ca me semble moins intrusif, même si ça ressemble plus à un hack que la première solution.

Sinon, il serait bien sûr possible d'implémenter une évolution du genre :
<persistence type="mysql" host="localhost" user="linknxtest" pass="" db="linknx_test" table="persist" logtable="log" prefix="machine1" />
afin d'avoir un préfixe inséré automatiquement lors du logging. Mais je ne te cache pas qu'il y a d'autres travaux prioritaires sur linknx donc si tu veux ça rapidement, tu pourrais patcher le fichier persistentstorage.cpp pour l'adapter à ton besoin. Si tu insères le préfixe en dur dans le cpp, il te suffit de compiler deux versions différentes et tu auras ce que tu veux. Pas très élégant mais efficace.

A+
Cyrille

chtiloft

unread,
Feb 5, 2019, 4:51:07 PM2/5/19
to Eibd / Linknx / KnxWeb
Hello Cyrille:) !!

Tu as très bien reformulé !
  • deux machines hébergeant chacune une instance de linknx
  • un synology hébergeant une DB
  • les deux linknx écrivent dans les mêmes tables de la même DB
non c'est pas vraiment ca qui m'intéresse.
en gros, j écris tous les relevés de toutes les machines dans la meme base, donc s il y a doublon (timestamp + object + valeur) , j'ai une erreur, mais pas grave.

en gros, j'aimerai ca :
Je reste sur le meme principe, la clé est (timestamp + object + valeur), l'origine est juste de l'info.

C'est quio les travaux sur linknx ???
a+
Arnaud

chtiloft

unread,
Feb 5, 2019, 4:56:16 PM2/5/19
to Eibd / Linknx / KnxWeb
je viens de voir le post refactoring ! bravo.

Il me faudrait me poser un peu pour tester tout ca...

Cyrille Defranoux

unread,
Feb 6, 2019, 12:40:41 AM2/6/19
to eibd-link...@googlegroups.com
Dans ce cas, le plus simple dans l'immédiat serait de patcher linknx pour rajouter une info dans une colonne supplémentaire de la db.

Le plus gros chantier pour linknx est l'élimination de pthsem comme cela a été fait pour knxd. C'est impératif pour assurer la survie du projet.

Cyrille

--
Vous recevez ce message, car vous êtes abonné à un sujet dans le groupe Google Groupes "Eibd / Linknx / KnxWeb".
Visitez ce groupe à l'adresse https://groups.google.com/group/eibd-linknx-knxweb.

Arnaud COGNARD

unread,
Feb 7, 2019, 5:19:55 PM2/7/19
to eibd-link...@googlegroups.com

ok, bon courage, et merci à toi d'avance pour ce boulot !

ps : j'essaie de dégager du temps ce we pour qqes tests.

 

Pour le reste, aucune idée de comment faire ce patch sur linknx (oui oui , on oublie vite les choses...)

 

see u

Arnaud

 

De : eibd-link...@googlegroups.com <eibd-link...@googlegroups.com> De la part de Cyrille Defranoux
Envoyé : mercredi 6 février 2019 06:40
À : eibd-link...@googlegroups.com
Objet : Re: Ajout un champ dans les log

--
Vous recevez ce message, car vous êtes abonné au groupe Google Groupes "Eibd / Linknx / KnxWeb".

image001.png
Reply all
Reply to author
Forward
0 new messages