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 eos_l2_interface invalid command #50644
Conversation
Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
@@ -176,9 +176,9 @@ def map_obj_to_commands(updates, module): | |||
commands.append('switchport trunk allowed vlan {0}'.format(trunk_allowed_vlans)) | |||
else: | |||
if obj_in_have['native_vlan']: | |||
commands.append('not switchport trunk native vlan {0}'.format(obj_in_have['native_vlan'])) | |||
commands.append('no switchport trunk native vlan {0}'.format(obj_in_have['native_vlan'])) |
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.
For EOS, following line is also required after this line:
commands.append( 'no switchport mode trunk')
if obj_in_have['trunk_allowed_vlans']: | ||
commands.append('not switchport trunk allowed vlan {0}'.format(obj_in_have['trunk_allowed_vlans'])) | ||
commands.append('no switchport trunk allowed vlan {0}'.format(obj_in_have['trunk_allowed_vlans'])) |
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.
For EOS, following line is also required after this line:
commands.append( 'no switchport mode trunk')
Line172 should be modified as follows:
|
Signed-off-by: Trishna Guha <trishnaguha17@gmail.com>
Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> (cherry picked from commit 1400d16)
Signed-off-by: Trishna Guha <trishnaguha17@gmail.com> (cherry picked from commit 1400d16)
Signed-off-by: Trishna Guha trishnaguha17@gmail.com
SUMMARY
fixes #50639
ISSUE TYPE
COMPONENT NAME
modules/network/eos/eos_l2_interface.py