Web lists-archives.com

Re: [Samba] DNS Log level - patch


Am 26.07.2017 um 07:01 schrieb Andrew Bartlett via samba:
>> I couldn't find a setting for DNS log level, so I went with:
>>   log level = 0 passdb:2 auth:2 winbind:2 locking:2
>> My log is still getting filled with "Not authoritative for...."
>> messages.
> Sadly the source4 code used for most of the AD DC doesn't have the
> debug classes implemented.  

It is already implemented for what John wants to do. Set:

   log level = 0 passdb:2 auth:2 winbind:2 locking:2 dns:1

and the "Not authoritative for" messages are gone (verified on 4.6.5).

The "dns" debug class already existed in 4.6, but wasn't listed in the
man page. However, it is in the 4.7 smb.conf man page.

> Patches are welcome, there isn't any technical reason why
> they can't, and likewise debug levels can be corrected,
> this should be about a 5 or more, it is normal traffic.

Find attached a patch that changes the log level for this entry to 5.
Please review and push. Thanks.

>From 086f4e1bd36c89b004095f0c46b41dbdd159e8d1 Mon Sep 17 00:00:00 2001
From: Marc Muehlfeld <mmuehlfeld@xxxxxxxxx>
Date: Wed, 26 Jul 2017 08:00:24 +0200
Subject: [PATCH] Set log level for "Not authorative for" from 2 to 5

Signed-off-by: Marc Muehlfeld <mmuehlfeld@xxxxxxxxx>
 source4/dns_server/dns_query.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/source4/dns_server/dns_query.c b/source4/dns_server/dns_query.c
index b8ecc2e7ac7..4b5bb0772a4 100644
--- a/source4/dns_server/dns_query.c
+++ b/source4/dns_server/dns_query.c
@@ -1056,7 +1056,7 @@ struct tevent_req *dns_server_process_query_send(
 	if ((req_state->flags & DNS_FLAG_RECURSION_DESIRED) &&
 	    (req_state->flags & DNS_FLAG_RECURSION_AVAIL)) {
-		DEBUG(2, ("Not authoritative for '%s', forwarding\n",
+		DEBUG(5, ("Not authoritative for '%s', forwarding\n",
 		subreq = ask_forwarder_send(state, ev, dns,

To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/options/samba