Mqtt.py log warning "No matching payload found for entity"

So the end result of all of this is that we have to hack the core to remove such spam to the log? There has to be a better solution for mqtt traffic that is all sent over the same topic (no way around this). My setup is working well, but I dont see why this was added. Should be an optional verbosity or something for people that do want this specific testing. I already have payload_on and off specified like so (snippet):

binary_sensor:
  - platform: mqtt
    name: OpenMQTTGateway
    state_topic: "home/OpenMQTTGateway/LWT"
    payload_on: "Online"
    payload_off: "Offline"
    device_class: "connectivity"
  - platform: mqtt
    name: outlet_1_rf
    state_topic: 'home/433toMQTT'
    payload_on: '4543795'
    payload_off: '4543804'
    retain: true
  - platform: mqtt
    name: outlet_2_rf
    state_topic: 'home/433toMQTT'
    payload_on: '4543939'
    payload_off: '4543948'
    retain: true
  - platform: mqtt
    name: outlet_3_rf
    state_topic: 'home/433toMQTT'
    payload_on: '4544259'
    payload_off: '4544268'
    retain: true
2 Likes

I installed home assistant a few days ago and am hitting this issue with 433mhz motion and window sensors using OpenMQTTGateway on an esp8266 as an MQTT broker. IDK who is “correct” here and if each device should use a different topic or not. For now I am implementing a custom_component by copying mqtt.py from /srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/binary_sensor/mqtt.py and commenting out the three lines that spam my log. I’ll have to try to remember to check for other changes to that file after upgrades now. A modification to OpenMQTTGateway would be nice I guess, with the topic as the sensor ID and “on” as the payload.

yep, It is planned here

1 Like

This fix seems to have been broken in 0.86.0. I have tried making a new folder under custom_components called mqtt/binary_sensor, and placing the file in there, but have had no success.

My Logs are being flooded once again. :frowning:

Are you sure that’s the right folder structure? mqtt.py is supposed to be in /custom_components/binary_sensor/

Or did something change? I’m still on 0.78.3 and haven’t been keeping up with release notes.

Yeah, MQTT has been changed with 0.86.0 from what I can tell. Everything that works under the umbrella of MQTT now goes in an MQTT folder, not the type of item, such as binary_sensor. I hope that makes sense.

I think I have just managed to fix it in Docker, 0.86.0 for anyone else interested.

My Docker container is called home-assistant-homeassistant currently, so rename yours to whatever your container name is and the path to your configs. Mine is at /home/homeassistant.

The custom_components folder now needs to have an MQTT folder, and then the file named binary_sensor.py

SSH into your machine and execute the following (for Docker).

sudo docker cp home-assistant-homeassistant:/usr/src/app/homeassistant/components/mqtt/binary_sensor.py /home/homeassistant/custom_components/mqtt/binary_sensor.py

Open (sudo nano) the binary_sensor.py file and comment out these lines, 144, 145, 146 & 147

else: # Payload is not for this entity
# _LOGGER.warning('No matching payload found'
# ' for entity: %s with state_topic: %s',
# self._config.get(CONF_NAME),
# self._config.get(CONF_STATE_TOPIC))
return

Reboot, and sorted! No more log spam again :slight_smile:

2 Likes

Excellent. I’m sure you just saved me at least an hour of my life for when I do get around to upgrading. Thanks.

Yep, that’s about how long it took me to work it out hahaha.

Can i download the file or find it somewhere as plain text ? I’m not into Unix/Linux at all and failed trying with PuTTY to copy the file.
Rgds. Henning

By the way, I can see no more warnings after upgrading to 0.86.1

Here is my file, with the offending lines #commented out. You need to create a folder in your HA called custom_components and then another one called mqtt then create a file called binary_sensor.py (make a .txt file and rename it to .py) Copy this text, save, and reboot.

I hope that makes sense and helps you.

"""
Support for MQTT binary sensors.

For more details about this platform, please refer to the documentation at
https://home-assistant.io/components/binary_sensor.mqtt/
"""
import logging

import voluptuous as vol

from homeassistant.core import callback
from homeassistant.components import mqtt, binary_sensor
from homeassistant.components.binary_sensor import (
    BinarySensorDevice, DEVICE_CLASSES_SCHEMA)
from homeassistant.const import (
    CONF_FORCE_UPDATE, CONF_NAME, CONF_VALUE_TEMPLATE, CONF_PAYLOAD_ON,
    CONF_PAYLOAD_OFF, CONF_DEVICE_CLASS, CONF_DEVICE)
from homeassistant.components.mqtt import (
    ATTR_DISCOVERY_HASH, CONF_QOS, CONF_STATE_TOPIC, MqttAttributes,
    MqttAvailability, MqttDiscoveryUpdate, MqttEntityDeviceInfo, subscription)
from homeassistant.components.mqtt.discovery import (
    MQTT_DISCOVERY_NEW, clear_discovery_hash)
import homeassistant.helpers.config_validation as cv
from homeassistant.helpers.dispatcher import async_dispatcher_connect
import homeassistant.helpers.event as evt
from homeassistant.helpers.typing import HomeAssistantType, ConfigType

_LOGGER = logging.getLogger(__name__)

DEFAULT_NAME = 'MQTT Binary sensor'
CONF_OFF_DELAY = 'off_delay'
CONF_UNIQUE_ID = 'unique_id'
DEFAULT_PAYLOAD_OFF = 'OFF'
DEFAULT_PAYLOAD_ON = 'ON'
DEFAULT_FORCE_UPDATE = False

DEPENDENCIES = ['mqtt']

PLATFORM_SCHEMA = mqtt.MQTT_RO_PLATFORM_SCHEMA.extend({
    vol.Optional(CONF_NAME, default=DEFAULT_NAME): cv.string,
    vol.Optional(CONF_PAYLOAD_OFF, default=DEFAULT_PAYLOAD_OFF): cv.string,
    vol.Optional(CONF_PAYLOAD_ON, default=DEFAULT_PAYLOAD_ON): cv.string,
    vol.Optional(CONF_DEVICE_CLASS): DEVICE_CLASSES_SCHEMA,
    vol.Optional(CONF_FORCE_UPDATE, default=DEFAULT_FORCE_UPDATE): cv.boolean,
    vol.Optional(CONF_OFF_DELAY):
        vol.All(vol.Coerce(int), vol.Range(min=0)),
    # Integrations should never expose unique_id through configuration.
    # This is an exception because MQTT is a message transport, not a protocol
    vol.Optional(CONF_UNIQUE_ID): cv.string,
    vol.Optional(CONF_DEVICE): mqtt.MQTT_ENTITY_DEVICE_INFO_SCHEMA,
}).extend(mqtt.MQTT_AVAILABILITY_SCHEMA.schema).extend(
    mqtt.MQTT_JSON_ATTRS_SCHEMA.schema)


async def async_setup_platform(hass: HomeAssistantType, config: ConfigType,
                               async_add_entities, discovery_info=None):
    """Set up MQTT binary sensor through configuration.yaml."""
    await _async_setup_entity(config, async_add_entities)


async def async_setup_entry(hass, config_entry, async_add_entities):
    """Set up MQTT binary sensor dynamically through MQTT discovery."""
    async def async_discover(discovery_payload):
        """Discover and add a MQTT binary sensor."""
        try:
            discovery_hash = discovery_payload[ATTR_DISCOVERY_HASH]
            config = PLATFORM_SCHEMA(discovery_payload)
            await _async_setup_entity(config, async_add_entities,
                                      discovery_hash)
        except Exception:
            if discovery_hash:
                clear_discovery_hash(hass, discovery_hash)
            raise

    async_dispatcher_connect(
        hass, MQTT_DISCOVERY_NEW.format(binary_sensor.DOMAIN, 'mqtt'),
        async_discover)


async def _async_setup_entity(config, async_add_entities, discovery_hash=None):
    """Set up the MQTT binary sensor."""
    async_add_entities([MqttBinarySensor(config, discovery_hash)])


class MqttBinarySensor(MqttAttributes, MqttAvailability, MqttDiscoveryUpdate,
                       MqttEntityDeviceInfo, BinarySensorDevice):
    """Representation a binary sensor that is updated by MQTT."""

    def __init__(self, config, discovery_hash):
        """Initialize the MQTT binary sensor."""
        self._config = config
        self._unique_id = config.get(CONF_UNIQUE_ID)
        self._state = None
        self._sub_state = None
        self._delay_listener = None

        device_config = config.get(CONF_DEVICE)

        MqttAttributes.__init__(self, config)
        MqttAvailability.__init__(self, config)
        MqttDiscoveryUpdate.__init__(self, discovery_hash,
                                     self.discovery_update)
        MqttEntityDeviceInfo.__init__(self, device_config)

    async def async_added_to_hass(self):
        """Subscribe mqtt events."""
        await super().async_added_to_hass()
        await self._subscribe_topics()

    async def discovery_update(self, discovery_payload):
        """Handle updated discovery message."""
        config = PLATFORM_SCHEMA(discovery_payload)
        self._config = config
        await self.attributes_discovery_update(config)
        await self.availability_discovery_update(config)
        await self._subscribe_topics()
        self.async_schedule_update_ha_state()

    async def _subscribe_topics(self):
        """(Re)Subscribe to topics."""
        value_template = self._config.get(CONF_VALUE_TEMPLATE)
        if value_template is not None:
            value_template.hass = self.hass

        @callback
        def off_delay_listener(now):
            """Switch device off after a delay."""
            self._delay_listener = None
            self._state = False
            self.async_schedule_update_ha_state()

        @callback
        def state_message_received(_topic, payload, _qos):
            """Handle a new received MQTT state message."""
            value_template = self._config.get(CONF_VALUE_TEMPLATE)
            if value_template is not None:
                payload = value_template.async_render_with_possible_json_value(
                    payload, variables={'entity_id': self.entity_id})
            if payload == self._config.get(CONF_PAYLOAD_ON):
                self._state = True
            elif payload == self._config.get(CONF_PAYLOAD_OFF):
                self._state = False
            else:  # Payload is not for this entity
