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 n3k separate to combined image upgrade #37017
Fix n3k separate to combined image upgrade #37017
Conversation
@trishnaguha Hopefully it's not too late, but we need this fix in the |
@@ -245,6 +245,8 @@ def parse_show_install(data): | |||
ud['server_error'] = True | |||
elif data == -32603: | |||
ud['server_error'] = True | |||
elif data == 'No install all data found': |
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 there is a check on L241 if isinstance(data, int)
.
From elif data == 'No install all data found:'
it looks like data is string. Will the code hit this condition?
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.
Good catch. Your right that it will never hit this condition. Let me move the check.
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.
Turns out this check is not needed. I removed it.
* Fix n3k separate to combined image upgrade * Address review comments (cherry picked from commit 51e8eab)
cherry-picked to |
SUMMARY
This fixes a problem when attempting to upgrade from a 6.x kickstart/system image to a combined 7.x image.
ISSUE TYPE
COMPONENT NAME
nxos_install_os
ANSIBLE VERSION
ADDITIONAL INFORMATION
NOTE This fix is needed in
2.5