Fix authorize in net agnostic modules (#33549)

* This should fix authorize for net_* modules

* transport: none means network_cli in eos

For now anyway
pull/33220/head
Nathaniel Case 7 years ago committed by GitHub
parent 961c3fcca7
commit fdc547601c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -130,9 +130,7 @@ def map_config_to_obj(module):
output = run_commands(module, ['show banner %s' % module.params['banner']])
obj = {'banner': module.params['banner'], 'state': 'absent'}
if output:
if module.params['transport'] == 'cli':
obj['text'] = output[0]
else:
if module.params['transport'] == 'eapi':
# On EAPI we need to extract the banner text from dict key
# 'loginBanner'
if module.params['banner'] == 'login':
@ -141,6 +139,8 @@ def map_config_to_obj(module):
banner_response_key = 'motd'
if isinstance(output[0], dict) and banner_response_key in output[0].keys():
obj['text'] = output[0]
else:
obj['text'] = output[0]
obj['state'] = 'present'
return obj

@ -70,6 +70,7 @@ class ActionModule(ActionBase):
if 'authorize' in self.provider.keys():
play_context.become = self.provider['authorize'] or False
play_context.become_pass = self.provider['auth_pass']
play_context.become_method = 'enable'
if self._play_context.connection == 'local':
socket_path = self._start_connection(play_context)

Loading…
Cancel
Save