-
-
Notifications
You must be signed in to change notification settings - Fork 30.4k
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
Revert "Allow MQTT device based auto discovery" #118746
Conversation
This reverts commit 585892f.
Hey there @emontnemery, @bdraco, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
@@ -91,13 +82,6 @@ def async_log_discovery_origin_info( | |||
message: str, discovery_payload: MQTTDiscoveryPayload, level: int = logging.INFO | |||
) -> None: | |||
"""Log information about the discovery and origin.""" | |||
# We only log origin info once per device discovery | |||
if not _LOGGER.isEnabledFor(level): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should remerge this in a separate pr
|
||
|
||
@callback | ||
def _replace_all_abbreviations(discovery_payload: Any | dict[str, Any]) -> None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some good improvements here that we could split into another PR as well while this is being reworked
#118757 is the rework |
Revert "Allow MQTT device based auto discovery (home-assistant#109030)" This reverts commit 585892f.
Reverts #109030 which introduced a new schema for MQTT discovery
The main benefit of the new schema is that discovery of many (1800+) MQTT entities is more reliable, simply thanks to the reduction of discovery messages by roughly a factor of 10.
However, there were some unresolved questions: