[PATCH] Moving RID manager causes replication issues
bobcampbell at catalyst.net.nz
Fri Oct 28 04:13:22 UTC 2016
Signed-off-by: Bob Campbell <bobcampbell at catalyst.net.nz>
On 28/10/2016 3:04 p.m., Clive Ferreira wrote:
> 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.
> Garming + Clive
More information about the samba-technical