fixes #712 : destroy container fails "500 CT not running"

[btorresgil]

Fixes #712

Previous behavior

Destroy attempts to stop VM even if it's currently stopped. API return HTTP status 500 (see below) which is returned to provider causing it to error out.

HTTP/1.1 500 CT 110 not running
Connection: close
Content-Length: 13
Cache-Control: max-age=0
Content-Type: application/json;charset=UTF-8
Date: Wed, 22 Mar 2023 21:55:11 GMT
Expires: Wed, 22 Mar 2023 21:55:11 GMT
Pragma: no-cache
Server: pve-api-daemon/3.0

{"data":null}

New behavior

Destroy checks if VM is running or stopped and only attempts to stop the VM if status is currently running.

[btorresgil]

Note: much easier to read diff with 'Hide whitespace' enabled. Very few lines actually changed, but indentation changed on several lines to accommodate new behavior.

[mleone87]

nice spot, thanks

[coaxial]

Thanks. Any idea when this will be released to the Terraform registry?

[maxkratz]

Any schedule to release a new version of the provider to include this fix?