270 ha addon add syntax check to config parameters (#271)

* quotation marks removed from monitor_sn

* validation for serial, ports and client_mode_host

* removed TODO:

* allow only serial with 16 digit and starting with R17, Y17, Y47

---------

Co-authored-by: Michael Metz <michael.metz@siemens.com>
This commit is contained in:
metzi
2025-01-13 19:49:12 +01:00
committed by GitHub
parent 962f6ee5fb
commit cf1563dd55
2 changed files with 8 additions and 7 deletions

View File

@@ -67,7 +67,7 @@ Example add-on configuration for GEN3PLUS inverters:
```yaml
inverters:
- serial: Y17000000000000
monitor_sn: '2000000000'
monitor_sn: 2000000000
node_id: PV-Garage
suggested_area: Garage
modbus_polling: true

View File

@@ -31,16 +31,16 @@ ports:
# parameters are available within the container as /data/options.json
# and should become picked up by the proxy - current workaround as a transfer script
# TODO: check again for multi hierarchie parameters
# TODO: implement direct reading of the configuration file
schema:
inverters:
- serial: str
- serial: match(^(R17|Y17|Y47).{13}$)
monitor_sn: int?
node_id: str
suggested_area: str
modbus_polling: bool
client_mode.host: str?
client_mode.port: int?
client_mode.host: match(\b((25[0-5]|2[0-4][0-9]|1[0-9]{2}|[1-9]?[0-9])\.){3}(25[0-5]|2[0-4][0-9]|1[0-9]{2}|[1-9]?[0-9])\b)
client_mode.port: port?
client_mode.forward: bool?
#strings: # leider funktioniert es nicht die folgenden 3 parameter im schema aufzulisten. möglicherweise wird die verschachtelung nicht unterstützt.
# - string: str
@@ -62,10 +62,11 @@ schema:
tsun.enabled: bool
solarman.enabled: bool
inverters.allow_all: bool
# optionale parameter
# TODO besser strukturieren und vervollständigen
mqtt.host: str?
mqtt.port: int?
mqtt.port: port?
mqtt.user: str?
mqtt.passwd: password?
ha.auto_conf_prefix: str? # suggeriert optionale konfigurationsoption -> es darf jedoch kein default unter "options" angegeben werden