ansible/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
Code Releases Activity

Revert nxos, ios, iosxr return_timestamps (#56206)

Browse source 
* Revert "nxos_command:run_commands results failure when commands array size >1 (#52670)"
This reverts commit 0df5b92af3.
* Revert "added timestamps to nxos_command module (#50261)"
This reverts commit e150943314.
* Revert "added timestamps to ios_command module (#50323)"
This reverts commit 2a432a093b.
* Revert "added response_timestamps to iosxr_command module (#50095)"
This reverts commit 2a0c356da9.
This commit is contained in:
Trishna Guha 2019-05-08 20:49:29 +05:30 committed by GitHub
parent deae5b1bce
commit 2e8a3efccb
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
13 changed files with 33 additions and 84 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/ansible/module_utils/basic.py
View file

@ -2674,7 +2674,3 @@ class AnsibleModule(object):
def get_module_path(): def get_module_path():
return os.path.dirname(os.path.realpath(__file__)) return os.path.dirname(os.path.realpath(__file__))
def get_timestamp():
return datetime.datetime.now().replace(microsecond=0).isoformat()

4
lib/ansible/module_utils/network/ios/ios.py
View file

@ -129,10 +129,10 @@ def get_config(module, flags=None):
return cfg return cfg
def run_commands(module, commands, check_rc=True, return_timestamps=False): def run_commands(module, commands, check_rc=True):
connection = get_connection(module) connection = get_connection(module)
try: try:
return connection.run_commands(commands=commands, check_rc=check_rc, return_timestamps=return_timestamps) return connection.run_commands(commands=commands, check_rc=check_rc)
except ConnectionError as exc: except ConnectionError as exc:
module.fail_json(msg=to_text(exc)) module.fail_json(msg=to_text(exc))

4
lib/ansible/module_utils/network/iosxr/iosxr.py
View file

@ -477,10 +477,10 @@ def load_config(module, command_filter, commit=False, replace=False,
return diff return diff
def run_commands(module, commands, check_rc=True, return_timestamps=False): def run_commands(module, commands, check_rc=True):
connection = get_connection(module) connection = get_connection(module)
try: try:
return connection.run_commands(commands=commands, check_rc=check_rc, return_timestamps=return_timestamps) return connection.run_commands(commands=commands, check_rc=check_rc)
except ConnectionError as exc: except ConnectionError as exc:
module.fail_json(msg=to_text(exc)) module.fail_json(msg=to_text(exc))

44
lib/ansible/module_utils/network/nxos/nxos.py
View file

@ -33,7 +33,7 @@ import json
import re import re
from ansible.module_utils._text import to_text from ansible.module_utils._text import to_text
from ansible.module_utils.basic import env_fallback, get_timestamp from ansible.module_utils.basic import env_fallback
from ansible.module_utils.network.common.utils import to_list, ComplexList from ansible.module_utils.network.common.utils import to_list, ComplexList
from ansible.module_utils.connection import Connection, ConnectionError from ansible.module_utils.connection import Connection, ConnectionError
from ansible.module_utils.common._collections_compat import Mapping from ansible.module_utils.common._collections_compat import Mapping
@ -154,13 +154,13 @@ class Cli:
self._device_configs[cmd] = cfg self._device_configs[cmd] = cfg
return cfg return cfg
def run_commands(self, commands, check_rc=True, return_timestamps=False): def run_commands(self, commands, check_rc=True):
"""Run list of commands on remote device and return results """Run list of commands on remote device and return results
""" """
connection = self._get_connection() connection = self._get_connection()
try: try:
out, timestamps = connection.run_commands(commands, check_rc) out = connection.run_commands(commands, check_rc)
if check_rc == 'retry_json': if check_rc == 'retry_json':
capabilities = self.get_capabilities() capabilities = self.get_capabilities()
network_api = capabilities.get('network_api') network_api = capabilities.get('network_api')
@ -170,10 +170,7 @@ class Cli:
if ('Invalid command at' in resp or 'Ambiguous command at' in resp) and 'json' in resp: if ('Invalid command at' in resp or 'Ambiguous command at' in resp) and 'json' in resp:
if commands[index]['output'] == 'json': if commands[index]['output'] == 'json':
commands[index]['output'] = 'text' commands[index]['output'] = 'text'
out, timestamps = connection.run_commands(commands, check_rc) out = connection.run_commands(commands, check_rc)
if return_timestamps:
return out, timestamps
else:
return out return out
except ConnectionError as exc: except ConnectionError as exc:
self._module.fail_json(msg=to_text(exc)) self._module.fail_json(msg=to_text(exc))
@ -344,7 +341,6 @@ class LocalNxapi:
headers = {'Content-Type': 'application/json'} headers = {'Content-Type': 'application/json'}
result = list() result = list()
timestamps = list()
timeout = self._module.params['timeout'] timeout = self._module.params['timeout']
use_proxy = self._module.params['provider']['use_proxy'] use_proxy = self._module.params['provider']['use_proxy']
@ -352,7 +348,6 @@ class LocalNxapi:
if self._nxapi_auth: if self._nxapi_auth:
headers['Cookie'] = self._nxapi_auth headers['Cookie'] = self._nxapi_auth
timestamp = get_timestamp()
response, headers = fetch_url( response, headers = fetch_url(
self._module, self._url, data=req, headers=headers, self._module, self._url, data=req, headers=headers,
timeout=timeout, method='POST', use_proxy=use_proxy timeout=timeout, method='POST', use_proxy=use_proxy
@ -380,13 +375,12 @@ class LocalNxapi:
self._error(output=output, **item) self._error(output=output, **item)
elif 'body' in item: elif 'body' in item:
result.append(item['body']) result.append(item['body'])
timestamps.append(timestamp)
# else: # else:
# error in command but since check_status is disabled # error in command but since check_status is disabled
# silently drop it. # silently drop it.
# result.append(item['msg']) # result.append(item['msg'])
return result, timestamps return result
def get_config(self, flags=None): def get_config(self, flags=None):
"""Retrieves the current config from the device or cache """Retrieves the current config from the device or cache
@ -400,18 +394,17 @@ class LocalNxapi:
try: try:
return self._device_configs[cmd] return self._device_configs[cmd]
except KeyError: except KeyError:
out, out_timestamps = self.send_request(cmd) out = self.send_request(cmd)
cfg = str(out[0]).strip() cfg = str(out[0]).strip()
self._device_configs[cmd] = cfg self._device_configs[cmd] = cfg
return cfg return cfg
def run_commands(self, commands, check_rc=True, return_timestamps=False): def run_commands(self, commands, check_rc=True):
"""Run list of commands on remote device and return results """Run list of commands on remote device and return results
""" """
output = None output = None
queue = list() queue = list()
responses = list() responses = list()
timestamps = list()
def _send(commands, output): def _send(commands, output):
return self.send_request(commands, output, check_status=check_rc) return self.send_request(commands, output, check_status=check_rc)
@ -422,22 +415,15 @@ class LocalNxapi:
item['output'] = 'json' item['output'] = 'json'
if all((output == 'json', item['output'] == 'text')) or all((output == 'text', item['output'] == 'json')): if all((output == 'json', item['output'] == 'text')) or all((output == 'text', item['output'] == 'json')):
out, out_timestamps = _send(queue, output) responses.extend(_send(queue, output))
responses.extend(out)
timestamps.extend(out_timestamps)
queue = list() queue = list()
output = item['output'] or 'json' output = item['output'] or 'json'
queue.append(item['command']) queue.append(item['command'])
if queue: if queue:
out, out_timestamps = _send(queue, output) responses.extend(_send(queue, output))
responses.extend(out)
timestamps.extend(out_timestamps)
if return_timestamps:
return responses, timestamps
else:
return responses return responses
def load_config(self, commands, return_error=False, opts=None, replace=None): def load_config(self, commands, return_error=False, opts=None, replace=None):
@ -450,7 +436,7 @@ class LocalNxapi:
commands = 'config replace {0}'.format(replace) commands = 'config replace {0}'.format(replace)
commands = to_list(commands) commands = to_list(commands)
msg, msg_timestamps = self.send_request(commands, output='config', check_status=True, msg = self.send_request(commands, output='config', check_status=True,
return_error=return_error, opts=opts) return_error=return_error, opts=opts)
if return_error: if return_error:
return msg return msg
@ -530,7 +516,7 @@ class HttpApi:
return self._connection_obj return self._connection_obj
def run_commands(self, commands, check_rc=True, return_timestamps=False): def run_commands(self, commands, check_rc=True):
"""Runs list of commands on remote device and returns results """Runs list of commands on remote device and returns results
""" """
try: try:
@ -548,10 +534,6 @@ class HttpApi:
if response[0] == '{': if response[0] == '{':
out[index] = json.loads(response) out[index] = json.loads(response)
if return_timestamps:
# workaround until timestamps are implemented
return out, list()
else:
return out return out
def get_config(self, flags=None): def get_config(self, flags=None):
@ -726,9 +708,9 @@ def get_config(module, flags=None):
return conn.get_config(flags=flags) return conn.get_config(flags=flags)
def run_commands(module, commands, check_rc=True, return_timestamps=False): def run_commands(module, commands, check_rc=True):
conn = get_connection(module) conn = get_connection(module)
return conn.run_commands(to_command(module, commands), check_rc, return_timestamps) return conn.run_commands(to_command(module, commands), check_rc)
def load_config(module, config, return_error=False, opts=None, replace=None): def load_config(module, config, return_error=False, opts=None, replace=None):

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

@ -198,7 +198,7 @@ def main():
match = module.params['match'] match = module.params['match']
while retries > 0: while retries > 0:
responses, timestamps = run_commands(module, commands, return_timestamps=True) responses = run_commands(module, commands)
for item in list(conditionals): for item in list(conditionals):
if item(responses): if item(responses):
@ -221,7 +221,6 @@ def main():
result.update({ result.update({
'stdout': responses, 'stdout': responses,
'stdout_lines': list(to_lines(responses)), 'stdout_lines': list(to_lines(responses)),
'timestamps': timestamps
}) })
module.exit_json(**result) module.exit_json(**result)

3
lib/ansible/modules/network/iosxr/iosxr_command.py
View file

@ -178,7 +178,7 @@ def main():
match = module.params['match'] match = module.params['match']
while retries > 0: while retries > 0:
responses, timestamps = run_commands(module, commands, return_timestamps=True) responses = run_commands(module, commands)
for item in list(conditionals): for item in list(conditionals):
if item(responses): if item(responses):
@ -201,7 +201,6 @@ def main():
result.update({ result.update({
'stdout': responses, 'stdout': responses,
'stdout_lines': list(to_lines(responses)), 'stdout_lines': list(to_lines(responses)),
'timestamps': timestamps
}) })
module.exit_json(**result) module.exit_json(**result)

4
lib/ansible/modules/network/nxos/nxos_command.py
View file

@ -188,7 +188,8 @@ def main():
match = module.params['match'] match = module.params['match']
while retries > 0: while retries > 0:
responses, timestamps = run_commands(module, commands, return_timestamps=True) responses = run_commands(module, commands)
for item in list(conditionals): for item in list(conditionals):
try: try:
if item(responses): if item(responses):
@ -213,7 +214,6 @@ def main():
result.update({ result.update({
'stdout': responses, 'stdout': responses,
'stdout_lines': list(to_lines(responses)), 'stdout_lines': list(to_lines(responses)),
'timestamps': timestamps
}) })
module.exit_json(**result) module.exit_json(**result)

9
lib/ansible/plugins/cliconf/ios.py
View file

@ -38,7 +38,6 @@ from itertools import chain
from ansible.errors import AnsibleConnectionFailure from ansible.errors import AnsibleConnectionFailure
from ansible.module_utils._text import to_text from ansible.module_utils._text import to_text
from ansible.module_utils.basic import get_timestamp
from ansible.module_utils.common._collections_compat import Mapping from ansible.module_utils.common._collections_compat import Mapping
from ansible.module_utils.six import iteritems from ansible.module_utils.six import iteritems
from ansible.module_utils.network.common.config import NetworkConfig, dumps from ansible.module_utils.network.common.config import NetworkConfig, dumps
@ -287,12 +286,11 @@ class Cliconf(CliconfBase):
return resp return resp
def run_commands(self, commands=None, check_rc=True, return_timestamps=False): def run_commands(self, commands=None, check_rc=True):
if commands is None: if commands is None:
raise ValueError("'commands' value is required") raise ValueError("'commands' value is required")
responses = list() responses = list()
timestamps = list()
for cmd in to_list(commands): for cmd in to_list(commands):
if not isinstance(cmd, Mapping): if not isinstance(cmd, Mapping):
cmd = {'command': cmd} cmd = {'command': cmd}
@ -303,18 +301,13 @@ class Cliconf(CliconfBase):
try: try:
out = self.send_command(**cmd) out = self.send_command(**cmd)
timestamp = get_timestamp()
except AnsibleConnectionFailure as e: except AnsibleConnectionFailure as e:
if check_rc: if check_rc:
raise raise
out = getattr(e, 'err', to_text(e)) out = getattr(e, 'err', to_text(e))
responses.append(out) responses.append(out)
timestamps.append(timestamp)
if return_timestamps:
return responses, timestamps
else:
return responses return responses
def get_defaults_flag(self): def get_defaults_flag(self):

9
lib/ansible/plugins/cliconf/iosxr.py
View file

@ -35,7 +35,6 @@ import json
from ansible.errors import AnsibleConnectionFailure from ansible.errors import AnsibleConnectionFailure
from ansible.module_utils._text import to_text from ansible.module_utils._text import to_text
from ansible.module_utils.basic import get_timestamp
from ansible.module_utils.common._collections_compat import Mapping from ansible.module_utils.common._collections_compat import Mapping
from ansible.module_utils.connection import ConnectionError from ansible.module_utils.connection import ConnectionError
from ansible.module_utils.network.common.config import NetworkConfig, dumps from ansible.module_utils.network.common.config import NetworkConfig, dumps
@ -184,11 +183,10 @@ class Cliconf(CliconfBase):
self.send_command(**cmd_obj) self.send_command(**cmd_obj)
def run_commands(self, commands=None, check_rc=True, return_timestamps=False): def run_commands(self, commands=None, check_rc=True):
if commands is None: if commands is None:
raise ValueError("'commands' value is required") raise ValueError("'commands' value is required")
responses = list() responses = list()
timestamps = list()
for cmd in to_list(commands): for cmd in to_list(commands):
if not isinstance(cmd, Mapping): if not isinstance(cmd, Mapping):
cmd = {'command': cmd} cmd = {'command': cmd}
@ -198,7 +196,6 @@ class Cliconf(CliconfBase):
raise ValueError("'output' value %s is not supported for run_commands" % output) raise ValueError("'output' value %s is not supported for run_commands" % output)
try: try:
timestamp = get_timestamp()
out = self.send_command(**cmd) out = self.send_command(**cmd)
except AnsibleConnectionFailure as e: except AnsibleConnectionFailure as e:
if check_rc: if check_rc:
@ -217,10 +214,6 @@ class Cliconf(CliconfBase):
pass pass
responses.append(out) responses.append(out)
timestamps.append(timestamp)
if return_timestamps:
return responses, timestamps
else:
return responses return responses
def discard_changes(self): def discard_changes(self):

6
lib/ansible/plugins/cliconf/nxos.py
View file

@ -34,7 +34,6 @@ import json
import re import re
from ansible.errors import AnsibleConnectionFailure from ansible.errors import AnsibleConnectionFailure
from ansible.module_utils.basic import get_timestamp
from ansible.module_utils._text import to_bytes, to_text from ansible.module_utils._text import to_bytes, to_text
from ansible.module_utils.common._collections_compat import Mapping from ansible.module_utils.common._collections_compat import Mapping
from ansible.module_utils.connection import ConnectionError from ansible.module_utils.connection import ConnectionError
@ -199,7 +198,6 @@ class Cliconf(CliconfBase):
raise ValueError("'commands' value is required") raise ValueError("'commands' value is required")
responses = list() responses = list()
timestamps = list()
for cmd in to_list(commands): for cmd in to_list(commands):
if not isinstance(cmd, Mapping): if not isinstance(cmd, Mapping):
cmd = {'command': cmd} cmd = {'command': cmd}
@ -209,7 +207,6 @@ class Cliconf(CliconfBase):
cmd['command'] = self._get_command_with_output(cmd['command'], output) cmd['command'] = self._get_command_with_output(cmd['command'], output)
try: try:
timestamp = get_timestamp()
out = self.send_command(**cmd) out = self.send_command(**cmd)
except AnsibleConnectionFailure as e: except AnsibleConnectionFailure as e:
if check_rc is True: if check_rc is True:
@ -228,8 +225,7 @@ class Cliconf(CliconfBase):
pass pass
responses.append(out) responses.append(out)
timestamps.append(timestamp) return responses
return responses, timestamps
def get_device_operations(self): def get_device_operations(self):
return { return {

5
test/units/modules/network/ios/test_ios_command.py
View file

@ -22,7 +22,6 @@ __metaclass__ = type
import json import json
from units.compat.mock import patch from units.compat.mock import patch
from ansible.module_utils.basic import get_timestamp
from ansible.modules.network.ios import ios_command from ansible.modules.network.ios import ios_command
from units.modules.utils import set_module_args from units.modules.utils import set_module_args
from .ios_module import TestIosModule, load_fixture from .ios_module import TestIosModule, load_fixture
@ -47,7 +46,6 @@ class TestIosCommandModule(TestIosModule):
def load_from_file(*args, **kwargs): def load_from_file(*args, **kwargs):
module, commands = args module, commands = args
output = list() output = list()
timestamps = list()
for item in commands: for item in commands:
try: try:
@ -57,8 +55,7 @@ class TestIosCommandModule(TestIosModule):
command = item['command'] command = item['command']
filename = str(command).replace(' ', '_') filename = str(command).replace(' ', '_')
output.append(load_fixture(filename)) output.append(load_fixture(filename))
timestamps.append(get_timestamp()) return output
return output, timestamps
self.run_commands.side_effect = load_from_file self.run_commands.side_effect = load_from_file

5
test/units/modules/network/iosxr/test_iosxr_command.py
View file

@ -20,7 +20,6 @@ from __future__ import (absolute_import, division, print_function)
__metaclass__ = type __metaclass__ = type
from units.compat.mock import patch from units.compat.mock import patch
from ansible.module_utils.basic import get_timestamp
from ansible.modules.network.iosxr import iosxr_command from ansible.modules.network.iosxr import iosxr_command
from units.modules.utils import set_module_args from units.modules.utils import set_module_args
from .iosxr_module import TestIosxrModule, load_fixture from .iosxr_module import TestIosxrModule, load_fixture
@ -46,7 +45,6 @@ class TestIosxrCommandModule(TestIosxrModule):
def load_from_file(*args, **kwargs): def load_from_file(*args, **kwargs):
module, commands = args module, commands = args
output = list() output = list()
timestamps = list()
for item in commands: for item in commands:
try: try:
@ -55,8 +53,7 @@ class TestIosxrCommandModule(TestIosxrModule):
command = item command = item
filename = str(command).replace(' ', '_') filename = str(command).replace(' ', '_')
output.append(load_fixture(filename)) output.append(load_fixture(filename))
timestamps.append(get_timestamp()) return output
return output, timestamps
self.run_commands.side_effect = load_from_file self.run_commands.side_effect = load_from_file

5
test/units/modules/network/nxos/test_nxos_command.py
View file

@ -22,7 +22,6 @@ __metaclass__ = type
import json import json
from units.compat.mock import patch from units.compat.mock import patch
from ansible.module_utils.basic import get_timestamp
from ansible.modules.network.nxos import nxos_command from ansible.modules.network.nxos import nxos_command
from .nxos_module import TestNxosModule, load_fixture, set_module_args from .nxos_module import TestNxosModule, load_fixture, set_module_args
@ -45,7 +44,6 @@ class TestNxosCommandModule(TestNxosModule):
def load_from_file(*args, **kwargs): def load_from_file(*args, **kwargs):
module, commands = args module, commands = args
output = list() output = list()
timestamps = list()
for item in commands: for item in commands:
try: try:
@ -55,8 +53,7 @@ class TestNxosCommandModule(TestNxosModule):
command = item['command'] command = item['command']
filename = '%s.txt' % str(command).replace(' ', '_') filename = '%s.txt' % str(command).replace(' ', '_')
output.append(load_fixture('nxos_command', filename)) output.append(load_fixture('nxos_command', filename))
timestamps.append(get_timestamp()) return output
return output, timestamps
self.run_commands.side_effect = load_from_file self.run_commands.side_effect = load_from_file