krystal / terraform-provider-katapult

Terraform Provider for Katapult.
https://registry.terraform.io/providers/krystal/katapult/latest/docs
MIT License
5 stars 2 forks source link

feat: virtual machine groups - add resource #54

Closed andrewjtait closed 3 years ago

andrewjtait commented 3 years ago

Adds the Terraform resource for managing Virtual Machine Groups in Katapult.

This is just the first part to fully supporting groups, as we will need to add support to use it as a data source and to add them to virtual machines.

jimeh commented 3 years ago

To expand further on testing, the convention I've gone with is have a _minimal test where I only specify required attributes and validate what everything gets set to. This could also double as the test for when segregate is not specified. Then I'd suggest a test for when it's set to true, and one for when it's set to false, in addition to a update test which changes the name, and segregate value back and fourth between true/false.