diff --git a/0008-libnetwork-resolve-use-structured-logs-for-DNS-error.patch b/0008-libnetwork-resolve-use-structured-logs-for-DNS-error.patch new file mode 100644 index 0000000000000000000000000000000000000000..92aef4e4a04ff7367fa7d48333be09028506935b --- /dev/null +++ b/0008-libnetwork-resolve-use-structured-logs-for-DNS-error.patch @@ -0,0 +1,40 @@ +From d9e082ff54e685a08b193ccb67c5fa3c0a205f3f Mon Sep 17 00:00:00 2001 +From: Sebastiaan van Stijn +Date: Tue, 20 Feb 2024 17:01:06 +0100 +Subject: [PATCH] libnetwork: resolve: use structured logs for DNS error + +I noticed that this log didn't use structured logs; + + [resolver] failed to query DNS server: 10.115.11.146:53, query: ;google.com.\tIN\t A" error="read udp 172.19.0.2:46361->10.115.11.146:53: i/o timeout + [resolver] failed to query DNS server: 10.44.139.225:53, query: ;google.com.\tIN\t A" error="read udp 172.19.0.2:53991->10.44.139.225:53: i/o timeout + +But other logs did; + + DEBU[2024-02-20T15:48:51.026704088Z] [resolver] forwarding query client-addr="udp:172.19.0.2:39661" dns-server="udp:192.168.65.7:53" question=";google.com.\tIN\t A" + DEBU[2024-02-20T15:48:51.028331088Z] [resolver] forwarding query client-addr="udp:172.19.0.2:35163" dns-server="udp:192.168.65.7:53" question=";google.com.\tIN\t AAAA" + DEBU[2024-02-20T15:48:51.057329755Z] [resolver] received AAAA record "2a00:1450:400e:801::200e" for "google.com." from udp:192.168.65.7 + DEBU[2024-02-20T15:48:51.057666880Z] [resolver] received A record "142.251.36.14" for "google.com." from udp:192.168.65.7 + +As we're already constructing a logger with these fields, we may as well use it. + +Signed-off-by: Sebastiaan van Stijn +--- + libnetwork/resolver.go | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +diff --git a/libnetwork/resolver.go b/libnetwork/resolver.go +index 9df2154499..816f00ad68 100644 +--- a/libnetwork/resolver.go ++++ b/libnetwork/resolver.go +@@ -576,7 +576,7 @@ func (r *Resolver) exchange(ctx context.Context, proto string, extDNS extDNSEntr + UDPSize: dns.MaxMsgSize, + }).ExchangeWithConn(query, &dns.Conn{Conn: extConn}) + if err != nil { +- r.log(ctx).WithError(err).Errorf("[resolver] failed to query DNS server: %s, query: %s", extConn.RemoteAddr().String(), query.Question[0].String()) ++ logger.WithError(err).Error("[resolver] failed to query external DNS server") + span.RecordError(err) + span.SetStatus(codes.Error, "ExchangeWithConn failed") + return nil +-- +2.43.0 +