[PATCH] Moving RID manager causes replication issues
Andrew Bartlett
abartlet at samba.org
Mon Oct 31 03:42:34 UTC 2016
On Fri, 2016-10-28 at 15:04 +1300, Clive Ferreira wrote:
> Hi,
>
> This patch relaxes the objectclass_attrs check which requires that
> all
> mandatory attributes are present. If the attribute is mandatory, but
> is
> not replicated (which is limited to only two attributes: rIDNextRid
> and
> rIDPreviousAllocationPool) then we should let the modification
> proceed.
>
> This change is required for moving the RID manager role and
> attempting
> to allocate a new RID pool from it. Although this check could be
> passed
> by actually setting the value consistently, it doesn't resolve
> existing
> databases and the edge case is only for two attributes (used for RID
> allocation). Furthermore, it removes the special case for requiring a
> dbcheck control to avoid errors.
>
> The new test passes against Windows, but errors out at the cleanup
> stage
> where we move back the RID master role to the original DC (which
> might
> be due to a time-delay in transfering roles).
>
> The second patch was missing a sign-off by Bob, which he can
> hopefully
> approve here.
>
> Any thoughts would be appreciated.
This looks great! Our users who move RID Manager roles around will
very much appreciate this fix.
Reviewed-by: Andrew Bartlett <abartlet at samba.org>
Thanks!
Andrew Bartlett
--
Andrew Bartlett
https://samba.org/~abartlet/
Authentication Developer, Samba Team https://samba.org
Samba Development and Support, Catalyst IT
https://catalyst.net.nz/services/samba
More information about the samba-technical
mailing list