Remove unmaintained ansi_term dependency

[kate-goldenring]

What this PR does / why we need it:
fixes #502 by bumping webhook's clap version to latest which was previously pulling in the unmaintained dependency

If applicable:

• version has been updated appropriately (./version.sh)
• all commits pass the DCO bot check by being signed off -- see the failing DCO check for instructions on how to retroactively sign commits

[adithyaj]

great simple fix for this issue, looks good to me

[kate-goldenring]

Looks like in the e2e tests we are failing to spin up a K8s v1.16 cluster. We probably need to change the install script, but better yet, i think it may be time to remove older versions of k8s from the test matrix. This could be part of the release steps. @bfjelds and @adithyaj, thoughts?

[adithyaj]

Looks like in the e2e tests we are failing to spin up a K8s v1.16 cluster. We probably need to change the install script, but better yet, i think it may be time to remove older versions of k8s from the test matrix. This could be part of the release steps. @bfjelds and @adithyaj, thoughts?

Yup this seems to be affecting #508 too. I believe all the kubernetes versions in our test matrix (1.16-1.21) are at EOL. Let's maybe test against the latest EOL version and the rest of the active supported branches (1.21-1.25)? We can also update the release process to check for this.

@michaelzhang114 let's talk about it as a quick agenda item as part of the release for tomorrow's community meeting.

[kate-goldenring]

@bfjelds @romoh @Britel can someone tap the approve to get this dep removed?

[adithyaj]

looks good to me, and the version got bumped 0.8.20 so we can merge this now

[kate-goldenring]

looks good to me, and the version got bumped 0.8.20 so we can merge this now

we need a maintainer approval to merge. @bfjelds @romoh