"retryOn" configuration on ServiceRouter CRD

[mr-miles]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request. Searching for pre-existing feature requests helps us consolidate datapoints for identical requirements into a single place, thank you!
• Please do not leave "+1" or other comments that do not add relevant new information or questions, 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.

---

Overview of the Issue

I set the retryOn field in a service-router crd. The field is specified here:
https://developer.hashicorp.com/consul/docs/v1.16.x/connect/config-entries/service-router#routes-destination-retryon

When I queried k8s for the object, the field was not present. It looks like the crds don't match:
https://github.com/hashicorp/consul-k8s/blob/main/charts/consul/templates/crd-servicerouters.yaml#L151

Github search also pulled up an open PR from June:
#2435

Expected behavior

I expected the fields to be set and used in sidecar config, as per the consul docs.

Environment details

EKS 1.27
Consul 1.16.3
Consul-k8s: 1.2.3

[david-yu]

Thanks for calling this out. This is a consequence of this PR hashicorp/consul#12890 only being merged in core by another contributor. So technically this is inaccurate: https://developer.hashicorp.com/consul/docs/v1.16.x/connect/config-entries/service-router#spec-routes-destination-retryon. This CRD config needs to be added to the ServiceRouter CRD.