-
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
Fixes #31056 - nxos_portchannel idempotence failure on N1 images #31057
Conversation
@@ -203,6 +203,8 @@ def get_portchannel(module, netcfg=None): | |||
for pc in pc_table: | |||
if pc['group'] == module.params['group']: | |||
portchannel_table = pc | |||
elif module.params['group'].isdigit() and pc['group'] == int(module.params['group']): |
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.
Did you test this change against all nxos platforms? Just want to make sure since this area has been historically prone to idempotence issues that were fixed in 2.4
.
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.
@mikewiebe ... I've tested our matrix and it passed the tests. I will rerun and confirm.
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.
@rahushen Did you retest this?
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.
@mikewiebe @trishnaguha Yes, retested and it passed the tests.
@trishnaguha ... this also needs to be cherry-picked to 2.4.1 |
(cherry picked from commit 046d430)
This will be out with 2.4.2 release as 2.4.1 rc1 is already out. |
SUMMARY
Output of show port-channel summary | json returns group as int as compared to a string on other platforms. This causes idempotence issues on N1 images. The fix is to check for both string and int equality.
Fixes #31056
ISSUE TYPE
COMPONENT NAME
ANSIBLE VERSION