Support recursive targeting on modules

[josephholsten]

Rebase of #9236

[apparentlymart]

Note to self, for later review in more detail: this patch changes the ResourceAddress population so that when no resource type is present Type is set to the magic string "module". Subsequent changes since this PR was originally submitted have added a HasResourceSpec method to ResourceAddress that would be broken by merging this, since it expects a resource-only address to have an empty Type.

[josephholsten]

also note: the original PR contained modification to terraform/transform_resource.go but the modern terraform/transform_resource_count.go uses a completely different implementation. I have no idea if that could break anything.

[dcosson]

Any chance of getting this reviewed/merged? From #9236 it sounded like this was blocked on being able to merge breaking changes for 0.10, and since there are now 2 betas out for 0.10 I'm guessing we've reached that point?

[apparentlymart]

Ahh, I had independently rebased and tweaked this in #15314, which went out in 0.10.0-beta1. Sorry I didn't catch this one while I was working on that.

[dcosson]

Oh awesome! Thanks for getting this through

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.