Skip to content
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

Unable to configure discovered devices #1

Open
chedim opened this issue Nov 19, 2023 · 3 comments
Open

Unable to configure discovered devices #1

chedim opened this issue Nov 19, 2023 · 3 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@chedim
Copy link
Contributor

chedim commented Nov 19, 2023

Great work on the plugin, thank you for creating it!

Right now I'm receiving ir-ble data from my solar charger and shunt using Signalk victron-ble plugin and then I push that data to HA through MQTT. Would really love to streamline the setup and was very excited about finding this plugin!

But:

After installing through HACS and restarting HA, both of my devices were discovered, but when I try to configure it, a configuration window with no inputs and only "Submit" button appears. Clicking "Submit" adds the devices in "failed to configure" state.

HA is running as a Docker container.

@chedim
Copy link
Contributor Author

chedim commented Nov 19, 2023

Got the following error messages in HA logs:

2023-11-19 14:43:17.955 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Sudo 12v Charger for victron_ir
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 402, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/victron_ir/__init__.py", line 24, in async_setup_entry
    data = VictronInstantReadoutData(entry.options[ENC_KEY])
                                     ~~~~~~~~~~~~~^^^^^^^^^
KeyError: 'enckey'
2023-11-19 14:43:25.030 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry Sudo 48v Shunt for victron_ir
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 402, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/victron_ir/__init__.py", line 24, in async_setup_entry
    data = VictronInstantReadoutData(entry.options[ENC_KEY])
                                     ~~~~~~~~~~~~~^^^^^^^^^
KeyError: 'enckey'

@chedim
Copy link
Contributor Author

chedim commented Nov 19, 2023

Update: I was able to configure the devices by adding Victron integrations manually from Devices configuration page.

@SWW13
Copy link
Owner

SWW13 commented Nov 20, 2023

Yeah this is a "known" issue, because i couldn't figure out how to make the discovery workflow work as expected.
Thanks for reporting anyway, maybe we can figure out how to fix it.

@SWW13 SWW13 added bug Something isn't working help wanted Extra attention is needed labels Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants