Make strip_query in google_compute_url_map optional

[Chupaka]

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

There are three places in docs where strip_query is described as "This field is required to ensure an empty block is not set." But as I can see, there is a better approach used, for example, in http_health_check: "at_least_one_of" https://github.com/GoogleCloudPlatform/magic-modules/blob/837e4128c650791b228b0adb7c86cd6683a50b03/products/compute/api.yaml#L4224

So google_compute_url_map can be refactored to remove that "dirty hack" with required field.

New or Affected Resource(s)

• google_compute_url_map

b/374162367

[ggtisc]

Hi @Chupaka!

There is this optional strip_query and these other 3 mandatory strip_query resources:

1. url_redirect
2. default_url_redirect
3. 2nd default_url_redirect

As you can see in terraform registry each one has different purposes according to user needs

[Chupaka]

@ggtisc aren't all your links the same?..

Anyway, what I was talking about is that "strip_query" should not be mandatory just "to ensure an empty block is not set". E.g., if I set https_redirect, I don't want to set also strip_query, because my url_redirect block is not empty already, but I still have to add that silly strip_query.