internet/community.general
3
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2026-03-26 21:33:12 +00:00
Code Issues Packages Projects Releases Wiki Activity

nmcli: fix setting_types() to properly handle routing_rules as a list type (#11635)

Browse Source 
* Fix setting_types() to properly handle routing_rules as a list type

* Add changelog fragment for ipv6.routing-rules bugfix

* Update changelogs/fragments/11630-nmcli-ipv6-routing-rules.yml

Co-authored-by: Alexei Znamensky <103110+russoz@users.noreply.github.com>

* Add PR URL to changelog fragment

---------

Co-authored-by: Alexei Znamensky <103110+russoz@users.noreply.github.com>
This commit is contained in:
Ted W.
2026-03-23 14:58:47 -04:00
committed by GitHub
parent 4dad53abac
commit 3c21ac961b
3 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
changelogs/fragments/11630-nmcli-ipv6-routing-rules.yml Normal file
View File

@@ -0,0 +1,2 @@
bugfixes:
- nmcli - add missing ``ipv6.routing-rules`` to ``settings_type()`` list type, preventing ``routing_rules6`` list from being corrupted (https://github.com/ansible-collections/community.general/issues/11630, https://github.com/ansible-collections/community.general/pull/11635).

1
plugins/modules/nmcli.py
View File

@@ -2372,6 +2372,7 @@ class Nmcli:
"ipv6.dns-search",
"ipv6.dns-options",
"ipv6.routes",
"ipv6.routing-rules",
"802-11-wireless-security.group",
"802-11-wireless-security.leap-password-flags",
"802-11-wireless-security.pairwise",

2
tests/unit/plugins/modules/test_nmcli.py
View File

@@ -2260,6 +2260,8 @@ def test_generic_connection_modify_routing_rules6(mocked_generic_connection_crea
args, kwargs = arg_list[0]
assert "ipv6.routing-rules" in args[0]
idx = args[0].index("ipv6.routing-rules")
assert args[0][idx + 1] == "priority 5 from fd00::/24 table 5000,priority 10 from fd01::/24 table 5001"
out, err = capfd.readouterr()
results = json.loads(out)