[Feature] [Operator] remove required trailing slash on head svc ingess regex for target rewrite

[davidlhw]

Search before asking

• I had searched in the issues and found no similar feature requirement.

Description

the regex pattern at https://github.com/ray-project/kuberay/blob/v1.1.1/ray-operator/controllers/ray/common/ingress.go#L57

Path: "/" + cluster.Name + "/(.*)"

meant that the path required a trailing slash to be routed to the cluster

example: /test/(.*)

• does not match /test, captures nothing
• matches /test/, captures nothing
• matches /test/#/overview, captures #/overview

perhaps we can improve on this regex pattern to not require the trailing slash. Maybe something like:

Path: "/" + cluster.Name + "(?:$|\/(.*))"

example: /test(?:$|/(.*))

• matches /test, captures nothing
• matches /test/, captures nothing
• matches /test/#/overview, captures #/overview

Use case

No response

Related issues

No response

Are you willing to submit a PR?

• Yes I am willing to submit a PR!