Skip to content

Commit

Permalink
Add logging in testDNSStats
Browse files Browse the repository at this point in the history
TestDNSStatsWithNAT is still flaky, I'm noticing that many of the failures are on prebuilt, and the CO-RE
failures have the suspicious 'clang timed out error'. I'm wondering if
this is an issue of the netlink conntracker being behind the real
tracer. These logs will help verify. If all the failures we see are on
prebuilt, we can either skip this test with netlink conntrack,  or add
some synchronization
  • Loading branch information
leeavital committed Dec 11, 2023
1 parent 2af43d0 commit 2315acb
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions pkg/network/tracer/tracer_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1007,6 +1007,9 @@ func testDNSStats(t *testing.T, tr *Tracer, domain string, success, failure, tim
tr.removeClient(clientID)
initTracerState(t, tr)

t.Logf("test: %s@%s running query on %s and expecting success=%d, failure=%d, timeout=%d with conntracker: %T",
t.Name(), domain, serverIP, success, failure, timeout, tr.conntracker)

dnsServerAddr := &net.UDPAddr{IP: net.ParseIP(serverIP), Port: 53}

queryMsg := new(dns.Msg)
Expand Down

0 comments on commit 2315acb

Please sign in to comment.