add stack_type attribute

[ehorning]

Add stack_type attribute for google_compute_interconnect_attachment resource.

Note: Explicit test for this attribute cannot be added in examples because this attribute cannot be added for attachments of type PARTNER, attachments of type DEDICATED require interconnect to be specified, and there is no Terraform resource for creating an interconnect. Attachments of type PARTNER_PROVIDER cannot be created without being an approved partner.

Fixes: hashicorp/terraform-provider-google#17129

Release Note Template for Downstream PRs (will be copied)

compute: added `stack_type` attribute for `google_compute_interconnect_attachment` resource.

[modular-magician]

Hello! I am a robot. It looks like you are a: Community Contributor Googler Core Contributor. Tests will run automatically.

@c2thorn, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look.

You can help make sure that review is quick by doing a self-review and by running impacted tests locally.

[modular-magician]

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 2 files changed, 42 insertions(+))
Terraform Beta: Diff ( 2 files changed, 42 insertions(+))
TF Conversion: Diff ( 1 file changed, 10 insertions(+))

Missing test report

Your PR includes resource fields which are not covered by any test.

Resource: google_compute_interconnect_attachment (2 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_compute_interconnect_attachment" "primary" {
  stack_type = # value needed
}

[modular-magician]

Tests analytics

Total tests: 868
Passed tests 804
Skipped tests: 64
Affected tests: 0

Click here to see the affected service packages

• compute

$\textcolor{green}{\textsf{All tests passed in REPLAYING mode.}}$
View the build log

[c2thorn]

LGTM given lack of testability

[ehorning]

LGTM given lack of testability

@c2thorn Do you mind merging in that case? I don't have access to.