-
Notifications
You must be signed in to change notification settings - Fork 5.3k
Closed
Labels
enhancementFeature requests. Not bugs or questions.Feature requests. Not bugs or questions.
Milestone
Description
When using the thrift proxy, we risk having very long route tables, e.g.:
"routes": [
{
"match": {
"headers": [
{
"exact_match": "foo",
"name": "x-mesh-cluster"
}
],
"service_name": "FooService"
},
"route": {
"cluster": "foo_cluster",
"strip_service_name": true
}
},
{
"match": {
"headers": [
{
"exact_match": "bar",
"name": "x-mesh-cluster"
}
],
"service_name": "BarService"
},
"route": {
"cluster": "bar_cluster",
"strip_service_name": true
}
},
....
We can simplify and collapse this by supporting cluster_header — like in the HCM — to determine the cluster to route to:
https://github.com/envoyproxy/envoy/blob/master/api/envoy/api/v2/route/route.proto#L676
@fishcakez @zuercher sounds reasonable?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementFeature requests. Not bugs or questions.Feature requests. Not bugs or questions.