Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix in eos get_config cliconf api #38682

Merged
merged 1 commit into from Apr 13, 2018

Conversation

ganeshrn
Copy link
Member

SUMMARY

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 not None.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

plugins/cliconf/eos.py

ANSIBLE VERSION
2.5
ADDITIONAL INFORMATION

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`.
@ansibot
Copy link
Contributor

ansibot commented Apr 12, 2018

@ansibot ansibot added bug This issue/PR relates to a bug. needs_triage Needs a first human triage before being processed. networking Network category support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Apr 12, 2018
@mkrizek mkrizek removed the needs_triage Needs a first human triage before being processed. label Apr 13, 2018
@ganeshrn ganeshrn merged commit 88662d0 into ansible:devel Apr 13, 2018
@ganeshrn ganeshrn deleted the eos_cliconf_get_config_fix branch April 13, 2018 07:50
ganeshrn added a commit to ganeshrn/ansible that referenced this pull request Apr 13, 2018
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`.
(cherry picked from commit 88662d0)

Update changelog
ganeshrn added a commit that referenced this pull request Apr 13, 2018
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`.
(cherry picked from commit 88662d0)

Update changelog
@ganeshrn ganeshrn moved this from Need CP into 2.5.1 to Done in zzz NOT USED: Networking Bugs Apr 16, 2018
ilicmilan pushed a commit to ilicmilan/ansible that referenced this pull request Nov 7, 2018
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`.
@ansible ansible locked and limited conversation to collaborators Apr 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug This issue/PR relates to a bug. networking Network category support:core This issue/PR relates to code supported by the Ansible Engineering Team.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants