jeans/home-assistant.io
jeans/home-assistant.io
1
0
Fork 0
mirror of https://github.com/home-assistant/home-assistant.io.git synced 2025-07-23 01:06:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

Host parameter should be named ip_address (#13032)

Browse Source 
The extension does not work with the 'host' :
Invalid config for [apache_kafka]: [host] is an invalid option for [apache_kafka]. Check: apache_kafka->apache_kafka->host. (See /home/joris/.homeassistant/configuration.yaml, line 18). Please check the docs at https://www.home-assistant.io/integrations/apache_kafka. 

Looking at the code it should be 'ip_address'. Tested and works. 
As this is my first PR, maybe a better solution would be to change the code to use host instead of ip_address. I could not find any documentation about this and this fix is less errorprone than changing the code.
This commit is contained in:
joris1989 2020-04-17 15:44:23 +02:00 committed by GitHub
parent d0eb55404b
commit eb46dab220
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
source/_integrations/apache_kafka.markdown
View File

@ -20,13 +20,13 @@ To use the `apache_kafka` integration in your installation, add the following to
```yaml ```yaml
apache_kafka: apache_kafka:
host: localhost ip_address: localhost
port: 9092 port: 9092
topic: home_assistant_1 topic: home_assistant_1
``` ```
{% configuration %} {% configuration %}
host: ip_address:
description: The IP address or hostname of an Apache Kafka cluster. description: The IP address or hostname of an Apache Kafka cluster.
required: true required: true
type: string type: string