Make uninstall more robust and informative

[carlisia]

Signed-off-by: Carlisia carlisia@vmware.com

Thank you for contributing to Velero!

Please add a summary of your change

• used the same existing API clients as the original code
• aggregated the errors until all operations are done
• added more informative messages
• added confirmation
• added --wait (for the namespace termination) and --force (to be script friendly and skip confirmation)
• the output from --wait will look the same to the user as the --wait for creating backups

PS: 2e2 tests pass on Kind, testing it against AWS..

Does your change fix a particular issue?

Fixes #3599

Please indicate you've done the following:

• Accepted the DCO. Commits without the DCO will delay acceptance.
• Created a changelog file or added /kind changelog-not-required.
• Updated the corresponding documentation in site/content/docs/main.

[dsu-igeek]

Looks good, just some fixes to the messages.

[zubron]

Thanks, @carlisia! This looks good, I really like the clearer feedback to the user about what's happening 👍 I just have a few additional comments along with the ones already mentioned by Dave.

[ashish-amarnath]

LGTM overall.
I have a few minor comments.

[carlisia]

For reviewers: if you think the return on LN 155 seem weird, you are thinking like me and expected the cancel() call to return but no, it continues that iteration til the end and then considers it "done". The return was needed.

Comments addressed

Comments addressed

[zubron]

Thanks, @carlisia!

[dsu-igeek]

LGTM!