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

Ring integration error #114334

Closed
firstof9 opened this issue Mar 27, 2024 · 10 comments
Closed

Ring integration error #114334

firstof9 opened this issue Mar 27, 2024 · 10 comments

Comments

@firstof9
Copy link
Contributor

The problem

Ring entities are not loading.

What version of Home Assistant Core has the issue?

core-2024.4.0b0

What was the last working version of Home Assistant Core?

core-2024.3.4

What type of installation are you running?

Home Assistant Container

Integration causing the issue

ring

Link to integration documentation on our website

https://www.home-assistant.io/integrations/ring/

Diagnostics information

config_entry-ring-564dc8f59189f053cd83f624e8387585.json

Example YAML snippet

n/a

Anything in the logs that might be useful for us?

2024-03-27 12:38:40.319 ERROR (MainThread) [homeassistant.components.binary_sensor] Error while setting up ring platform for binary_sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 356, in _async_setup_platform
    await asyncio.shield(awaitable)
  File "/usr/src/homeassistant/homeassistant/components/ring/binary_sensor.py", line 58, in async_setup_entry
    RingBinarySensor(ring, device, notifications_coordinator, description)
  File "/usr/src/homeassistant/homeassistant/components/ring/binary_sensor.py", line 82, in __init__
    super().__init__(
  File "/usr/src/homeassistant/homeassistant/components/ring/entity.py", line 70, in __init__
    identifiers={(DOMAIN, device.device_id)},  # device_id is the mac
                          ^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/ring_doorbell/generic.py", line 61, in device_id
    return self._attrs["device_id"]
           ~~~~~~~~~~~^^^^^^^^^^^^^
KeyError: 'device_id'

Additional information

No response

@home-assistant
Copy link

Hey there @sdb9696, mind taking a look at this issue as it has been labeled with an integration (ring) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of ring can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign ring Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


ring documentation
ring source
(message by IssueLinks)

@joostlek joostlek added this to the 2024.4.0 milestone Mar 27, 2024
@frenck frenck modified the milestone: 2024.4.0 Mar 27, 2024
@paddywills
Copy link

Same issue, cameras not loading

@sdb9696
Copy link
Contributor

sdb9696 commented Apr 2, 2024

Hi @firstof9 it seems like you may have an amazon camera linked to your ring account and that is causing the error? You said this was working in 2024.3.4 but I wouldn't expect it to in that version either. Is this suddenly preventing your doorbell from being added when it was working before? Any information you could add would be helpful.

Update: I think the issue here is caused by the new intercom changes along with your third party camera. I have a PR open to fix this #114631.

@firstof9
Copy link
Contributor Author

firstof9 commented Apr 2, 2024

it seems like you may have an amazon camera linked to your ring account and that is causing the error?

Incorrect, it's a Ring Doorbell, I've lost my motion sensors, ding status, etc.

@sdb9696
Copy link
Contributor

sdb9696 commented Apr 2, 2024

There's definitely another camera linked to your account showing in the diagnostics you uploaded and that is breaking the Doorbell. This should be fixed by the PR referenced above.

@firstof9
Copy link
Contributor Author

firstof9 commented Apr 2, 2024

Now I wonder where this hidden camera is...

@sdb9696
Copy link
Contributor

sdb9696 commented Apr 2, 2024

If you look at the second device in the "Diagnostics information" json you uploaded does it give any clues?

@firstof9
Copy link
Contributor Author

firstof9 commented Apr 2, 2024

No idea, I only have a ring doorbell, the only other amazon thing is a single Amazon Dot and as far as I am aware it hasn't a camera.

@sdb9696
Copy link
Contributor

sdb9696 commented Apr 2, 2024

Putting the mystery aside for a moment, 2024.4.0b6 should be available now for you to test the fix.

@firstof9
Copy link
Contributor Author

firstof9 commented Apr 2, 2024

image
Confirmed, back in business.

This is the only device to show up as well.
image

Cheers!

@frenck frenck closed this as not planned Won't fix, can't repro, duplicate, stale Apr 3, 2024
@github-actions github-actions bot locked and limited conversation to collaborators May 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants