Bug 9386 - Winbind does't failover to next DC if netlogon is not available.
Summary: Winbind does't failover to next DC if netlogon is not available.
Status: RESOLVED FIXED
Alias: None
Product: Samba 4.0
Classification: Unclassified
Component: Winbind (show other bugs)
Version: 4.0.0rc4
Hardware: All All
: P5 normal (vote)
Target Milestone: ---
Assignee: Karolin Seeger
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks: 9176
  Show dependency tree / graph
 
Reported: 2012-11-12 11:30 UTC by Andreas Schneider
Modified: 2012-11-13 17:26 UTC (History)
1 user (show)

See Also:

Attachments
master patch (3.02 KB, patch)
2012-11-12 11:32 UTC, Andreas Schneider 		metze: review+
Details
Patch for v4-0-test and v3-6-test (3.27 KB, patch)
2012-11-13 05:29 UTC, Stefan Metzmacher 		asn: 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 Andreas Schneider 2012-11-12 11:30:31 UTC 
If a Windows DC crashes and is restarted, it rebuilds the sysvol. During that time the netlogon service is not available and users are not able to login. They get strange errors like NT_STATUS_BUFFER_TOO_LONG.

We don't failover to another DC in this case.
Comment 1 Andreas Schneider 2012-11-12 11:32:33 UTC 
Created attachment 8183 [details]
master patch

This patch improves the situation. First it retries to connect to the netlogon pipe. On the second try it closes the connection and reconnects. On the third third try it adds the DC to the negative cache entry to failover to the next DC.
Comment 2 Andreas Schneider 2012-11-12 11:39:14 UTC 
@metze: If the patch is fine please add your review line and push it to master.


The patch applies cleanly to v3-6-test too!
Comment 3 Stefan Metzmacher 2012-11-12 13:16:00 UTC 
Comment on attachment 8183 [details]
master patch

This patch is for master
Comment 4 Stefan Metzmacher 2012-11-12 13:16:42 UTC 
(In reply to comment #3)
> Comment on attachment 8183 [details]
> master patch
> 
> This patch is for master

I'm pushing it now and will upload the cherry-picked version for v4-0-test
Comment 5 Stefan Metzmacher 2012-11-13 05:29:08 UTC 
Created attachment 8187 [details]
Patch for v4-0-test and v3-6-test
Comment 6 Andreas Schneider 2012-11-13 08:32:46 UTC 
Karolin, can you please add the patch to v4-0-test and v3-6-test? Thanks!
Comment 7 Karolin Seeger 2012-11-13 08:50:31 UTC 
Pushed to autobuild-v4-0-test and v3-6-test.
Comment 8 Karolin Seeger 2012-11-13 10:42:41 UTC 
Pushed to v4-0-test.
Closing out bug report.

Thanks!