[cluster-autoscaler] Backport fixes for packet provider to release-1.17

[detiber]

No description provided.

[detiber]

/assign @MaciekPytel

[detiber]

/assign @displague

[MaciekPytel]

The tests have failed with what seems like an unrelated error in Cluster API tests. At a glance it seems to be the same issue as #3597. @elmiko Can you take a look and possibly open cherry-pick PRs as you suggested in #3597 (comment)?

I'm going to re-open to try and trigger a travis rerun.

[elmiko]

@elmiko Can you take a look and possibly open cherry-pick PRs as you suggested in #3597 (comment)?

sure thing, i should be able to take a look today

[displague]

/lgtm

[MaciekPytel]

Looks like the same test has failed again. I've already included cherry-picks of this in 1.19 and 1.18 patch releases, ideally it should also go into 1.17. I'd appreciate help fixing this, I'm happy to delay 1.17 release by a day or two if needed.

[elmiko]

@MaciekPytel i creaated this pr, #3662

looks like it shoud fix the issue

$ stress ./clusterapi.test -test.run='TestControllerNodeGroups$' -test.cpu=8
106 runs so far, 0 failures
226 runs so far, 0 failures
324 runs so far, 0 failures

[MaciekPytel]

Thanks for fixing it so quickly @elmiko.
I'm logging off now, I'll merge this PR tomorrow after #3662 is merged.

[MaciekPytel]

/lgtm
/approve

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: detiber, MaciekPytel

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:

• cluster-autoscaler/OWNERS [MaciekPytel]

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