From 5174e3f31fdaeceb438526a2f12d6bbcdd1d6574 Mon Sep 17 00:00:00 2001 From: GGabriele Date: Mon, 19 Sep 2016 14:27:00 +0200 Subject: [PATCH 1/2] Fixing nxos_portchannel --- network/nxos/nxos_portchannel.py | 34 +++++++++++++++++++++----------- 1 file changed, 23 insertions(+), 11 deletions(-) diff --git a/network/nxos/nxos_portchannel.py b/network/nxos/nxos_portchannel.py index ab07385effc..b6654c3db64 100644 --- a/network/nxos/nxos_portchannel.py +++ b/network/nxos/nxos_portchannel.py @@ -18,7 +18,6 @@ DOCUMENTATION = ''' --- - module: nxos_portchannel version_added: "2.2" short_description: Manages port-channel interfaces. @@ -107,14 +106,14 @@ end_state: "Ethernet2/5": {"mode": "on", "status": "D"}, "Ethernet2/6": {"mode": "on", "status": "D"}}, "min_links": null, "mode": "on"} -commands: - description: command string sent to the device +updates: + description: command sent to the device returned: always - type: string - sample: "interface Ethernet2/6 ; no channel-group 12 ; - interface Ethernet2/5 ; no channel-group 12 ; - interface Ethernet2/6 ; channel-group 12 mode on ; - interface Ethernet2/5 ; channel-group 12 mode on ;" + type: list + sample: ["interface Ethernet2/6", "no channel-group 12", + "interface Ethernet2/5", "no channel-group 12", + "interface Ethernet2/6", "channel-group 12 mode on", + "interface Ethernet2/5", "channel-group 12 mode on" changed: description: check to see if a change was made on the device returned: always @@ -330,12 +329,24 @@ def execute_config_command(commands, module): clie = get_exception() module.fail_json(msg='Error sending CLI commands', error=str(clie), commands=commands) + except AttributeError: + try: + commands.insert(0, 'configure') + module.cli.add_commands(commands, output='config') + output = module.cli.run_commands() + except ShellError: + clie = get_exception() + module.fail_json(msg='Error sending CLI commands', + error=str(clie), commands=commands) return output def get_cli_body_ssh(command, response, module): try: - body = [json.loads(response[0])] + if isinstance(response[0], str): + body = [json.loads(response[0])] + else: + body = response except ValueError: module.fail_json(msg='Command does not support JSON output', command=command) @@ -714,12 +725,13 @@ def main(): output = execute_config_command(cmds, module) changed = True end_state, interface_exist = get_existing(module, args) + if 'configure' in cmds: + cmds.pop(0) results = {} results['proposed'] = proposed results['existing'] = existing results['end_state'] = end_state - results['state'] = state results['updates'] = cmds results['changed'] = changed @@ -730,4 +742,4 @@ def main(): if __name__ == '__main__': - main() + main() \ No newline at end of file From d632cce54603356e7d7fb978d7e3a820edc1d430 Mon Sep 17 00:00:00 2001 From: GGabriele Date: Mon, 19 Sep 2016 14:35:51 +0200 Subject: [PATCH 2/2] Fixed docstring --- network/nxos/nxos_portchannel.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/network/nxos/nxos_portchannel.py b/network/nxos/nxos_portchannel.py index b6654c3db64..68af16c8f1d 100644 --- a/network/nxos/nxos_portchannel.py +++ b/network/nxos/nxos_portchannel.py @@ -113,7 +113,7 @@ updates: sample: ["interface Ethernet2/6", "no channel-group 12", "interface Ethernet2/5", "no channel-group 12", "interface Ethernet2/6", "channel-group 12 mode on", - "interface Ethernet2/5", "channel-group 12 mode on" + "interface Ethernet2/5", "channel-group 12 mode on"] changed: description: check to see if a change was made on the device returned: always