Bug 9386 - Winbind does't failover to next DC if netlogon is not available.
Winbind does't failover to next DC if netlogon is not available.
Status: RESOLVED FIXED
Product: Samba 4.0
Classification: Unclassified
Component: Winbind
4.0.0rc4
All All
: P5 normal
: ---
Assigned To: Karolin Seeger
Samba QA Contact
:
Depends on:
Blocks: 9176
  Show dependency treegraph
 
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

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!