Fix in eos get_config cliconf api (#38682)

If format is passed as None to get_config api, wrong command is
genereted ie. `show running-configuration | None | section interface`.
Add format type in command only if format value is either not `text`
or  `None`.
pull/38721/head
Ganesh Nalawade 7 years ago committed by GitHub
parent 6de49f084c
commit 88662d0c56
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -51,13 +51,12 @@ class Cliconf(CliconfBase):
lookup = {'running': 'running-config', 'startup': 'startup-config'} lookup = {'running': 'running-config', 'startup': 'startup-config'}
if source not in lookup: if source not in lookup:
return self.invalid_params("fetching configuration from %s is not supported" % source) return self.invalid_params("fetching configuration from %s is not supported" % source)
if format == 'text':
cmd = b'show %s ' % lookup[source]
else:
cmd = b'show %s | %s' % (lookup[source], format)
flags = [] if flags is None else flags cmd = b'show %s ' % lookup[source]
cmd += ' '.join(flags) if format and format is not 'text':
cmd += b'| %s ' % format
cmd += ' '.join(to_list(flags))
cmd = cmd.strip() cmd = cmd.strip()
return self.send_command(cmd) return self.send_command(cmd)

Loading…
Cancel
Save