-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
DSMR-reader v5 #264
Comments
Goed werk Dennis! Ik ben benieuwd! Ik maak alvast een branch aan voor v5 zodat ik ook voor Docker (iig voor mezelf) een Docker release kan testen. Je hebt al genoeg moois gedaan dus laat dat beetje effort op Docker maar bij mij liggen hoor, mits je er natuurlijk mee bezig wilt zijn om meer over Docker te weten te komen. Het is erg druk op mijn werk maar wanneer we een keer iets inplannen komt het vast goed om een keer met elkaar te spreken. |
Lekkere samenwerking 👍. Als ik jullie beide ergens mee kan helpen.... dan hoor ik het graag! |
Branch ready misschien dat jullie het even na kunnen lopen? Ik ga er vanavond een devcontainer aan toevoegen, dan kan ik meteen eenvoudig lokaal in Docker testen vanuit vscode. |
Ik heb even een rc-release voor je getagged:
|
Dat zeggende hebben maakt de rc1 tag de update checker in DSMR-reader stuk: https://github.com/dsmrreader/dsmr-reader/runs/5010925150?check_suite_focus=true#step:11:141 Ironisch, maar moet ik dus ook nog even fixen, want dat is wel superlelijk. |
Wacht nog even met het gebruiken van de tag dus. Wellicht dat ik het formaat iets anders moet doen, want het is niet helemaal semver zie ik: https://semver.org/spec/v2.0.0-rc.2.html |
Oke ik ben er uit. Ik moet het sowieso fixen in de v4-releases, maar aangezien het lange tijd duurt voordat iedereen de melding ziet of upgrade, kan ik er niet op wachten dus zullen we toch die rc's aanmaken. Alleen verwijder ik die op een later moment wel, nadat v5.0 stable gereleased is. Fixes via dsmrreader/dsmr-reader#1566 https://api.github.com/repos/dsmrreader/dsmr-reader/releases Plus dat ik dan ook |
Ik zal bovenstaande morgen als v4.19.1 releasen. |
Inmiddels is er een tweede release candidate. Ook deze verwijder ik later bij de v5.0 stable release. |
Ik zal vandaag een Docker release maken op basis van de RC van v5.0, en die ook testen. Ik verwacht dat er niet veel aangepast hoeft te worden. Maar mijn complimenten, goed begzig geweest! |
Helaas heb ik wat issues met S6-overlay (niet gerelateerd aan DSMR Reader 😃). Ik probeer er vanavond nog even in te duiken maar het kan zijn dat het morgenvroeg wordt: https://github.com/xirixiz/dsmr-reader-docker/runs/5027145887?check_suite_focus=true |
v4.19.1 is nu ook gereleased met een bugfix voor de update checker: https://github.com/dsmrreader/dsmr-reader/releases/tag/v4.19.1 |
Wel fijn overigens dat je het allemaal naar Actions hebt verplaatst. Alles op 1 plek en ook lekker inzichtelijk zo! |
S6 heeft versie 3.x released waarin veel veranderd is. Ik draai voorlopig nog even op de laatste 2.2.x variant. De build loopt nu dus: https://github.com/xirixiz/dsmr-reader-docker/runs/5027290554?check_suite_focus=true De master branch zal ik morgen bijwerken zodat de laatste "productie" Docker image gebasseerd is op verdie v4.19.1. |
Maar idd, het was even wat werk, maar wel beter dat het nu in GH actions staat idd. Azure DevOps was ook fijn, maar nu kan iedereen de pipeline zien en ben ik van een complexe Makefile af. |
Voor die laatste fout met PIL, ik had zelf deze nodig op Debian-like systems:
Het is niet exact dezelfde fout, maar ging wel op PIL stuk in ieder geval. |
Helaas kreeg ik een fout bij het installeren van de requirements, waarschijnlijk vanwege de Docker base image die ik gebruik:
Package: jpeg-dev toegevoegen aan de Dockerfile heeft het issue verholpen. Build succesvol! |
Kan het zijn dat het pad is aangepast van var/tmp naar tmp?
|
Ja klopt! Ik zal het nog toevoegen aan de changelog, want het is een change, maar alleen gerelateerd aan hoe het gehost/gedraaid wordt. Stond er niet bij stil voor Docker! |
Het is wat slordig aan mijn kant, want er was wel een dedicated issue voor en ook gekoppeld aan v5, maar blijkbaar nog niet in de changelog. Ik zal hem toevoegen! |
De development release werkt helemaal naar behoren voor zover ik het kan testen. Ik had er aan de Docker kant nog een minor bugje inzitten en heb die inmiddels ook opgelost (versie weergave bij starten Docker container) |
Gisterenavond viel mij het trouwens op dat er een soort dark mode actief was, niet op de frontend, maar bij het Admin panel. |
Oke top, dank! Dan is het denk ik vooral aan wat gebruikers die wachten op InfluxDB 2 enzo om te gaan testen. Inhoudelijk is DSMR-reader weinig veranderd, veel backwards incompatible vooral. Idem voor de koppeling met Dropbox, die moet ook "opnieuw" bij iedereen die upgrade. Hoe kunnen ze testen? Door de |
Goeie, ik heb dat zelf nog niet gezien. Misschien is het een feature van de Django Admin interface/module zelf. Was het toevallig op je mobiel? |
@xirixiz draait v5rc2 bij jou nog steeds goed? En heb je nog issues vernomen van v5-gebruikers? Al zullen het er hooguit een handvol zijn die het uberahupt geprobeerd hebben. Als er verder geen grote issues zijn, dan release ik vanavond of morgen wel. |
Yes! Geen issues tegengekomen. |
Mocht er trouwens nog wat toelichting nodig zijn in je docs qua backwards-incompatibility, laat maar weten. Ik kan altijd even een opzetje maken. Makkelijkste is om gewoon naar de v5.0 changelog in DSMR-reader zelf te verwijzen, met een waarschuwing dat het niet perse een naadloze upgrade is. Via Docker is wel de makkelijkste manier, maar afhankelijk van de gebruikte features kan er alsnog wat handwerk voor de gebruikers inzitten qua herconfigureren. |
Ik heb in DSMR-reader's github nu de default clone branch ook bijgewerkt naar Alleen laat ik de twee v5.x release candidates nog even (wellicht een weekje) staan, zodat nog niet iedereen de upgrade notificatie krijgt (als het goed is). Ik verwacht dat dat het aantal gebruikers drukt dat direct een upgrade doet. |
De ervaring is dat de meeste mensen er wel uitkomen wanneer ze met Docker werken. Al moet ik (oneerbiedig) bekennen dat Jan met de pet tegenwoordig Docker gebruikt zonder echt te weten hoe het werkt, waardoor er toch vaker vragen opduiken die eigenlijk niet DSMR maar Docker gerelateerd zijn. Nou ja, we gaan het zien 😄 Als je wilt kan ik wel v5.0.0 in "productie" uitbrengen, dan blijft het aantal upgrades nu beperkt tot de Docker gebruikers en kunnen we kijken welke issues daarbij naar voren komen? |
Done! v5.0 is "live" in Docker.... ben benieuwd! |
Wederom dank voor jullie werk, @xirixiz and @dennissiemensma |
Thanks voor jouw hulp Rogier! Goed om te weten dat het voor jou ook naar behoren werkt! |
Bedankt voor jullie super werk! Ik heb nog een deel uit de portainer log kunnen halen. s6-svwait: fatal: supervisor died. ik gebruik docker-compose met de 'latest' tag (zoals al ruim 2 jaar). |
Dit is een portainer issue. Kijk even in de closed issue section en zoek daar op portainer. |
Hey! Dank voor al het werk! edit: seems I lack patience... thanks again! |
De error komt door je healthcheck voor de Postgres database: Verder zie ik in de logs ook web requests voorbij komen, waarom werkt het volgens jou niet? Ik heb zelf 4 verschillende instances omgezet, gewoon de nieuwste image binnen gehaald vanaf 4.19 en ze werken allemaal zonder problemen! :) |
Ik was weer eens te snel met oordelen, hij moest zijn first boot nog afmaken ^^ het werkte gewoon na een refresh na het posten hier... Ga inderdaad wel even achter die fout melding aan, thanks. |
Thanks, fijn!
Ja eens, al vind ik dat we daar als developers wel een rol in hebben om gebruikers er enigzins voor te beschermen. Dat zeggende hebben vind ik het ook weer lastig om de exacte grens daarin te bepalen. Waar ik wel naar toe werk is dat ik uiteindelijk alleen nog maar het draaien via Docker/containers ga ondersteunen. Ik merk dat Docker welliswaar zorgt voor complexiteit, maar anderszijds ook een hoop andere issues wegneemt. |
Goed om te horen en mooi dat je ook Docker bent gaan gebruiken. Maak je gebruik van devcontainers in vscode? |
Ja zoiets maar dan in PyCharm: Het maakt het vooral makkelijk om even een InfluxDB of MQTT broker ernaast te draaien en ook te wisselen van hun versies. |
Zojuist ook mijn docker container bijgewerkt van v4.19 naar v5.0, alles lijkt perfect te werken! |
Moeten jullie beide projecten niet mergen? |
Yes, we gaan het daar binnenkort een keer over hebben hoe we daar vorm aan kunnen geven. |
Goed om deze voortgang te lezen heren. Ook met de Home Assistant addon (weer wat complexiteit bovenop het Bram's docker image) lijkt het nu eindelijk te gaan werken. Na wat testen kom ik deze melding tegen: #262 (comment) Is dat nog iets waar de niet-HA addon gebruikers ook tegenaan lopen? |
Het upgraden van 4.20 naar 5.0 was iets meer werk dan de reguliere updates maar inmiddels heb ik succesvol kunnen upgraden naar versie 5. Ik gebruik DSMR-reader nu bijna 2 jaar en werkt altijd probleemloos. DSMR-reader geeft duidelijk inzicht in je energie verbruik en maakt je bewust van je energie verbruik. Dankjewel! |
Ik heb zojuist de twee release candidates verwijderd, zowel qua releases als tags. Hierdoor krijgen naar verwachting alle v4.x gebruikers een melding dat er een nieuwe versie is. Hopelijk upgraden ze eerst rustig naar v4.20 zodat niet alles tegelijkertijd komt. |
Melding is gesloten, maar toch een vraag... ik draai nog op 4.19 (4.19.0 dus denk ik). Hoe kan ik naar 4.20 updaten? Kan ik gewoon |
4.20 heb ik overgeslagen voor DSMR reader in Docker (geen value). Wanneer je dus latest gebruikt ga je direct over op versie 5 van DSMR reader. Is er een reden waarom je eerst naar 4.20 wilt? |
@zuidwijk je kunt nog even de changelog bekijken voor v5. Upgraden via Docker is relatief weinig werk, maar (v5) vereist mogelijk wat extra werk afhankelijk van of je influxdb en dropbox gekoppeld hebt. https://dsmr-reader.readthedocs.io/en/v5/reference/changelog.html#v5-0-0-2022-02-07
|
Omdat ik gelezen had dat je eerst naar de laatste versie van 4 moest updaten voordat je naar 5 kon updaten. Kon er verder ook niet 123 wat van vinden, vandaar m'n vraag. Als ik gewoon latest kan blijven gebruiken zal ik dat wel doen. |
Zojuist even de dockers gestopt, gebackup-t, docker pull en docker-compose up -d gedaan ... works like a charm! |
Request
Het heeft even geduurd, maar DSMR-reader v5 is nu zo goed als af. Ik ben momenteel nog zelf aan het draaien en zal ook een opzet maken voor alle wijzigingen qua envvars voor Docker.
Hoe dan ook zal die niet compatible zijn met v4.x, tenzij je zelf de oude envvars wilt mappen qua namen. Zie de changelog hieronder.
Uiteindelijk zal DSMR-reader v5.0 eerst gereleased worden zodat er rustig getest kan worden. Wellicht dat ik zelfs eerst een release candidate tag, zodat het ook voor jou te splitsen en goed te testen is.
Later volgt DSMR-reader v4.20, met enkele bugfixes (die ook al in v5 zitten), maar vooral met de automatische boodschap dat v5 er is.
Ik verwacht nu geen actie van je hoor, maar dan weet je dat dit er aan komt. Ik zal sowieso zelf even een opzet proberen te maken in de vorm van een PR.
We zouden sowieso nog een keertje contact hebben over het vergemakkelijken van onderhoud, dus wellicht is dat te combineren. 😄
Additional information
Dit is nu een directe link naar de (nog) verborgen documentatie met changelog/upgrade guide (voor niet-Docker):
The text was updated successfully, but these errors were encountered: