Skip to content

Commit b9f82af

Browse files
committed
Merge branch 'release-v5.9.2'
2 parents 18b711b + 819acce commit b9f82af

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

netfoundry/network_group.py

+5-5
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
"""Use a network group and find its networks."""
22

33
from .network import Networks
4-
from .utility import NET_RESOURCES, RESOURCES, STATUS_CODES, any_in, find_generic_resources, get_generic_resource, http, is_uuidv4, normalize_caseless
4+
from .utility import NET_RESOURCES, RESOURCES, STATUS_CODES, any_in, find_generic_resources, get_generic_resource, http, is_uuidv4, normalize_caseless, caseless_equal
55

66

77
class NetworkGroup:
@@ -19,22 +19,22 @@ def __init__(self, Organization: object, network_group_id: str = None, network_g
1919
if is_uuidv4(group):
2020
network_group_id = group
2121
else:
22-
network_group_name = group
22+
network_group_name = normalize_caseless(group)
2323
if network_group_id:
2424
self.network_group_id = network_group_id
2525
self.network_group_name = [ng['organizationShortName'] for ng in self.network_groups if ng['id'] == network_group_id][0]
2626
# TODO: review the use of org short name ref https://mattermost.tools.netfoundry.io/netfoundry/pl/gegyzuybypb9jxnrw1g1imjywh
2727
elif network_group_name:
2828
self.network_group_name = network_group_name
29-
network_group_matches = [ng['id'] for ng in self.network_groups if ng['organizationShortName'] == network_group_name]
29+
network_group_matches = [ng['id'] for ng in self.network_groups if caseless_equal(ng['organizationShortName'], self.network_group_name)]
3030
if len(network_group_matches) == 1:
31-
self.network_group_id = [ng['id'] for ng in self.network_groups if ng['organizationShortName'] == network_group_name][0]
31+
self.network_group_id = network_group_matches[0]
3232
else:
3333
raise RuntimeError(f"there was not exactly one network group matching the name '{network_group_name}'")
3434
elif len(self.network_groups) > 0:
3535
# first network group is typically the only network group
3636
self.network_group_id = self.network_groups[0]['id']
37-
self.network_group_name = self.network_groups[0]['organizationShortName']
37+
self.network_group_name = normalize_caseless(self.network_groups[0]['organizationShortName'])
3838
# warn if there are other groups
3939
if len(self.network_groups) > 1:
4040
self.logger.warning(f"using first network group {self.network_group_name} and ignoring {len(self.network_groups) - 1} other(s) e.g. {self.network_groups[1]['organizationShortName']}, etc...")

0 commit comments

Comments
 (0)