From fb6a8c0133267ee7d4ad825d5ff9f0e11a999381 Mon Sep 17 00:00:00 2001 From: Peter Sprygada Date: Mon, 26 Feb 2018 10:56:22 -0500 Subject: [PATCH] fix cliconf get_config method to match base signature (#36682) This commit fixes up the get_config method to match the minimum method signature as defined by the base class. Without this patch, the get_config method calls will fail in some cirumstances. --- lib/ansible/plugins/cliconf/aireos.py | 2 +- lib/ansible/plugins/cliconf/aruba.py | 2 +- lib/ansible/plugins/cliconf/asa.py | 2 +- lib/ansible/plugins/cliconf/dellos10.py | 2 +- lib/ansible/plugins/cliconf/dellos6.py | 2 +- lib/ansible/plugins/cliconf/dellos9.py | 2 +- lib/ansible/plugins/cliconf/edgeos.py | 2 +- lib/ansible/plugins/cliconf/enos.py | 2 +- lib/ansible/plugins/cliconf/ios.py | 2 +- lib/ansible/plugins/cliconf/iosxr.py | 2 +- lib/ansible/plugins/cliconf/ironware.py | 2 +- lib/ansible/plugins/cliconf/nxos.py | 5 +++-- lib/ansible/plugins/cliconf/onyx.py | 2 +- lib/ansible/plugins/cliconf/vyos.py | 2 +- 14 files changed, 16 insertions(+), 15 deletions(-) diff --git a/lib/ansible/plugins/cliconf/aireos.py b/lib/ansible/plugins/cliconf/aireos.py index 46df3b33bb..1496977832 100644 --- a/lib/ansible/plugins/cliconf/aireos.py +++ b/lib/ansible/plugins/cliconf/aireos.py @@ -55,7 +55,7 @@ class Cliconf(CliconfBase): return device_info @enable_mode - def get_config(self, source='running'): + def get_config(self, source='running', format='text'): if source not in ('running', 'startup'): return self.invalid_params("fetching configuration from %s is not supported" % source) if source == 'running': diff --git a/lib/ansible/plugins/cliconf/aruba.py b/lib/ansible/plugins/cliconf/aruba.py index 89fe8f4901..580e20580c 100644 --- a/lib/ansible/plugins/cliconf/aruba.py +++ b/lib/ansible/plugins/cliconf/aruba.py @@ -56,7 +56,7 @@ class Cliconf(CliconfBase): return device_info @enable_mode - def get_config(self, source='running'): + def get_config(self, source='running', format='text'): if source not in ('running', 'startup'): return self.invalid_params("fetching configuration from %s is not supported" % source) if source == 'running': diff --git a/lib/ansible/plugins/cliconf/asa.py b/lib/ansible/plugins/cliconf/asa.py index e8de50b931..575c974544 100644 --- a/lib/ansible/plugins/cliconf/asa.py +++ b/lib/ansible/plugins/cliconf/asa.py @@ -53,7 +53,7 @@ class Cliconf(CliconfBase): return device_info @enable_mode - def get_config(self, source='running'): + def get_config(self, source='running', format='text'): if source not in ('running', 'startup'): return self.invalid_params("fetching configuration from %s is not supported" % source) if source == 'running': diff --git a/lib/ansible/plugins/cliconf/dellos10.py b/lib/ansible/plugins/cliconf/dellos10.py index 98291b52f8..f5ea7ec487 100644 --- a/lib/ansible/plugins/cliconf/dellos10.py +++ b/lib/ansible/plugins/cliconf/dellos10.py @@ -57,7 +57,7 @@ class Cliconf(CliconfBase): return device_info @enable_mode - def get_config(self, source='running'): + def get_config(self, source='running', format='text'): if source not in ('running', 'startup'): return self.invalid_params("fetching configuration from %s is not supported" % source) if source == 'running': diff --git a/lib/ansible/plugins/cliconf/dellos6.py b/lib/ansible/plugins/cliconf/dellos6.py index df9d64fea1..6952e02dba 100644 --- a/lib/ansible/plugins/cliconf/dellos6.py +++ b/lib/ansible/plugins/cliconf/dellos6.py @@ -57,7 +57,7 @@ class Cliconf(CliconfBase): return device_info @enable_mode - def get_config(self, source='running'): + def get_config(self, source='running', format='text'): if source not in ('running', 'startup'): return self.invalid_params("fetching configuration from %s is not supported" % source) # if source == 'running': diff --git a/lib/ansible/plugins/cliconf/dellos9.py b/lib/ansible/plugins/cliconf/dellos9.py index e445da3f14..96815ca84e 100644 --- a/lib/ansible/plugins/cliconf/dellos9.py +++ b/lib/ansible/plugins/cliconf/dellos9.py @@ -57,7 +57,7 @@ class Cliconf(CliconfBase): return device_info @enable_mode - def get_config(self, source='running'): + def get_config(self, source='running', format='text'): if source not in ('running', 'startup'): return self.invalid_params("fetching configuration from %s is not supported" % source) # if source == 'running': diff --git a/lib/ansible/plugins/cliconf/edgeos.py b/lib/ansible/plugins/cliconf/edgeos.py index d0958b01b3..34dbf03d59 100644 --- a/lib/ansible/plugins/cliconf/edgeos.py +++ b/lib/ansible/plugins/cliconf/edgeos.py @@ -38,7 +38,7 @@ class Cliconf(CliconfBase): return device_info - def get_config(self): + def get_config(self, source='running', format='text'): return self.send_command(b'show configuration commands') def edit_config(self, command): diff --git a/lib/ansible/plugins/cliconf/enos.py b/lib/ansible/plugins/cliconf/enos.py index b4e943d38a..882e4c0eab 100644 --- a/lib/ansible/plugins/cliconf/enos.py +++ b/lib/ansible/plugins/cliconf/enos.py @@ -58,7 +58,7 @@ class Cliconf(CliconfBase): return device_info @enable_mode - def get_config(self, source='running'): + def get_config(self, source='running', format='text'): if source not in ('running', 'startup'): msg = "fetching configuration from %s is not supported" return self.invalid_params(msg % source) diff --git a/lib/ansible/plugins/cliconf/ios.py b/lib/ansible/plugins/cliconf/ios.py index 6f3fe20651..f8b3deb9b9 100644 --- a/lib/ansible/plugins/cliconf/ios.py +++ b/lib/ansible/plugins/cliconf/ios.py @@ -53,7 +53,7 @@ class Cliconf(CliconfBase): return device_info @enable_mode - def get_config(self, source='running', flags=None): + def get_config(self, source='running', format='text', flags=None): if source not in ('running', 'startup'): return self.invalid_params("fetching configuration from %s is not supported" % source) if source == 'running': diff --git a/lib/ansible/plugins/cliconf/iosxr.py b/lib/ansible/plugins/cliconf/iosxr.py index 37afb3f21c..9f4746681e 100644 --- a/lib/ansible/plugins/cliconf/iosxr.py +++ b/lib/ansible/plugins/cliconf/iosxr.py @@ -56,7 +56,7 @@ class Cliconf(CliconfBase): return device_info - def get_config(self, source='running', filter=None): + def get_config(self, source='running', format='text', filter=None): lookup = {'running': 'running-config'} if source not in lookup: return self.invalid_params("fetching configuration from %s is not supported" % source) diff --git a/lib/ansible/plugins/cliconf/ironware.py b/lib/ansible/plugins/cliconf/ironware.py index 66c52e2572..10dfa6b71e 100644 --- a/lib/ansible/plugins/cliconf/ironware.py +++ b/lib/ansible/plugins/cliconf/ironware.py @@ -49,7 +49,7 @@ class Cliconf(CliconfBase): return device_info @enable_mode - def get_config(self, source='running', flags=None): + def get_config(self, source='running', format='text', flags=None): if source not in ('running', 'startup'): return self.invalid_params("fetching configuration from %s is not supported" % source) diff --git a/lib/ansible/plugins/cliconf/nxos.py b/lib/ansible/plugins/cliconf/nxos.py index 27de3d60bf..1b056a04a4 100644 --- a/lib/ansible/plugins/cliconf/nxos.py +++ b/lib/ansible/plugins/cliconf/nxos.py @@ -50,11 +50,12 @@ class Cliconf(CliconfBase): return device_info - def get_config(self, source='running', flags=None): + def get_config(self, source='running', format='text', flags=None): lookup = {'running': 'running-config', 'startup': 'startup-config'} cmd = 'show {} '.format(lookup[source]) - cmd += ' '.join(flags) + if flags: + cmd += ' '.join(flags) cmd = cmd.strip() return self.send_command(cmd) diff --git a/lib/ansible/plugins/cliconf/onyx.py b/lib/ansible/plugins/cliconf/onyx.py index b487254db4..6754a7a6bf 100644 --- a/lib/ansible/plugins/cliconf/onyx.py +++ b/lib/ansible/plugins/cliconf/onyx.py @@ -48,7 +48,7 @@ class Cliconf(CliconfBase): return device_info @enable_mode - def get_config(self, source='running'): + def get_config(self, source='running', format='text'): if source not in ('running',): return self.invalid_params("fetching configuration from %s is not supported" % source) cmd = b'show running-config' diff --git a/lib/ansible/plugins/cliconf/vyos.py b/lib/ansible/plugins/cliconf/vyos.py index d341a2bd35..f4ad687dc8 100644 --- a/lib/ansible/plugins/cliconf/vyos.py +++ b/lib/ansible/plugins/cliconf/vyos.py @@ -51,7 +51,7 @@ class Cliconf(CliconfBase): return device_info - def get_config(self): + def get_config(self, source='running', format='text'): return self.send_command(b'show configuration commands') def edit_config(self, command):