vultr / vultr-cli

Official command line tool for Vultr services
Apache License 2.0
478 stars 76 forks source link

Adjust DBaaS VPC pointer to detect changes #366

Closed christhemorse closed 1 year ago

christhemorse commented 1 year ago

Description

Current implementation on master has an issue where not including a vpc-id will treat it as passing an empty one, which detaches the current VPC from a Managed Database. Swaps that to only include VPCID in the update request if the value was explicitly passed, whether it is empty or not (with empty being remove VPC and an ID string setting or changing VPC).

Checklist:

github-actions[bot] commented 1 year ago

Unit Tests and Coverage

Show Output ``` ? github.com/vultr/vultr-cli/v2 [no test files] ```

Pusher: @christhemorse, Action: pull_request_target