[PATCH] Moving RID manager causes replication issues

Bob Campbell 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:
> 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.
>
>
> Cheers,
>
> Garming + Clive
>




More information about the samba-technical mailing list