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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix missing azure event hub instance name #52049
Conversation
Please add a link to a changelog or GitHub commit compare view for the version bump in the PR description. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
We should update the docs, right? https://www.home-assistant.io/integrations/azure_event_hub/#configuration-variables Please link a docs PR in this PR description. |
Yeah was already planning to do so! |
Done! |
Breaking change
Fix for using this component with IoTHub, event_hub_name is now a required field, can be filled by the DeviceID when using IoTHub.
Proposed change
Updated the package to azure-eventhub=5.5 (https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/eventhub/azure-eventhub/CHANGELOG.md).
Made a change in the config to fix the issue, now needs to include event_hub_name which has to be filled by the DeviceID when using IoTHub.
Added typing.
Type of change
Additional information
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: