8000 Regression of Flume device/entities not created #77739 in 2022.11 · Issue #81438 · home-assistant/core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Regression of Flume device/entities not created #77739 in 2022.11 #81438

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.

Sign up for GitHub

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

Jump to bottom
Closed
nathan-curtis opened this issue Nov 3, 2022 · 1 comment · Fixed by #81441
Closed

Regression of Flume device/entities not created #77739 in 2022.11 #81438

nathan-curtis opened this issue Nov 3, 2022 · 1 comment · Fixed by #81441
Assignees
@bdraco
Labels
integration: flume
Milestone
2022.11.1

Comments

@nathan-curtis
Copy link

The problem

Issue in Flume device/entities not created #77739 recurs after update to 2022.11.1
Was working in 2022.10.5 (After being broken in previous builds)

Log:

Logger: homeassistant.components.binary_sensor
Source: components/flume/util.py:17
Integration: Binary sensor (documentation, issues)
First occurred: 8:22:35 PM (1 occurrences)
Last logged: 8:22:35 PM

Error while setting up flume platform for binary_sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
await asyncio.shield(task)
File "/usr/src/homeassistant/homeassistant/components/flume/binary_sensor.py", line 94, in async_setup_entry
flume_devices = get_valid_flume_devices(flume_devices)
File "/usr/src/homeassistant/homeassistant/components/flume/util.py", line 14, in get_valid_flume_devices
return [
File "/usr/src/homeassistant/homeassistant/components/flume/util.py", line 17, in
if KEY_DEVICE_LOCATION_NAME in device[KEY_DEVICE_LOCATION]
KeyError: 'location'

What version of Home Assistant Core has the issue?

core-2022.11.1

What was the last working version of Home Assistant Core?

core-2022.10.5

What type of installation are you running?

Home Assistant OS

Integration causing the issue

flume

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.components.binary_sensor
Source: components/flume/util.py:17
Integration: Binary sensor (documentation, issues)
First occurred: 8:22:35 PM (1 occurrences)
Last logged: 8:22:35 PM

Error while setting up flume platform for binary_sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 281, in _async_setup_platform
    await asyncio.shield(task)
  File "/usr/src/homeassistant/homeassistant/components/flume/binary_sensor.py", line 94, in async_setup_entry
    flume_devices = get_valid_flume_devices(flume_devices)
  File "/usr/src/homeassistant/homeassistant/components/flume/util.py", line 14, in get_valid_flume_devices
    return [
  File "/usr/src/homeassistant/homeassistant/components/flume/util.py", line 17, in <listcomp>
    if KEY_DEVICE_LOCATION_NAME in device[KEY_DEVICE_LOCATION]
KeyError: 'location'

Additional information

No response
@home-assistant
Copy link
home-assistant bot commented Nov 3, 2022

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

Code owner commands

Code owners of flume can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Change the title of the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign flume Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)

flume documentation
flume source
(message by IssueLinks)

@bdraco bdraco added this to the 2022.11.1 milestone Nov 3, 2022
bdraco added a commit to bdraco/home-assistant that referenced this issue Nov 3, 2022
@bdraco
Skip flume devices with location missing
b1df5e0 
fixes home-assistant#81438
@bdraco bdraco mentioned this issue Nov 3, 2022
Merged
19 tasks
MartinHjelmare pushed a commit that referenced this issue Nov 3, 2022
@bdraco
Skip flume devices with location missing (#81441)
adf35e5 
fixes #81438
balloob pushed a commit that referenced this issue Nov 3, 2022
@bdraco @balloob
Skip flume devices with location missing (#81441)
6322319 
fixes #81438
@github-actions github-actions bot locked and limited conversation to collaborators Dec 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bdraco bdraco

Labels
integration: flume
Projects
None yet
Milestone
3C26 2022.11.1
Development

Successfully merging a pull request may close this issue.

Skip flume devices with location missing bdraco/home-assistant
2 participants
@bdraco @nathan-curtis
0