[release-1.10] fix AuthorizedIPRanges disable -> enable bug

[nawazkh]

What type of PR is this?
/kind bug

What this PR does / why we need it:

• Cherrypick of 7d98805
• Also copies over //nolint:gocyclo to the Parameters() function from main since [release-1.10] fix AuthorizedIPRanges disable -> enable bug #3800 was erroring out on go-cyclomexic complexity (A lot of nil checks).

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):

Special notes for your reviewer:

• cherry-pick candidate

TODOs:

• squashed commits
• includes documentation
• adds unit tests

Release note:

authorized IP ranges will be set to []string then to nil upon deletion

[nawazkh]

I will squash the commits once lgtm-ed.

[mboersma]

/retitle [release-1.10] fix AuthorizedIPRanges disable -> enable bug

[nawazkh]

/test pull-cluster-api-provider-azure-test-v1beta1

[codecov]

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.05% 🎉

Comparison is base (50232eb) 54.05% compared to head (8e98a39) 54.10%.
Report is 1 commits behind head on release-1.10.

❗ Current head 8e98a39 differs from pull request most recent head 934e1bd. Consider uploading reports for the commit 934e1bd to get more accurate results

Additional details and impacted files

@@               Coverage Diff                @@
##           release-1.10    #3805      +/-   ##
================================================
+ Coverage         54.05%   54.10%   +0.05%     
================================================
  Files               186      186              
  Lines             18837    18847      +10     
================================================
+ Hits              10182    10198      +16     
+ Misses             8107     8103       -4     
+ Partials            548      546       -2     

Files Changed	Coverage Δ
azure/services/managedclusters/spec.go	48.71% <100.00%> (+2.72%)	⬆️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mboersma]

Also copies over //nolint:gocyclo to the Parameters() function

I opened issue #3810 to track this, so we can remove the //nolint someday.

[CecileRobertMichon]

/lgtm

/hold

Can we squash commits 2 and 3 together before merge?

[k8s-ci-robot]

LGTM label has been added.

Git tree hash: a8452090f66117ba4effa7ad46b884896945b802

[nawazkh]

Can we squash commits 2 and 3 together before merge?

Done! :)
/unhold

[mboersma]

/lgtm
/approve

[k8s-ci-robot]

LGTM label has been added.

Git tree hash: 748458f280f1f16e18600a3db36ceb928054f483

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mboersma

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

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [mboersma]

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