google_monitoring_notification_channel requires display_name

[nesl247]

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 an 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 an issue is assigned to a user, that user is claiming responsibility for the issue. If an issue is assigned to hashibot, a community member has claimed the issue already.

Terraform Version

N/A

Affected Resource(s)

• google_monitoring_notification_channel

Terraform Configuration Files

N/A

Debug Output

N/A

Panic Output

N/A

Expected Behavior

display_name should not be required or the description should not say it is optional

Actual Behavior

display_name is required

Steps to Reproduce

See https://www.terraform.io/docs/providers/google/r/monitoring_notification_channel.html#display_name

Important Factoids

N/A

References

• https://www.terraform.io/docs/providers/google/r/monitoring_notification_channel.html#display_name

[edwardmedia]

display_name is required. Please update doc for this field accordingly

[nesl247]

@edwardmedia Out of curiosity, why is it required? Because it isn't required on GCP's web console.

As in, we should default to '' instead of making this required.

[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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!