This PR contains the update as well as changes needed to get the provider to compile, and the tests to pass. This includes fixes for changes introduced by the new plugin SDK, as well as other issues that came up from referencing different template IDs in Skytap.
ff880c8 - stops panicking when using a VM template with a network interface detached from a network
516df50 - pauses between test steps to allow things to settle within Skytap system
b8093fc - adds a validation check* for the number of vCPUs to make sure it doesn't exceed the number of GBs of RAM
76a83ee - increases the timeouts for the VM and environment resources to allow larger deploys to complete
*: the validation check fixes a source of a major timeout issue. Previously, if the user specified these values incorrectly (i.e. more CPUs than RAM), then the API would return a 409 error. This would cause the Skytap Go SDK to go into a perpetual loop until the context deadline was exceed, i.e. the timeout had been reached. The resource will now check for this case explicitly and return an error early.
This PR contains the update as well as changes needed to get the provider to compile, and the tests to pass. This includes fixes for changes introduced by the new plugin SDK, as well as other issues that came up from referencing different template IDs in Skytap.
*: the validation check fixes a source of a major timeout issue. Previously, if the user specified these values incorrectly (i.e. more CPUs than RAM), then the API would return a 409 error. This would cause the Skytap Go SDK to go into a perpetual loop until the context deadline was exceed, i.e. the timeout had been reached. The resource will now check for this case explicitly and return an error early.