feat(GKE): add sample for using GKE Managed OpenTelemetry

[giuliano-sider]

Description

This adds a Terraform config sample for enabling GKE Managed OpenTelemetry.

Checklist

Readiness

• [ X ] Yes, merge this PR after it is approved

Style

• [ X ] My sample follows the rules described for Terraform in the Effective Samples style guide
• [ X ] My sample follows the other requirements and best practices in the Contributing
  guide

Testing

• I have performed tests described in the Contributing guide:

  • Tests pass: terraform apply
  • [ X ] Lint pass: terraform fmt check

Intended location

• [ X ] Yes, this sample will be (or already is) included on cloud.google.com
  Location(s): https://docs.cloud.google.com/kubernetes-engine/docs/how-to/managed-otel-gke

• No, this sample won't be included on cloud.google.com
  Reason:

API enablement

• If the sample needs an API enabled to pass testing, I have added the service to the Test setup file

Review

• If this sample adds a new directory, I have added codeowners to the CODEOWNERS file

[snippet-bot]

Here is the summary of changes.

You are about to add 1 region tag.

• gke/standard/regional/managed_opentelemetry/main.tf:24, tag gke_standard_regional_with_managed_otel

---

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

• Refresh this comment

[apeabody]

Do we need to pessimistic pin to the patch, or is the minor version sufficient?

Suggested change

	version = "~> 7.17.0"
	version = "~> 7.17"

[giuliano-sider]

I actually meant to use a minimum version for the google-beta provider (fixed now).

[apeabody]

/gcbrun

[iennae]

/gcbrun

[apeabody]

Thanks for the contribution @giuliano-sider!

Please complete the "Location" in the Checklist.

[apeabody]

Please drop the terraform block: "(Required) Samples maintained in terraform-docs-samples must include only resource and data blocks. Don't include module blocks in your samples."

[glasnt]

There's an exception for google-beta in https://googlecloudplatform.github.io/samples-style-guide/#language-specific; Ideally all new samples that require google-beta will have these requirements removed once the functionality moves to GA, but this format ensures it works today.

[apeabody]

Please drop the output block: "(Required) Samples maintained in terraform-docs-samples must include only resource and data blocks. Don't include module blocks in your samples."

[giuliano-sider]

The terraform and output blocks are encouraged by https://googlecloudplatform.github.io/samples-style-guide/. Should we clarify the different requirements in this repo mentioned in https://github.com/terraform-google-modules/terraform-docs-samples/blob/main/CONTRIBUTING.md#code-style?

[apeabody]

Thoughts @glasnt?

[giuliano-sider]

I added the GCP docs location for where to use this sample.

Should I just drop the terraform and output blocks then? I think they're useful, but if the repo style guide forbids them, I'll just edit them out.

[glasnt]

Per earlier comment, because this resource currently requires a minimum google-beta provider, keep that block.

I'm not sure what purpose the output block here has, other than confirming configuration success. If the output was a generated value that the user needed to gather for further operations, I could understand it's inclusion, but in this case, if the Terraform applies correctly, this doesn't need to be confirmed with the output block.