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 for asa_config #25319 #25680
Fix for asa_config #25319 #25680
Conversation
I've added this to the agenda for the Network IRC meeting |
sendonly=dict(default=False) | ||
), module) | ||
else: | ||
transform = ComplexDict(dict( |
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.
I don't think there's any reason to keep this path, ComplexDict needs a module parameter
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.
No, this was just to get it working and not touch anything for other modules. But I agree that this and other things needs to be changed.
SUMMARY
This would fix #25319, however more things will need to be done to restore the full functionality of the ASA modules.
ISSUE TYPE
COMPONENT NAME
ANSIBLE VERSION
ADDITIONAL INFORMATION
I don't like the way I solved this, and this is mostly pushed to show what could be done. The changes to shell.py have been done so that it shouldn't impact any other module. However there's a good chance that other modules using that code doesn't work now either.