xirixiz / dsmr-reader-docker

DSMR Reader in Docker.
https://hub.docker.com/r/xirixiz/dsmr-reader-docker
114 stars 33 forks source link

Empty environment variable DOCKER_TARGET_RELEASE #328

Closed hoorna closed 1 year ago

hoorna commented 1 year ago

Support guidelines

I've found an issue and checked that ...

Description

The last published docker image contains the empty DOCKER_TARGET_RELEASE environment variable. I don't know if this variable existed before in previous image releases, but at least it wasn't empty then.

After creating an new DSMR container with a last update and published image (using docker-compose over an SSH connection), I always change the value of one of the environment variables using the Synology Docker package. Since the last release, this is no longer possible because the Synology Docker package does not accept empty variables. In the past I had no problem with that.

I don't know how to fix this. As a workaround, can I remove the DOCKER_TARGET_RELEASE environment variable with impunity or is it a bug in the docker image?

Expected behaviour

-

Actual behaviour

-

Steps to reproduce

-

Docker info

-

Version

-

Docker compose

-

Container logs

-

Additional info

No response

xirixiz commented 1 year ago

HI, the DOCKER_TARGET_RELEASE is only used in the build proces and not required (should even not be defined!) on starting a container. Please do not set this variable and you are fine.

The variable is only meant to set a certain version on the docker image, so there is a distinction between the DSMR Reader version and the Docker image version.

hoorna commented 1 year ago

Bram, ik schrijf maar even in het Nederlands. Dan kan ik me gemakkelijker uitdrukken.

Uit je reactie begrijp ik dat de variabele DOCKER_TARGET_RELEASE alleen wordt gebruikt tijdens het build proces en dat het verder niet meer nodig is. Ik begrijp dat.

De variabele blijft blijkbaar wel achter in de image want hij is na de container build aanwezig terwijl ik hem niet heb gedefinieerd. Mijn punt is dat ik van deze variabele 'last' heb op mijn Synology DiskStation in het Docker package. De variabele is leeg en juist daarom kan ik andere variabelen niet meer wijzigen in het Docker package. Een lege variabele blokkeert dat ik in het Docker package een (gewijzigde) variabele set kan opslaan. Is het mogelijk dat je aan het eind van het build proces de variabele uit de image verwijdert; dan hebben Synology gebruikers zoals ik er geen last meer van.

xirixiz commented 1 year ago

Ah, I see now :).

DSMR Reader Reader Docker release: during startup. Something is wrong in the build process.

hoorna commented 1 year ago

Ga je het build proces aanpassen en komt er dan een nieuwe image release?

xirixiz commented 1 year ago

Ja, ik begrijp het issue. De variabele wordt gedefinieerd in het build proces en alleen gebruikt tijdens het run proces. Ik heb ook een Synology, maar heb dat issue niet eerlijk gezegd. Ik krijg gewoon bij het starten een lege regel te zien.

hoorna commented 1 year ago

probeer nu maar eens (met deze lege variabele) een andere variabele te wijzigen in het docker package. Opslaan lukt niet meer.

xirixiz commented 1 year ago

Ah, je gebruikt de UI van Synology.... daar kom ik nooit hehehe

xirixiz commented 1 year ago

Maar hoe dan ook goed dat je dit gevonden hebt. Ik heb het issue opgelost en een nieuwe build loopt op dit moment.

hoorna commented 1 year ago

Dankjewel voor het compliment. Fijn om te horen dan er een nieuwe build komt. Want dit issue was voor mij wel lastig.

Voor jou informatie: ik gebruiker docker-compose via SSH om de container te maken en daarna gebruik ik de GUI voor het monitoren van de container en voor eventuele aanpassingen.

xirixiz commented 1 year ago

Fixed:

[ INFO ] DSMR Reader Reader release: 5.10.3
[ INFO ] DSMR Reader Reader Docker release: 2023.03.02
hoorna commented 1 year ago

It's working again as normal. Thank you for the very quick (with a few hours) service.