Commit 7292e27
otel: opt into port-aware no_proxy matching for integration_tests_otel (#54151) ## Summary -
## Test plan
- [x] `integration_tests_otel` passes in CI on this branch under fabric-proxy-injected runners without timing out
Co-authored-by: hugo.beauzee <hugo.beauzee@datadoghq.com>
(cherry picked from commit 1053fe7)
___
Co-authored-by: Jorge Torres Martinez <88950055+jorgetomtz@users.noreply.github.com>integration_tests_otel times out when the CI runner has a fabric-proxy injected. The Agent's legacy no_proxy bypass logic compares the full host:port against bare no_proxy entries, so it never bypasses the proxy for loopback test servers/health checks bound to ephemeral ports — those calls get routed through the proxy, which has no route to them and hangs instead of failing fast, until the job times out. - Sets DD_NO_PROXY_NONEXACT_MATCH: "true" on the job so it uses the port-aware no_proxy matching path (golang.org/x/net/http/httpproxy) instead of the legacy exact-match path. - This is a targeted CI-level workaround, not a fix to the underlying Agent behavior (the legacy exact-match logic in pkg/util/http/transport.go still has the same defect for other callers). Proposing this narrowly scoped fix first since the existence of no_proxy_nonexact_match as a deprecation-flagged, opt-in setting suggests the team is already aware of this gap and staging a fix; happy to also raise the broader fix if the team wants it.1 parent 6957c42 commit 7292e27
1 file changed
Lines changed: 4 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
13 | 17 | | |
14 | 18 | | |
15 | 19 | | |
| |||
0 commit comments