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
use show run instead of section pipeline ios_l2_interface #39658
Conversation
@@ -146,7 +146,7 @@ def is_switchport(name, module): | |||
|
|||
def interface_is_portchannel(name, module): | |||
if get_interface_type(name) == 'ethernet': | |||
config = get_config(module, flags=[' | section interface']) | |||
config = run_commands(module, ['show run interface {0}'.format(name)])[0] | |||
if 'channel group' in config: |
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.
It looks like this should be added possibly or something similar?
if 'channel group' in config:
return True
elif 'channel-group' in config:
return True
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.
@mrlesmithjr Thanks for bringing up the other scenario. The latest commit should fix that.
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.
Works as it should! Great job!
Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
e2dc2da
to
9ba36b3
Compare
cherry-picked to 2.5 |
) Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> (cherry picked from commit dddcbb7)
* nxos_linkagg normalize interface (#39591) Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> (cherry picked from commit 3c35dd4) * use show run instead of section pipeline ios_l2_interface (#39658) Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> (cherry picked from commit dddcbb7) * add changelog Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * fix nxos_snmp_user issues (#39760) * fix nxos_snmp_user issues * shipppable fix (cherry picked from commit e3bfbe5) * changelog Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> * fix shippable Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
Signed-off-by: Trishna Guha trishnaguha17@gmail.com
SUMMARY
fixes #39011
| section
isn't supported in older IOS devices, useshow run interface
instead.ISSUE TYPE
COMPONENT NAME
modules/network/ios/ios_l2_interface.py
ANSIBLE VERSION