feat: set full URI for the envoy-gateway service using name and namespace#4533
feat: set full URI for the envoy-gateway service using name and namespace#4533arkodg merged 3 commits intoenvoyproxy:mainfrom
Conversation
…pace Signed-off-by: Rajat Vig <rvig@etsy.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4533 +/- ##
==========================================
- Coverage 65.72% 65.67% -0.05%
==========================================
Files 211 211
Lines 31669 31672 +3
==========================================
- Hits 20813 20801 -12
- Misses 9656 9668 +12
- Partials 1200 1203 +3 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Rajat Vig <rvig@etsy.com>
| TrustedCA: filepath.Join("/sds", common.SdsCAFilename), | ||
| }, | ||
| MaxHeapSizeBytes: maxHeapSizeBytes, | ||
| XdsServerHost: ptr.To(fmt.Sprintf("envoy-gateway.%s.svc.%s", namespace, dnsDomain)), |
There was a problem hiding this comment.
can we use config.EnvoyGatewayServiceName here ?
|
can you share more details about the benifit? reduce DNS pressure? |
Signed-off-by: Rajat Vig <rvig@etsy.com>
Yes. It would reduce the number of DNS resolve requests as just The typical You can test it out by running and comparing against a CoreDNS pod with logging turned on to see the diference in number of queries. We see Should add, this is not true when |
|
@rajatvig looks like this broken xds_cluster dns reslove on IPv6 cluster? |
…nd namespace (envoyproxy#4533)" This reverts commit 3e8730f. Signed-off-by: zirain <zirain2009@gmail.com>
Signed-off-by: Rajat Vig rvig@etsy.com
What type of PR is this?
Sets the
xdsServerHostin Kubernetes to be fully qualified Service Name using the namespace.What this PR does / why we need it:
Instead of defaulting to
envoy-gatewaysets thexdsServerHostto beenvoy-gateway.envoy-gateway-system.svc.cluster.local.Release Notes: No