-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
sub-interface support for nxos module #37392
Conversation
Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
726e8b7
to
8269949
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd make the substitution for simplicity, but looks reasonable otherwise
@@ -316,7 +316,7 @@ def normalize_interface(name): | |||
def _get_number(name): | |||
digits = '' | |||
for char in name: | |||
if char.isdigit() or char == '/': |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
or char in '/.'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fair enough!
Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
8269949
to
68d80d7
Compare
* nxos sub-interface support Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * update l3_interface test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * Add integration test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
* nxos sub-interface support Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * update l3_interface test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * Add integration test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
* nxos sub-interface support Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * update l3_interface test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * Add integration test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
* nxos sub-interface support Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * update l3_interface test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * Add integration test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
* nxos sub-interface support Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * update l3_interface test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * Add integration test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
* nxos sub-interface support Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * update l3_interface test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * Add integration test Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
SUMMARY
fixes #29164
Ethernet1/6.20
no switchport
by default in nxos_l3_interface. User should do make routed interface (mode: layer3) on his/her own before usingnxos_l3_interface
.ISSUE TYPE
COMPONENT NAME
modules/network/nxos/{nxos_interface,nxos_l3_interface}.py
test/integration/targets/nxos_l3_interface/tests/{cli,nxapi}/sanity.yaml
test/integration/targets/nxos_interface/tests/common/sub_int.yaml
ANSIBLE VERSION