diff options
author | Kevin Breit <kevin.breit@kevinbreit.net> | 2019-04-24 11:16:51 -0500 |
---|---|---|
committer | Toshio Kuratomi <a.badger@gmail.com> | 2019-04-25 09:19:03 -0600 |
commit | e0a3c6214d0bbcd418c16e654c7e7d4cd2edfe2e (patch) | |
tree | 5cef5540ae1c77d1973722d5d21bbdf7755a1d93 | |
parent | 74e90f89bd69bed528db52c0daf1e2cff85aa1a4 (diff) | |
download | ansible-e0a3c6214d0bbcd418c16e654c7e7d4cd2edfe2e.tar.gz |
meraki_static_route - Remove unnecessary API call (#55528)
* Make module not get all nets every time it's executed with net_id
* Add changelog fragment
* Update changelogs/fragments/meraki_static_route_api_calls.yml
Co-Authored-By: kbreit <kevin.breit@kevinbreit.net>
(cherry picked from commit 7b7d6a1fef42eed7e144a7c391817a0a8263c3ae)
-rw-r--r-- | changelogs/fragments/meraki_static_route_api_calls.yml | 3 | ||||
-rw-r--r-- | lib/ansible/modules/network/meraki/meraki_static_route.py | 2 |
2 files changed, 4 insertions, 1 deletions
diff --git a/changelogs/fragments/meraki_static_route_api_calls.yml b/changelogs/fragments/meraki_static_route_api_calls.yml new file mode 100644 index 0000000000..b8f2996480 --- /dev/null +++ b/changelogs/fragments/meraki_static_route_api_calls.yml @@ -0,0 +1,3 @@ +--- +bugfixes: +- "meraki_static_route - Module would make unnecessary API calls to Meraki when ``net_id`` is specified in task." diff --git a/lib/ansible/modules/network/meraki/meraki_static_route.py b/lib/ansible/modules/network/meraki/meraki_static_route.py index 2a9c5af221..304eaa16d5 100644 --- a/lib/ansible/modules/network/meraki/meraki_static_route.py +++ b/lib/ansible/modules/network/meraki/meraki_static_route.py @@ -328,9 +328,9 @@ def main(): org_id = meraki.params['org_id'] if not org_id: org_id = meraki.get_org_id(meraki.params['org_name']) - nets = meraki.get_nets(org_id=org_id) net_id = meraki.params['net_id'] if net_id is None: + nets = meraki.get_nets(org_id=org_id) net_id = meraki.get_net_id(net_name=meraki.params['net_name'], data=nets) if meraki.params['state'] == 'query': |