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

nxos_static_route issues #37612

Closed
saichint opened this issue Mar 19, 2018 · 2 comments · Fixed by #37614
Closed

nxos_static_route issues #37612

saichint opened this issue Mar 19, 2018 · 2 comments · Fixed by #37614
Assignees
Labels
affects_2.6 This issue/PR affects Ansible v2.6 bug This issue/PR relates to a bug. cisco Cisco technologies module This issue/PR relates to a module. networking Network category nxos Cisco NXOS community support:network This issue/PR relates to code supported by the Ansible Network Team.
Milestone

Comments

@saichint
Copy link
Contributor

ISSUE TYPE
  • Bug Report
COMPONENT NAME

nxos_static_route

ANSIBLE VERSION
ansible 2.6.0 (devel fed20b825f) last updated 2018/02/15 12:51:12 (GMT -400)
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/root/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /root/agents-ci/ansible/lib/ansible
  executable location = /root/agents-ci/ansible/bin/ansible
  python version = 2.7.6 (default, Oct 26 2016, 20:30:19) [GCC 4.8.4]
OS / ENVIRONMENT

Ansible Server : Ubuntu 14.04
Device: N9K running 7.0(3)I2(5)

SUMMARY
STEPS TO REPRODUCE

There are few issues with nxos_static_route.

  • There is no 'default' keyword support for optional parameters
  • For removing a static route, we MUST specify all the optional parameters as well because of certain checks in the code and this is not clean because they are optional and not really needed.
  • If the same static route exists on default and non-default vrfs, there is a potential for set (and idempotent) failures.

Note: I will open a PR shortly to address the above issues

@ansibot
Copy link
Contributor

ansibot commented Mar 19, 2018

Files identified in the description:

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

@ansibot
Copy link
Contributor

ansibot commented Mar 19, 2018

@ansibot ansibot added affects_2.6 This issue/PR affects Ansible v2.6 bug This issue/PR relates to a bug. module This issue/PR relates to a module. needs_triage Needs a first human triage before being processed. networking Network category nxos Cisco NXOS community support:network This issue/PR relates to code supported by the Ansible Network Team. labels Mar 19, 2018
@trishnaguha trishnaguha self-assigned this Mar 19, 2018
@trishnaguha trishnaguha removed the needs_triage Needs a first human triage before being processed. label Mar 19, 2018
@trishnaguha trishnaguha added this to the 2.5.0 milestone Mar 19, 2018
@trishnaguha trishnaguha moved this from To do to Done in zzz NOT USED: Networking Bugs Apr 6, 2018
@dagwieers dagwieers added the cisco Cisco technologies label Feb 23, 2019
@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
affects_2.6 This issue/PR affects Ansible v2.6 bug This issue/PR relates to a bug. cisco Cisco technologies module This issue/PR relates to a module. networking Network category nxos Cisco NXOS community support:network This issue/PR relates to code supported by the Ansible Network Team.
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

4 participants