Dom Delnano

Upgrade finagle to version that includes tls tracing compatible netty-tcnative

1

This PR upgrades finagle to install a version that includes the netty changes I upstreamed (https://github.com/netty/netty/pull/12438, https://github.com/netty/netty-tcnative/pull/731) to support #407. Once this is upgraded, the next step is to refresh...

Consider allowing changes to VM resource's `cloud_config` argument to force recreation

5

This was discussed in [Discord](https://discord.com/channels/704471459279667330/705514553437323360/994317562538115194). It is surprising that the `cloud_config` argument is capable of being changed on a `xenorchestra_vm` when it likely will have no effect. The `aws_instance` resource...

Improve acceptance test suite build quality (decrease flakiness)

4

The current acceptance test suite has grown since the project started. It is now comprised of 30 `xenorchestra_vm` resource tests, which often cause the `go test` command to timeout (running...

Stopped VM does not trigger dirty terraform plan

1

When investigating #196, I noticed that if a VM created with terrraform is stopped a subsequent `terraform plan` will not show that the VM will be started. This means the...

Allow vm disks to be resized on update

4

When a user creates a VM from a template that contains a disk, the disk will be expanded to the size of the VM's specified size. For example, if you...

While working on #179 and past tasks, it's become apparent that the XO client (`github.com/terra-farm/terraform-provider-xenorchestra/client` module) has an inconsistent API design. API calls that operate on VMs typically require a...

Support missing parameters in `vm.set` for Vm updates

2

While #144 improved vm resource updates significantly (specifically #40 and #111), there are many parameters that are not supported with the vm resource. This issue tracks supporting all the `vm.set`...

After debugging the following situation with a user in discord ([link](https://discordapp.com/channels/704471459279667330/705514553437323360/874896211956613180)), I realized that the `xenorchestra_vdi` data source implicitly requires a `pool_id` argument. The problem is this code [here](https://github.com/terra-farm/terraform-provider-xenorchestra/blob/48489d5501720ca44003b4881e3285d476197eeb/client/vdi.go#L31-L36) that...

In #155 I added PXE support to the VM resource. I wanted to have an acceptance test for this feature, however, my main testing environment's PXE menu defaults to "local...

Right now the `xo.getAllObjects` rpc calls filter based on the object's type and then do the remaining filtering on the client side. There was some work done in https://github.com/terra-farm/terraform-provider-xenorchestra/pull/126/files to...