diff --git a/changelogs/fragments/meraki_orgnet_fix.yml b/changelogs/fragments/meraki_orgnet_fix.yml new file mode 100644 index 00000000000..d9ff1581024 --- /dev/null +++ b/changelogs/fragments/meraki_orgnet_fix.yml @@ -0,0 +1,2 @@ +bugfixes: + - Meraki - Lookups using org_name or net_name no longer query Meraki twice, only once. Major performance improvements. diff --git a/lib/ansible/module_utils/network/meraki/meraki.py b/lib/ansible/module_utils/network/meraki/meraki.py index 4519f87a3f9..3684b008ed4 100644 --- a/lib/ansible/module_utils/network/meraki/meraki.py +++ b/lib/ansible/module_utils/network/meraki/meraki.py @@ -157,7 +157,7 @@ class MerakiModule(object): response = self.request('/organizations', method='GET') if self.status != 200: self.fail_json(msg='Organization lookup failed') - self.orgs = self.request('/organizations', method='GET') + self.orgs = response return self.orgs def is_org_valid(self, data, org_name=None, org_id=None): @@ -205,7 +205,7 @@ class MerakiModule(object): r = self.request(path, method='GET') if self.status != 200: self.fail_json(msg='Network lookup failed') - self.nets = self.request(path, method='GET') + self.nets = r templates = self.get_config_templates(org_id) for t in templates: self.nets.append(t)