Match multiple methods in HTTPRouteMatch

[tamalsaha]

What would you like to be added:

gateway-api/apis/v1beta1/httproute_types.go

Line 546 in f844222

Method *HTTPMethod `json:"method,omitempty"`

It will be better DX to support multiple method matches in HTTPRouteMatch

Method []HTTPMethod json:"method,omitempty"``

Why this is needed:

While this can be done using multiple match rules, it is better DX to support []HTTPMethod. Also, the other matches (header, query params) support array. So, it will be also more consistent.

Similar issue also exists in GRPC api.

[youngnick]

I think this is a reasonable request, but unfortunately, it's a breaking API change, so we'll probably need to add a plural field (i.e. Methods instead of Method, with semantics for cross-populating Method into Methods etc). This sucks to some extent but we don't want to make a breaking API change for HTTPRoute at this point, sadly.

[youngnick]

The upstream reference on the right way to make a singular field plural is https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/api_changes.md#making-a-singular-field-plural

[shaneutt]

The upstream reference on the right way to make a singular field plural is https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/api_changes.md#making-a-singular-field-plural

So it would seem this is doable, albeit it's not going to be simple or quick. Marking this as accepted and giving it a priority for now and we'll allow some time to see if anyone wants to take it on:

/remove-triage
/priority backlog
/help

[k8s-ci-robot]

@shaneutt:
This request has been marked as needing help from a contributor.

Guidelines

Please ensure that the issue body includes answers to the following questions:

• Why are we solving this issue?
• To address this issue, are there any code changes? If there are code changes, what needs to be done in the code and what places can the assignee treat as reference points?
• Does this issue have zero to low barrier of entry?
• How can the assignee reach out to you for help?

For more details on the requirements of such an issue, please see here and ensure that they are met.

If this request no longer meets these requirements, the label can be removed
by commenting with the /remove-help command.

In response to this:

The upstream reference on the right way to make a singular field plural is https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/api_changes.md#making-a-singular-field-plural

So it would seem this is doable, albeit it's not going to be simple or quick. Marking this as accepted and giving it a priority for now and we'll allow some time to see if anyone wants to take it on:

/remove-triage
/priority backlog
/help

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.