Bug 10015 - Fix/Improve debug options
Summary: Fix/Improve debug options
Status: RESOLVED FIXED
Alias: None
Product: Samba 4.1 and newer
Classification: Unclassified
Component: AD: LDB/DSDB/SAMDB (show other bugs)
Version: unspecified
Hardware: All All
: P5 normal (vote)
Target Milestone: ---
Assignee: Karolin Seeger
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-07-12 09:13 UTC by Stefan Metzmacher
Modified: 2013-07-29 19:37 UTC (History)
1 user (show)

See Also:

Attachments
Patches for v4-0-test (7.09 KB, patch)
2013-07-12 09:13 UTC, Stefan Metzmacher 		abartlet: review+
Details
Patches for v4-1-test (5.70 KB, patch)
2013-07-19 07:47 UTC, Stefan Metzmacher 		abartlet: review+
Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Metzmacher 2013-07-12 09:13:40 UTC 
Created attachment 9042 [details]
Patches for v4-0-test

The options for debugging should match between testparm and 'samba-tool testparm'

testparm/smbd/nmbd/winbindd:

        debug timestamp = Yes
        debug prefix timestamp = No
        debug hires timestamp = Yes
        debug pid = No
        debug uid = No
        debug class = No
        syslog = 1
        syslog only = No

samba-tool testparm/samba:

        debug timestamp = No (eventively Yes)
        debug prefix timestamp = No
        debug hires timestamp = No
        debug pid = No
        debug uid = No
        debug class = No
        syslog = 0
        syslog only = No

It's also bad that ldb debugs tracings at level 5.
We should move that to level 10 and add a debug class 'ldb' for people
who want it at level 5.
Comment 1 Andrew Bartlett 2013-07-12 12:04:27 UTC 
Assigning to Karolin for 4.0 and 4.1
Comment 2 Karolin Seeger 2013-07-15 19:02:08 UTC 
The patch does not apply to v4-1-test.
I think it's because "[PATCH 1/5] debug: Add debugclass for DNS server" has already been pushed to this branch.
Patches with two reviews should at least apply...
Comment 3 Karolin Seeger 2013-07-15 19:07:02 UTC 
Pushed to autobuild-v4-0-test.
Waiting for a version for v4-1-test.
Comment 4 Karolin Seeger 2013-07-18 17:52:54 UTC 
Pushed to v4-0-test.
Comment 5 Karolin Seeger 2013-07-18 17:53:21 UTC 
Re-assigning to Metze.
Comment 6 Stefan Metzmacher 2013-07-19 07:47:12 UTC 
Created attachment 9066 [details]
Patches for v4-1-test

This should apply to v4-1-test
Comment 7 Karolin Seeger 2013-07-24 18:31:26 UTC 
(In reply to comment #6)
> Created attachment 9066 [details]
> Patches for v4-1-test
> 
> This should apply to v4-1-test

Thanks!
Pushed to auobuild-v4-1-test.
Comment 8 Karolin Seeger 2013-07-29 19:37:57 UTC 
Pushed to v4-1-test.
Closing out bug report.

Thanks!