search

moroen / IKEA-Tradfri-plugin

A Python plugin for Domoticz to controll IKEA Gateway
68 stars 23 forks source link

ikea-tradfri-plugin Docker Synology importerror #76

Closed Appiejs closed 5 years ago

Appiejs commented 5 years ago

Hi there, I tried to install the plugin with docker on my Synology, but the plugin keeps restarting in docker with the following message:

Traceback (most recent call last): File "tradfri.tac", line 21, in from ikeatradfri.device_classes import ikeaGroup, ikeaLight, ikeaSocket ImportError: No module named 'ikeatradfri'

I noticed that the tradfri.tac file is modified 14 days ago, maybe it has to do something with the error? If I could try and test something please let me know. Gr Ap

moroen commented 5 years ago

There was a COPY directive missing in the dockerfiles, the latest commit just pushed should fix this.

Regards, M

Appiejs commented 5 years ago

HI thanks for the quick reply. What do I need to do to get the new file?

moroen commented 5 years ago

You need to pull the latest changes using git:

$ cd IKEA-Tradfri $ git pull

Appiejs commented 5 years ago

probably i'm doing something wrong, but still getting the same error. I'm using de docker method on a Synology NAS and walked trough the process from start. After running the the container it's giving the same errors. What am I missing in the process?

moroen commented 5 years ago

I've build a image with the latest changes, and the adapter works. Have you rebuild the docker image without using the cache?

$ docker build --no-cache -t ikea-tradfri-plugin:latest .

Appiejs commented 5 years ago

Hi There, i tried with the no-cache without any luck. Even the log has another message: No module named 'ikeatradfri' from ikeatradfri.device_classes import ikeaGroup, ikeaLight, ikeaSocket File "tradfri.tac", line 21, in Traceback (most recent call last): Please check IP/KEY. Error: Connection to gateway timed out!

any idea

moroen commented 5 years ago

Are you sure you have the latest Dockerfile? Could you please post it here?

Appiejs commented 5 years ago

attached is the file from where i run the command, I think you mean this file right?

Op di 28 mei 2019 om 15:05 schreef moroen notifications@github.com:

Are you sure you have the latest Dockerfile? Could you please post it here?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/moroen/IKEA-Tradfri-plugin/issues/76?email_source=notifications&email_token=AMFBZE75FNGKKXM3NN77QIDPXUUYVA5CNFSM4HPQCHP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWMBXEY#issuecomment-496507795, or mute the thread https://github.com/notifications/unsubscribe-auth/AMFBZE5IUX4JNXQ57D7OEITPXUUYVANCNFSM4HPQCHPQ .

moroen commented 5 years ago

I'm not sure what has happened, but it looks like the latest changes somehow haven't been pushed to the repo. I've made the changes to the dockerfiles now, please pull the latest version and try again!

Appiejs commented 5 years ago

Hi, i see the new docker file, but now i'm back at the first issue. de .tac file is still 18 days old, probably that file is also not synced?

moroen commented 5 years ago

The .tac file does't need changing, my on the fly correction of the dockerfiles were wrong. The versions available now, should work...

Appiejs commented 5 years ago

Perfect that fixed the issue, many thanks!