congatudo / Congatudo

Cloud replacement for Conga vacuum robots enabling local-only operation
https://congatudo.cloud
Apache License 2.0
72 stars 22 forks source link

Congatudo display unknown error in banner #84

Closed Nismonx closed 7 months ago

Nismonx commented 10 months ago

Describe the bug

After update 0.2.5 - 2022-10-21 congatudo is no longer parce messages correctly from conga 4090. ## To Reproduce 1Remove dust bin: should display 'an error occured: dustbin not exist' but instead get 'an unknown error occurred'. 2Turn conga upside-down should display 'an error occurred: wheels up' but yet again it shows unknown error. I'm sure there are other error or messages not parse correctly, those are just the ones I've experienced. ### Screenshots

Screenshot_20230905_093842_Telegram

Old version of congatudo:

Screenshot_20230905_095130_Telegram Screenshot_20230905_095152_Telegram Screenshot_20230905_095807_Telegram

Vacuum Model

Conga 4090

Congatudo Version

Expected behavior

## Additional context

I don't know if this is relevant but it's all it shows in the log from Congatudo add-on:

2023-08-28T09:53:18.245Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Nismonx commented 10 months ago

Relate to issue #77

Only just noticed that 😳

Thanks everyone...

elraro commented 9 months ago

This release is really old. Could you update please?

Nismonx commented 9 months ago

Congatudo 2023.05 May 22 release? I have tried it and get the errors.

That's why I had to revert all the way back to 0.2.4

Screenshot_20231015_163551_Home Assistant

Nismonx commented 9 months ago

I will update to the latest release again and restarting HA and Conga4090.

I'll report back soon 👍

elraro commented 9 months ago

Ok, understood. Please, try again, and provide logs from the container or the Log section from Congatudo, not screenshots.

Thank you!

Nismonx commented 9 months ago

So far so good:

2023-10-15T15:43:19.208Z
Info
Set Logfile to /tmp/valetudo.log
2023-10-15T15:43:19.230Z
Info
Starting Valetudo 2023.05
2023-10-15T15:43:19.232Z
Info
Commit ID: 093d853701a06a4d07aa494c7b84650cb9ea2f3e
2023-10-15T15:43:19.233Z
Info
Configuration file: /config/valetudo.json
2023-10-15T15:43:19.234Z
Info
Logfile: /tmp/valetudo.log
2023-10-15T15:43:19.235Z
Info
Robot: Cecotec Conga (CecotecCongaRobot)
2023-10-15T15:43:19.235Z
Info
JS Runtime Version: v18.12.1-Valetudo
2023-10-15T15:43:19.236Z
Info
Arch: arm64
2023-10-15T15:43:19.238Z
Info
Max Heap Size: 64 MiB
2023-10-15T15:43:19.238Z
Info
Node Flags: --expose-gc --max-heap-size=64
2023-10-15T15:43:19.247Z
Info
Autogenerated System ID: FairWindingDuck
2023-10-15T15:43:23.874Z
Info
Webserver running on port 8080
2023-10-15T15:43:24.324Z
Info
Connected successfully to MQTT broker
2023-10-15T15:43:24.760Z
Info
MQTT configured
2023-10-15T15:43:25.886Z
Info
Added new robot with id '8136'

I'll just need restart everything....
Nismonx commented 9 months ago

OK, so after restarting both,

I have this: (the 3 last errors are normal I think because of restarting)

2023-10-15T15:43:19.208Z
Info
Set Logfile to /tmp/valetudo.log
2023-10-15T15:43:19.230Z
Info
Starting Valetudo 2023.05
2023-10-15T15:43:19.232Z
Info
Commit ID: 093d853701a06a4d07aa494c7b84650cb9ea2f3e
2023-10-15T15:43:19.233Z
Info
Configuration file: /config/valetudo.json
2023-10-15T15:43:19.234Z
Info
Logfile: /tmp/valetudo.log
2023-10-15T15:43:19.235Z
Info
Robot: Cecotec Conga (CecotecCongaRobot)
2023-10-15T15:43:19.235Z
Info
JS Runtime Version: v18.12.1-Valetudo
2023-10-15T15:43:19.236Z
Info
Arch: arm64
2023-10-15T15:43:19.238Z
Info
Max Heap Size: 64 MiB
2023-10-15T15:43:19.238Z
Info
Node Flags: --expose-gc --max-heap-size=64
2023-10-15T15:43:19.247Z
Info
Autogenerated System ID: FairWindingDuck
2023-10-15T15:43:23.874Z
Info
Webserver running on port 8080
2023-10-15T15:43:24.324Z
Info
Connected successfully to MQTT broker
2023-10-15T15:43:24.760Z
Info
MQTT configured
2023-10-15T15:43:25.886Z
Info
Added new robot with id '8136'
2023-10-15T15:49:00.182Z
Error
Failed auto refresh: DomainException: Timeout waiting for response from opcode 'DEVICE_MAPID_GET_CONSUMABLES_PARAM_RSP'
    at Timeout.fail [as _onTimeout] (/snapshot/app/node_modules/@agnoc/core/lib/emitters/robot.emitter.js:1024:16)
    at listOnTimeout (node:internal/timers:564:17)
    at process.processTimers (node:internal/timers:507:7) {
  metadata: undefined
}
2023-10-15T15:49:08.502Z
Error
Error: read ECONNRESET
    at TCP.onStreamRead (node:internal/stream_base_commons:217:20) {
  errno: -104,
  code: 'ECONNRESET',
  syscall: 'read'
}
2023-10-15T15:49:09.625Z
Error
Error: read ECONNRESET
    at TCP.onStreamRead (node:internal/stream_base_commons:217:20) {
  errno: -104,
  code: 'ECONNRESET',
  syscall: 'read'
}

I then remove the dustbin to simulate a message and get this instead:

Edit: (photo rmoved) From the conga log:

2023-10-15T15:55:48.682Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2023-10-15T15:55:49.131Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2023-10-15T15:55:49.666Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2023-10-15T15:55:49.715Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2023-10-15T15:55:49.718Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2023-10-15T15:55:51.673Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2023-10-15T15:55:53.497Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2023-10-15T15:55:55.421Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at runNextTicks (node:internal/process/task_queues:64:3)
    at process.processImmediate (node:internal/timers:442:9)
2023-10-15T15:55:56.149Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async RobotMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:13)
    at async RobotMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/RobotMqttHandle.js:153:9)
2023-10-15T15:55:59.149Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
2023-10-15T15:56:00.668Z
Warn
Failed to parse value to be sent for topic valetudo/MeagerCreamyRail/StatusStateAttribute/error: TypeError: Cannot read properties of undefined (reading 'severity')
    at PropertyMqttHandle.getter (/snapshot/app/backend/lib/mqtt/status/StatusStateMqttHandle.js:115:41)
    at PropertyMqttHandle.get (/snapshot/app/backend/lib/mqtt/handles/PropertyMqttHandle.js:104:21)
    at PropertyMqttHandle.getHomie (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:298:53)
    at MqttController.refresh (/snapshot/app/backend/lib/mqtt/MqttController.js:716:36)
    at PropertyMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:237:31)
    at StatusStateMqttHandle.refresh (/snapshot/app/backend/lib/mqtt/handles/MqttHandle.js:240:25)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

It looks like messages are not being pasrsed correctly for my robot from version: 0.2.5 onwards

elraro commented 9 months ago

This seems a problem in the mqtt. I need to analyze. ETA a few weeks.