Add integration with MQTT for GRDF #20
Conversation
|
Je ne sais pas pourquoi le docker file est affecté. J'ai juste pris ton repo et crée un fork en ne changent que le py |
C'est l'intégration continue qui applique pre-commit dans lequel j'ai mis en place divers outils de vérification + le "début de flow" pour pousser des images pour haos fait ses vérifs aussi. J'ai poussé une mise à jour en appliquant "yamllint" et "prettier" et en replacant le nom de l'image à pousser par une variable. |
|
J'ai créé un compte sur dokcer hub, et je vois que le nom de l'image ne pourra pas être mdeweerd/... car je ne peux pas créer ce 'namespace'. |
|
Mais, avant t'as pas eu qq sur docker, non? C'est pas obligatoire. |
|
Oui, en fait j'aimerai bien publier les images pour HAOS et éventuellement une image tel que tu l'a définit là par exemple. |
|
Pour info: pre-commit s'install en local - pip install pre-commit. Puis pre-commit install dans le répertoire de travail. |
|
Plus de temps pour me concentrer :) ... js suis en route... j'espère de tourver du temps la semaine prochaine au plus tôt |
|
J'ai aussi mis à jour la config pre-commit (workflow github). Sur le fond c'est pareil, mais je comptais faire cela avant la prochaine release. |
|
je m;'excuse mais je ne peux pas donner suite a n'importe quoi .. pour les jours qui viennent. on en parleravia discord (vingerha) |
attribs start with the type, and end with the 'unit', this was not the case for 'period' attribs
…into meters-to-ha
|
Dernier commit: alignement des noms des attributes :) |
|
Un peu de changement en // - j'ai apporté des modifications - normalement tout doit passer. |
|
le pre-commit passe que pour 2 soucis autour les auth=auth ... je dois chercher pourquoi |
Je l'ai réglé - il fallait juste mettre "AuthParameters" comme type (juste=quand même pas très documenté). |
|
J'ai aussi mon propre image maintenant, pas encore eu le temps de voir sur quelle OS :) |
|
L'image par défaut (Dockerfile) c'est Ubuntu. |
|
Je vais modifier l'image par défaut. |
|
J'avais pas vu qu'il y avait 2 build - je viens de mettre la variable pour le Dockerfile dans le premier build aussi. Je ne sais pas pourquoi c'est utile de farie 2 build dans le même CI - je n'ai pas creusé le workflow. |
|
Petit pas pour une solution meilleur qu'avant :) |
|
Mon devis c'est "Faire plus avec moins". Souvent quand j'interviens le code est plus petit, plus rapide, a plus de fonctionnalités mais moins de bogues. Donc quand cela fonctionne pour toi, je peux accepter ce PR. |
|
Je vais tester mon fork avec son image |
only removed empty line
|
Le container restarts chaque 10/15s et pas d'accès vers un shell possible e tpas des logs |
|
create: docker run -itd --net=host --name meters2ha -v /home/arjan/docker/meters2ha:/workdir vingerha/meters2ha:latest |
|
Ah oui, c'est logique car les fichiers docker s'utilisent actuellement avec docker-compose. On peut mettre comme commande par défaut: Et ajouter les variables d'environnement par défaut: et l'option: Le montage du volume reste à la charge de l'utilisateur je pense: Je peux accepter le PR tel que et ces points peuvent être abordés dans un autre. En ce qui concerne le cron - je trouve dommage de laisser un docker actif pour l'exécuter une fois par jour, mais pour certains cela peut-être l'approche souhaité. |
|
ok... continue avec le PR et on ajoute après. |
|
GRDF donne une MAJ tous les jours sauf "incidents" de retard - perso je tente 2 fois en fin de journée car la MAJ se fait peu avant. |
|
Depuis m2ha plus de maj depuis avant-hier. Leur site web ne me donne que l'option vers 19Fev :) ... c'est assez pareil que ave Enedis...ils s'ent foutes de leurs clients |
|
C'est principalement une obligation légale... Il y avait même une loi qui obligait à installer un écran d'information dans les foyers et finalement cela s'est transformé en un affichage dans application mobile - pas le même impact. |
No description provided.