Azure: add force delete option

[Jont828]

What type of PR is this?

/kind feature

What this PR does / why we need it: Add force delete option in Azure to the Helm chart and Azure autoscaler.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?

Azure: add force delete option

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

[jackfrancis]

You'll want to cut a new chart version (Chart.yaml) w/ these changes. I'd say that this warrants a minor bump.

[jackfrancis]

/assign @gandhipr

[gandhipr]

Can we add some testing details performed?

[Jont828]

Sure, also forgot to mention this is still WIP

[Jont828]

@gandhipr @jackfrancis Fixed unit tests, ready for final review pass before I squash.

[gandhipr]

/lgtm

[jackfrancis]

@Jont828 you'll need to bump the chart version (which will mean whichever of your PRs merges first will conflict with the other one that also needs a chart version bump, fyi)

[jackfrancis]

/test pull-cluster-autoscaler-e2e-azure

[jackfrancis]

/test ?

[k8s-ci-robot]

@jackfrancis: The following commands are available to trigger optional jobs:

• /test pull-cluster-autoscaler-e2e-azure

Use /test all to run all jobs.

In response to this:

/test ?

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Jont828 Once this PR has been reviewed and has the lgtm label, please ask for approval from gandhipr. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• cluster-autoscaler/cloudprovider/azure/OWNERS

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

[jackfrancis]

/test pull-cluster-autoscaler-e2e-azure

[Jont828]

@jackfrancis @gandhipr Are we good to merge this?

[jackfrancis]

Seems like something got lost here? I only see a fraction of the original changes.

[Jont828]

@jackfrancis We forgot to remove the commit containing force delete in #6447 so force delete actually got merged already. This is just a follow up to address comments and for consistency.

[Shubham82]

/remove-area helm-charts removing the /area helm-chart label as there are no changes/modifications by this PR (now) in the Charts directory.