Closed pemontto closed 1 week ago
Hey there @gjohansson-st, mind taking a look at this pull request as it has been labeled with an integration (dnsip
) you are listed as a code owner for? Thanks!
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks :+1:
@pemontto could we add a documentation PR to this? I think it would be nice to have this behavior and attributes described in the documentation. Thanks 👍
I think this PR looks go and ready to merge 👍
@gjohansson-ST I've added some wording to the documentation on the 2nd dot point where a hostname is specified.
---------- coverage: platform linux, python 3.12.3-final-0 -----------
Name Stmts Miss Cover Missing
-----------------------------------------------------------------------------
homeassistant/components/dnsip/__init__.py 12 0 100%
homeassistant/components/dnsip/config_flow.py 65 0 100%
homeassistant/components/dnsip/const.py 14 0 100%
homeassistant/components/dnsip/sensor.py 61 0 100%
-----------------------------------------------------------------------------
TOTAL 152 0 100%
Breaking change
This PR tries to avoid breaking changes by keeping the default behaviour and only exposing the round-robin IPs in the attributes.
Proposed change
Similar to #91927
This PR supports hostnames that return multiple round-robin A records and sorts the IPs (both v4 and v6) naturally.
IPs
will be added with a list of IPs in ascending orderType of change
Additional information
Checklist
ruff format 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
.To help with the load of incoming pull requests: