-
Notifications
You must be signed in to change notification settings - Fork 194
-
Notifications
You must be signed in to change notification settings - Fork 194
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
dbus[70]: arguments to dbus_connection_get_object_path_data() were incorrect #79
Comments
Appreciate the report, but since docker isn't the default method for running this script (although it may be in the future), I'll go ahead and close this. Please follow up if you notice anything negative happening. Thanks! |
i know that this issue is closed but i managed to fix the bluetooth problem under docker. i'm using
i managed to create an image using this Docker file:
the run.sh file
my docker-compose.
just run |
Thanks for the follow-up! |
@hatemzidi do you have a working dockerfile with mosquitto 1.5+ for the latest monitor update? |
@Sjeff since i'm using mosquitto with other services, i just added it in my docker-compose as a separate service and linked monitor with mosquitto. |
@hatemzidi can you share the docker-compose file? |
@Sjeff
since monitor is in host mode, i call the mosquitto using the loopback (localhost:1883) ps: my images aren't shared yet on docker hub. |
Thanks @hatemzidi compose file is kinda similar to mine. When updating to latest version 0.2.068 we need mosquitto_pub and mosquitto_sub version 1.5+. |
@Sjeff i just verified my mosquitto image and it’s v1.5.8
|
When i update the monitor image to the latest version :
Dockerfile:
mosquitto version:
mosquitto log:
|
did you force the version in your preferences ?
|
Here are the mosquitto_preferences: same error occurse when i use
|
@Sjeff
i have a feeling that this package isn't updated. |
From inside the monitor container.
|
aha ... here is it ... |
yes we should be able to add : https://packages.debian.org/buster/mosquitto-clients |
When building the dockerfile FROM debian:buster-slim the problem seems to be solved.
|
@hatemzidi I am trying to build on my Intel NUC monitor.sh with your docker files, I am almost there but when on Home Assistant restart publish the topic "monito/scan/restart" to refresh the devices I get this: monitor.sh | monitor.sh: line 1201: systemctl: command not found any help? |
I will probably have to fix this on my side. The restart command doesn't work easily with docker. |
Could you please share your latest Dockerfile, docker-compose and run.sh? |
@xenio hi, sorry for the late reply. as @andrewjfreyer said, i guess that's how monitor works for restart. my dockerfile above is to update too, as @Sjeff montioned by changing the image to |
@xenio you can always trigger a restart of monitor by restarting the container |
Running Intel NUC Docker Configuration.
After i updated to latest beta i receive the following error.
Looks like everything is working as is should be.
Thought i would be good to report this error anyway.
The text was updated successfully, but these errors were encountered: