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 nxos_vpc_interface peer_link idempotence #35737
Conversation
commands.append('no vpc peer-link') | ||
commands.insert(0, 'interface port-channel{0}'.format(portchannel)) | ||
|
||
elif delta.get('peer-link') or not existing.get('vpc'): | ||
elif delta.get('peer-link') and existing.get('peer-link') is not 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.
Please modify this to elif delta.get('peer-link') and not existing.get('peer-link'):
@@ -194,11 +194,14 @@ def get_commands_to_config_vpc_interface(portchannel, delta, config_value, exist | |||
commands = [] | |||
|
|||
if delta.get('peer-link') is False and existing.get('peer-link') is 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.
Modify this one as well? if not delta.get('peer-link') and existing.get('peer-link'):
@saichint please push the requested change and we are good to merge after that. |
Made changes and pushed |
SUMMARY
Fixes #35736
ISSUE TYPE
COMPONENT NAME
nxos_vpc_interface
ANSIBLE VERSION
ADDITIONAL INFORMATION