#                _LOGGER.warning('No matching payload found'
#                                ' for entity: %s with state_topic: %s',
#                                self._config.get(CONF_NAME),
#                                self._config.get(CONF_STATE_TOPIC))
                return

            if self._delay_listener is not None:
                self._delay_listener()
                self._delay_listener = None

            off_delay = self._config.get(CONF_OFF_DELAY)
            if (self._state and off_delay is not None):
                self._delay_listener = evt.async_call_later(
                    self.hass, off_delay, off_delay_listener)

            self.async_schedule_update_ha_state()

        self._sub_state = await subscription.async_subscribe_topics(
            self.hass, self._sub_state,
            {'state_topic': {'topic': self._config.get(CONF_STATE_TOPIC),
                             'msg_callback': state_message_received,
                             'qos': self._config.get(CONF_QOS)}})

    async def async_will_remove_from_hass(self):
        """Unsubscribe when removed."""
        self._sub_state = await subscription.async_unsubscribe_topics(
            self.hass, self._sub_state)
        await MqttAttributes.async_will_remove_from_hass(self)
        await MqttAvailability.async_will_remove_from_hass(self)

    @property
    def should_poll(self):
        """Return the polling state."""
        return False

    @property
    def name(self):
        """Return the name of the binary sensor."""
        return self._config.get(CONF_NAME)

    @property
    def is_on(self):
        """Return true if the binary sensor is on."""
        return self._state

    @property
    def device_class(self):
        """Return the class of this sensor."""
        return self._config.get(CONF_DEVICE_CLASS)

    @property
    def force_update(self):
        """Force update."""
        return self._config.get(CONF_FORCE_UPDATE)

    @property
    def unique_id(self):
        """Return a unique ID."""
        return self._unique_id
2 Likes

Ok everyone slam on the brakes editing the .py files!!! I may have stumbled upon a stupid simple solution that should be much easier for anyone to implement.

The MQTT messages are logged as level “warning”, so just configure the logger to ignore anything less than “error”. Like this in configuration.yaml:

logger:
  default: warning
  logs:
    homeassistant.components.mqtt: error

And done. Everything else (the default: line) will log at “warning” or higher, while MQTT will log at “error” or higher. My log now has the same messages as before I started. I have deleted my custom MQTT component.

Here are the two docs that describe this:


I’m on 0.86.2 now. No idea when the above logger configuration ability was introduced, but it’s there now.

3 Likes

Great solution!

I just wish they would remove the problem for us though. Some bright spark decided it was a good idea to create this logging in the first place and even after being made aware of the problem, no one has fixed it.

Frustrating.

1 Like

As mentioned earlier in this thread, this is the PR that caused the issue. Would be great if this could be reverted to remove this logging error. @pvizeli ?

Are you still getting these warnings?
I did nothing about it, and see no sign of them since upgrading to 0.86.1

I’m not sure, I have edited my binary_sensor.py file, so I won’t know until I remove that file and upgrade - i’m still on 86.0

Edit: I’ve gone back through the changes for 86.0/1/2 and can not find any specific reference to this being changed for 86.1, although I must admit to skimming through the changes a little sometimes, so may have missed it.

I can confirm on 86.2 I was still getting the message, motivating me to mess with the logging severity levels to make it go away.

you are right, couldn’t see it between stupid rflink debug messages :
the login levels approach looks absolutely valid, but does not work on my Hass.io 0.86.1
tried both default: warning and default: error. :slightly_frowning_face:
EDIT: actually, I can see no mqtt warnings in /config/home-assistant.log, but they are still there if I go to http://hassio.local:8123/dev-info - didn’t think it’s not the same…

Thanks a million, kanga_who. Works perfectly ! :+1:
Saved me a lot of time searching for the original souce code to patch myself (couldn’t find it on Github ?)

Still wonder why the he#%¤" this PR hasn’t been reverted months ago. All using 433MHz to Wifi bridges like me are stuck with one topic for multiple sensors. It’s annoying that i have to patch core Home Assistant components to get rid off warnings for no reason at all.

I’ll still continue to patch for now, as the change of log level will remove the warnings in log file. But they are still persistent in HA’s dev-info ??

Welcome to the frustration.

Just keep in mind the logging level solution does NOT remove warnings from the log file. Only those specific to MQTT. For the rest, defined by the default value, you can make it as verbose as you want by choosing, for example, “info”.

I just found that when set to level “warning” I was seeing usual warning in my log that I expected to see. Minus of course the MQTT messages.

1 Like