ansible-collections/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2025-04-05 10:10:31 -07:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

nmcli: fix reordering of DNS nameservers and search suffixes (#9880)

Browse source 
* nmcli: fix reordering of DNS nameservers and search suffixes

- Fixes #8724

* Update changelog fragment index in line with PR number

- Now I understand what that number is for

* Use correct changelog format for nmcli PR #9880
This commit is contained in:
ashleyghooper 2025-03-22 09:04:30 +13:00 • committed by GitHub
parent 6a2d9ccd99
commit 3922b82a57
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
changelogs/fragments/9880-nmcli-fix-reorder-same-dns-nameservers-search-suffixes.yml Normal file
View file

@ -0,0 +1,2 @@
bugfixes:
- "nmcli - enable changing only the order of DNS servers or search suffixes (https://github.com/ansible-collections/community.general/issues/8724, https://github.com/ansible-collections/community.general/pull/9880)."

4
plugins/modules/nmcli.py
View file

@ -2514,9 +2514,11 @@ class Nmcli(object):
if isinstance(current_value, list) and isinstance(value, list): if isinstance(current_value, list) and isinstance(value, list):
# compare values between two lists # compare values between two lists
if key in ('ipv4.addresses', 'ipv6.addresses'): if key in ('ipv4.addresses', 'ipv6.addresses', 'ipv4.dns', 'ipv6.dns', 'ipv4.dns-search', 'ipv6.dns-search'):
# The order of IP addresses matters because the first one # The order of IP addresses matters because the first one
# is the default source address for outbound connections. # is the default source address for outbound connections.
# Similarly, the order of DNS nameservers and search
# suffixes is important.
changed |= current_value != value changed |= current_value != value
else: else:
changed |= sorted(current_value) != sorted(value) changed |= sorted(current_value) != sorted(value)