cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

removes unneeded check_args function from ios_command

Browse source 
The module would raise a KeyError trying to find the save_config key
which is not present in the argument_spec.  This was caused by the
check_args() function.  Since the ios shared argument spec isn't used
the check_args function is not needed and has been removed.
This commit is contained in:
Peter Sprygada 2016-07-11 15:12:06 -07:00 committed by Matt Clay
parent 030dcccad4
commit 43ba7f3b3b
1 changed files with 2 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
lib/ansible/modules/network/ios/ios_command.py
View file

@ -132,13 +132,10 @@ warnings:
type: list
sample: ['...', '...']
"""
from ansible.module_utils.basic import get_exception
from ansible.module_utils.netcmd import CommandRunner, FailedConditionsError
from ansible.module_utils.ios import NetworkModule, NetworkError
def check_args(module, warnings):
if module.params['save_config'] is True:
warnings.append('save_config argument will be ignored')
def to_lines(stdout):
for item in stdout:
if isinstance(item, basestring):
@ -148,7 +145,7 @@ def to_lines(stdout):
def main():
spec = dict(
commands=dict(type='list', required=True),
wait_for=dict(type='list'),
wait_for=dict(type='list', aliases=['waitfor']),
retries=dict(default=10, type='int'),
interval=dict(default=1, type='int')
)
@ -161,7 +158,6 @@ def main():
conditionals = module.params['wait_for'] or list()
warnings = list()
check_args(module, warnings)
runner = CommandRunner(module)