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

Multiple PCs integrated into Home Assistant (Add a second host in the integration config) #11

Closed
LeoLTM opened this issue Aug 30, 2021 · 9 comments
Labels
bug Something isn't working

Comments

@LeoLTM
Copy link

LeoLTM commented Aug 30, 2021

I want to integrate my two machines running OpenRGB into Home Assistant.
The setup of the first one worked perfect (Installed OpenRGB, started server, installed HACS Integration, added in integrations section, entered IP of the first desktop), but I was not able to find a way to add a second desktop with a different IP address to the openRGB integration.

@koying koying added the bug Something isn't working label Sep 1, 2021
@malkie-hass
Copy link

Same issue here. Would like to add more than only one PC.

@Sjorsa
Copy link

Sjorsa commented Jan 16, 2022

I get this "aborted" error when trying to add a 2nd PC.
image
I guess the integration is not set up to handle multiple instances?

@LeoLTM
Copy link
Author

LeoLTM commented Jan 16, 2022

Yeah, sadly there is no way to connect a 2nd or 3rd machine yet... Would be very useful!

@WhimsySpoon
Copy link

Agreed. Have multiple PCs in a single room and would love to be able to control the colours at the same time.

@stoniwankenobii
Copy link

If not multiple PCs but multiple IPs, I guess loosely multiple PCs, would be fantastic! I sometimes forget to plug my ethernet cable back into my main PC and would love to add both IP addresses for my WiFi MAC and Ethernet MAC to ensure I am always able to control my lights. I use this extension with Lumia Streams integration of Home Assistant to help add effects for things when I stream. Other than this no issue at all!

@koying koying closed this as completed in ce0b1c4 Feb 5, 2022
@WhimsySpoon
Copy link

Thanks for taking care of this, @koying! Unfortunately I've encountered a minor bug with this. When adding a second instance of the integration, it finds the new devices from the new computer, but also the devices found from the first instance.

For example, if the first instance has 4 devices, and the second instance has 3, upon adding the second computer, 7 devices are reported as being added.

The devices for the first computer are now tied to both instances of the integration (see last four devices in the screenshot below):

image

@WhimsySpoon
Copy link

Been experimenting with this a bit more. When a second device is added, the entities from the first device are sometimes allocated to the second device, and the ones for the second device are lost.

I've been poking around in config_entities, device_registry and entity_registry and when additional devices are added, the links between the entities -> devices become mismatched.

What might be relevant: I have two identical computers that I'm adding, so the devices will share the same model numbers etc.

@WhimsySpoon
Copy link

WhimsySpoon commented Feb 12, 2022

I have removed both instances, added one, then set logging level for the component to debug then added a second instance. The following appeared in the log:

2022-02-12 21:54:59 INFO (MainThread) [homeassistant.components.light] Setting up light.openrgb
2022-02-12 21:54:59 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new light.openrgb entity: light.asus_rog_strix_rtx_2070s_a8g_gaming_8g_0
2022-02-12 21:54:59 ERROR (MainThread) [homeassistant.components.light] Platform openrgb does not generate unique IDs. ID light.corsair_vengeance_pro_rgb_1 already exists - ignoring light.ddp_desktop_01_ram_0
2022-02-12 21:54:59 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new light.openrgb entity: light.corsair_vengeance_pro_rgb_2
2022-02-12 21:54:59 INFO (MainThread) [homeassistant.helpers.entity_registry] Registered new light.openrgb entity: light.asus_rog_crosshair_vii_hero_wi_fi_3
2022-02-12 21:54:59 ERROR (MainThread) [homeassistant.components.light] Platform openrgb does not generate unique IDs. ID 00000000001A already exists - ignoring light.ddp_desktop_01_case
2022-02-12 21:55:11 ERROR (MainThread) [homeassistant.components.light] Platform openrgb does not generate unique IDs. ID light.corsair_vengeance_pro_rgb_0 already exists - ignoring light.ddp_desktop_01_ram_1
2022-02-12 21:55:11 ERROR (MainThread) [homeassistant.components.light] Platform openrgb does not generate unique IDs. ID light.corsair_vengeance_pro_rgb_1 already exists - ignoring light.ddp_desktop_01_ram_0
2022-02-12 21:55:11 ERROR (MainThread) [homeassistant.components.light] Platform openrgb does not generate unique IDs. ID light.asus_rog_crosshair_vii_hero_(wi-fi)_2 already exists - ignoring light.ddp_desktop_01_motherboard
2022-02-12 21:55:11 ERROR (MainThread) [homeassistant.components.light] Platform openrgb does not generate unique IDs. ID 00000000001A already exists - ignoring light.ddp_desktop_01_case

The devices that share the same name will reappear as ERRORs in the log every ~20 seconds after the initial view. It appears that the component is attempting to add the entities using their model names name. One of my computers has a different GPU which can be added without issue (light.asus_rog_strix_rtx_2070s_a8g_gaming_8g_0).

@WhimsySpoon
Copy link

Latest release (2.2) resolves this. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

6 participants