[Freeipa-devel] [PATCH] 0087 Fix winsync agreements setup
Rob Crittenden
rcritten at redhat.com
Mon Feb 28 16:18:45 UTC 2011
Simo Sorce wrote:
> On Mon, 28 Feb 2011 10:49:29 -0500
> Rob Crittenden<rcritten at redhat.com> wrote:
>
>> Simo Sorce wrote:
>>>
>>> Setting up a winsync agreement was broken.
>>>
>>> This patch fixes the code to allow setting up a winsync agreement
>>> that requires access to a non-IPA ldap server.
>>>
>>> Simo.
>>
>> This changes the side we initiate the replication startup on. I don't
>> know a ton about the internals of 389-ds replication but is this
>> necessary? It has been this way for years.
>
> Sorry, I don't see that.
> Where am I doing that ?
>
> Simo.
>
This is what I saw:
mod = [(ldap.MOD_ADD, 'nsds5BeginReplicaRefresh', 'start')]
- other_conn.modify_s(dn, mod)
+ conn.modify_s(dn, mod)
It looks like you renamed the variable from other_conn to to conn so
this change is ok.
rob
More information about the Freeipa-devel
mailing list