Alex Pilon

Closing as per @tenthirtyam recommendation

I'll re-open and have it close once we at least provide a documentation update

Hello @bloudraak, Plural datasources is a concept that has been brought up a few times over the years. The demand for them has made more sense with Terraform's 0.13 release...

Naively, I would expect the provider to error if it was unable to delete something upstream (would expect to get an error from vsphere API/govmomi) and have that bubble up...

I have taken the liberty of rebasing and making necessary changes to this PR. The provided acceptance test seems acceptable, however it's not an environment/feature we have the ability to...

After further inspection, one thing I am struggling with on the PR is the awkward polymorphic behavior between the 2 disk types. Some things are nil for one type and...

@tenthirtyam Coming back to this, the best way forward I believe is introducing a new nested block `disk_rdm`, that way we can keep the attributes required by different disk types...

I will go ahead and tweak this PR to that implementation, from there, as stated earlier, I'll ask the community to test the new disk type, as we do not...

@sumitAgrawal007 @tenthirtyam After digging into this PR further, it's unclear to me how the RDM disk relevant data made its way to the backend (vsphere)? Specifically the compatibility mode, after...

I will have to get back to you on what the other providers have standardized on in terms of linting, `tfproviderlint` was built around avoiding common tropes in provider development...