Add retries to delete on load balancers. Additionally upgrade this file to use retry.StateChangeConf instead of the deprecated resource.StateChangeConf.
There is an issue with the Vultr Update Load Balancer API - it sets healthcheck.path to healthcheck.protocol instead. I'm not sure where to file a bug report, but I have changed the test to work around this issue for now and added some TODO's in the LB code.
Related Issues
Fixes #398
Checklist:
[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[X] Have you linted your code locally prior to submission?
[X] Have you successfully ran tests with your changes locally?
Yes - only for this resource. The full suite doesn't work.
Description
Add retries to delete on load balancers. Additionally upgrade this file to use
retry.StateChangeConf
instead of the deprecatedresource.StateChangeConf
.There is an issue with the Vultr Update Load Balancer API - it sets
healthcheck.path
tohealthcheck.protocol
instead. I'm not sure where to file a bug report, but I have changed the test to work around this issue for now and added some TODO's in the LB code.Related Issues
Fixes #398
Checklist: