bieniu / ha-shellies-discovery-gen2

Script that adds MQTT discovery support for Shellies Gen2 devices
Apache License 2.0
106 stars 18 forks source link

Shelly plus smoke - non discovered by MQTT #477

Closed balgerion closed 1 month ago

balgerion commented 1 month ago

Describe the bug Shelly plus smoke configured (with RPC) and visible in MQTT broker (hosted on HA) {3BB8518F-A72D-4C40-9F8A-929D165F5DFA}

Gen2 script started manually (few times) while device is awake (hold button to wake/test shelly smoke)

{AF99D289-6610-4FEA-AC8A-B5A4B0FEC788}

via shelly API , HA autodiscover works without issue - this is to confirm network connection. {61D3E023-EE02-49C3-8367-68761B8C25A4}

Expected behavior: show shelly smoke as a recognized device in MQTT

Versions:

Shellies Discovery GEN2 automation:

Shelly device MQTT settings: image

bieniu commented 1 month ago

Show me please shellies announce automation and a debug log.

How to catch a debug log -> https://github.com/bieniu/ha-shellies-discovery-gen2?tab=readme-ov-file#how-to-debug

balgerion commented 1 month ago

home-assistant.log {F95C9E65-593D-49B4-8EA7-CB64D9CE34DA} {DE0C19A2-09CE-40AE-BF51-897FF2ABF787}

im using both scripts gen1 and gen2 (gen1 works awesome - you are a legend bro!)

bieniu commented 1 month ago

you are a legend bro!

Thanks!

device_ids:

  • shellyplus2pm-485519a1ff8c
  • custom-prefix/shelly-kitchen

Where is the Plus Smoke ID?

balgerion commented 1 month ago

There is none , but smoke is communicating with mqtt (screenshot from mqtt explorer) and via Shelly api to HA. And I don’t own any Shelly 2pm device. :(

balgerion commented 1 month ago

{8A236ACF-CE97-49EE-B98E-DECDD33C0106} log from python: home-assistant_python_script_2024-10-05T04-42-03.419Z.log

shelly smoke factory rested and added via shellly api for test: {25CF75FA-CB73-410A-AF94-2FF98801F86B}

bieniu commented 1 month ago

And I don’t own any Shelly 2pm device. :(

You copied a sample automation and didn't fill it with your own data. You need to enter your device ID here. ID = MQTT prefix.

balgerion commented 1 month ago

What a noob error! Thank you, and sry for trouble. Everything is working.

bieniu commented 1 month ago

No worries