Make metric_descriptor argument optional and computer for resource_logging_metric

[LucaPrete]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment. If the issue is assigned to the "modular-magician" user, it is either in the process of being autogenerated, or is planned to be autogenerated soon. If the issue is assigned to a user, that user is claiming responsibility for the issue. If the issue is assigned to "hashibot", a community member has claimed the issue already.

Description

Make metric_descriptor argument optional and computer for resource_logging_metric.
If not specified the field is automatically populated from Google APIs.

Affected Resource(s)

• google_logging_metric

Potential Terraform Configuration

Need to:

• remove required: false in logging api.yaml
• add default_from_apis: true in terraform.yaml
• remove metric_descriptor where not needed in tests

References

• API reference which mentions the field as optional

[roaks3]

This appears to have been fixed by GoogleCloudPlatform/magic-modules#6937

[github-actions]

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.