Error handling for controller objects #1018
Conversation
There was a problem hiding this comment.
There are a couple errors in the markdown linter and also sanity checks are failing, can you try and fix those. I think if we were to accept this we would also want it to be more then just controller related objects, but I will wait to hear feedback from others to make sure they are happy with the current implementation before asking you to continue the work to the rest of the roles. @Tompage1994 @sean-m-sullivan
Sure if I could, but currently we don't have AAP 2.5 so I only worked on objects that I could test, and we still don't use EDA. |
What does this PR do?
This PR is related to #984.
I've prepared the changes for all controller objects. What is different that the default behaviour stay mostly the same, unless variable collect_logs is set to true.
Known issue:
['invocation']['module_args'], I believe it is minor bug in the module itself.How should this be tested?
Manually
Is there a relevant Issue open for this?
resolves #984
Other Relevant info, PRs, etc
N/A