Yeah awesome, it looks great!
For the sensor, if I’m going to choose, I’d prefer one with a display for the temperature
Like:
https://www.domadoo.fr/fr/domotique/5789-moes-capteur-de-temperature-humidite-et-luminosite-zigbee.html?domid=17
As for the remote, it’s not a bad idea either
I’ve gone through the products, we already cover a good portion of the features anyway, we have reason to be proud!
They have some really nice stuff, I fell for this one!
https://www.domadoo.fr/fr/peripheriques/5882-heiman-capteur-de-qualite-d-air-co2-temperature-humidite-zigbee-30-alarme-visuelle-et-sonore.html?domid=17
@Tlse-vins thanks for your participation, I’ll get that to you shortly. Thanks thanks <3
@AlexTrovato that works for me
@AlexTrovato Mrs. Claus Gladys Assistant stopped by
1 Like
Functional tests OK for me
Here are the update logs during startup
2023-01-29T20:53:15+0100 <info> checkForContainerUpdates.js:14 (Zigbee2mqttManager.checkForContainerUpdates) Checking for current installed versions and required updates...
2023-01-29T20:53:15+0100 <info> checkForContainerUpdates.js:18 (Zigbee2mqttManager.checkForContainerUpdates) MQTT container: update #2 of the container required...
2023-01-29T20:53:15+0100 <info> checkForContainerUpdates.js:35 (Zigbee2mqttManager.checkForContainerUpdates) MQTT container: update #2 of the container done
2023-01-29T20:53:15+0100 <info> checkForContainerUpdates.js:40 (Zigbee2mqttManager.checkForContainerUpdates) Z2M container: update #2 of the container required...
2023-01-29T20:53:15+0100 <info> checkForContainerUpdates.js:57 (Zigbee2mqttManager.checkForContainerUpdates) Z2M container: update #2 of the container done
2023-01-29T20:53:28+0100 <info> installZ2mContainer.js:31 (Zigbee2mqttManager.installZ2mContainer) Zigbee2mqtt is being installed as Docker container...
2023-01-29T20:53:28+0100 <info> installZ2mContainer.js:32 (Zigbee2mqttManager.installZ2mContainer) Pulling koenkk/zigbee2mqtt:latest image...
2023-01-29T20:53:29+0100 <info> installZ2mContainer.js:35 (Zigbee2mqttManager.installZ2mContainer) Configuration of Device /dev/ttyUSB0
2023-01-29T20:53:29+0100 <info> installZ2mContainer.js:40 (Zigbee2mqttManager.installZ2mContainer) Creation of container...
2023-01-29T20:53:29+0100 <info> installZ2mContainer.js:43 (Zigbee2mqttManager.installZ2mContainer) Zigbee2mqtt successfully installed and configured as Docker container
2023-01-29T20:53:29+0100 <info> installZ2mContainer.js:56 (Zigbee2mqttManager.installZ2mContainer) Preparing Zigbee2mqtt environment...
2023-01-29T20:53:29+0100 <info> installZ2mContainer.js:69 (Zigbee2mqttManager.installZ2mContainer) Zigbee2mqtt container is starting...
2023-01-29T20:53:35+0100 <info> installZ2mContainer.js:75 (Zigbee2mqttManager.installZ2mContainer) Zigbee2mqtt container successfully started
And the new docker zigbee2mqtt instance now contains the option (thanks to the runlike tool):--log-opt max-size=10m
Nice job
2 Likes
The key was sent this morning, you should receive it before the end of the week.
2 Likes
@Tlse-vins je viens de rentrer chez moi, j’ai bien reçu la clé !!
Je suis en train de tenter de connecter les appareils reçus en même temps, mais je suis un débutant en la matière.
Merci encore à tous !!!
3 Likes
I haven’t used it in a while.
1 Like
Indeed, I had to flash the key, but everything is working perfectly, the devices too.@pierre-gilles and @Tlse-vins
4 Likes
@pierre-gilles it would seem that this PR is ready:
master ← atrovato:z2m-docker-limit-logs
ouvert 09:35AM - 21 Jan 23 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [x] If your changes affects code, did your write the tests?
- [x] Are tests passing? (`npm test` on both front/server)
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- ~~[ ] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)~~
- [ ] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to [the community](https://community.gladysassistant.com/) for testing before merging.
- ~~[ ] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)~~
- ~~[ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo]~~(https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- ~~[ ] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).~~
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Add log limit on Z2M containers.
I add a new service variable to track the container update.
Image : `atrovato/gladys:z2m-log-limit`
⚠️ I am not able to test it on my side due to lack of devices.
I’ve just updated it with master.
AlexTrovato:
Indeed, I had to flash the key, but everything works perfectly, the devices as well.@pierre-gilles and @Tlse-vins
Great
Yes, I was waiting for you to come back from vacation to merge, it’s fine with me then if it has been functionally tested.
I’ve just merged!
Thanks @cicoub13 for the tests
1 Like
Can you send me your address in a private message?
Keep us posted!