Skip to content

Add to_text for vrf names #772

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Add to_text for vrf names #772

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
e507cd0
Add to_text for vrf names
steve-wormley Oct 19, 2023
3109d53
One line fix to the 2 line commit
steve-wormley Oct 19, 2023
91735d6
Merge branch 'main' into bgp_global_vrf_name_fix
NilashishC Oct 25, 2023
b23aa9d
Merge branch 'main' into bgp_global_vrf_name_fix
KB-perByte Oct 25, 2023
7676341
Merge branch 'main' into bgp_global_vrf_name_fix
NilashishC Nov 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion plugins/module_utils/network/nxos/facts/bgp_global/bgp_global.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
based on the configuration.
"""

from ansible.module_utils._text import to_text
from ansible.module_utils.six import iteritems
from ansible_collections.ansible.netcommon.plugins.module_utils.network.common import utils

Expand Down Expand Up @@ -71,7 +72,7 @@ def populate_facts(self, connection, ansible_facts, data=None):

# transform vrfs into a list
if vrfs:
obj["vrfs"] = sorted(list(obj["vrfs"].values()), key=lambda k, sk="vrf": k[sk])
obj["vrfs"] = sorted(list(obj["vrfs"].values()), key=lambda k, sk="vrf": to_text(k[sk]))
for vrf in obj["vrfs"]:
self._post_parse(vrf)

Expand Down