Merge lp://staging/~ahasenack/apport/samba-security-share-bugpattern into lp://staging/~ubuntu-bugcontrol/apport/ubuntu-bugpatterns

Proposed by Andreas Hasenack
Status: Merged
Merged at revision: 568
Proposed branch: lp://staging/~ahasenack/apport/samba-security-share-bugpattern
Merge into: lp://staging/~ubuntu-bugcontrol/apport/ubuntu-bugpatterns
Diff against target: 14 lines (+5/-0)
1 file modified
bugpatterns.xml (+5/-0)
To merge this branch: bzr merge lp://staging/~ahasenack/apport/samba-security-share-bugpattern
Reviewer Review Type Date Requested Status
Brian Murray Approve
Review via email: mp+327251@code.staging.launchpad.net

Description of the change

Bug pattern for bug #1634119

This has been a very common bug opened against samba via apport. An ancient configuration option in /etc/samba/smb.conf, called "security = share", was finally deprecated by upstream, and is no longer valid.

To post a comment you must log in.
Revision history for this message
Brian Murray (brian-murray) wrote :

This looks good although it is a little strange that the pattern fails with bug 1634119, looking at the bug I understand this is because the user commented out the "security = share" line. It might have been nice to mention this in the merge request.

Also keep in mind its possible to use search-bugs, in this branch, to search for and consolidate duplicate bug reports if you want e.g.:

 $ ./search-bugs --package samba
LP: #1703604: bug description must contain standard apport format data
LP: #1703253 (Incomplete, Undecided): Matched bug pattern: https://launchpad.net/bugs/1634119 with 0 dupes
LP: #1703061 (Incomplete, Undecided): Matched bug pattern: https://launchpad.net/bugs/1634119 with 0 dupes
LP: #1702529: bug description must contain standard apport format data

review: Approve
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Thanks for the review.

I didn't want to use search-bugs with --consolidate before this was merged :)

Regarding why it didn't caught #1634119 itself, yes, it's because we need the option to be actually used in the config file, or else the reason smbd isn't restarting is something else. That user did some debugging on his own and uploaded the smb.conf file after he had fixed it.

Some of the bugs (most actually) I left as incomplete expecting the users to come back after they fixed the config, in which case we would close them as invalid or duplicate. That would have been a nice closure. But it's not happening, and the bugs keep coming in, that's why I decided it's time for a bug pattern :)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.