[Freeipa-devel] [PATCH] 426, 430 Winsync re-initialize should not run memberOf fixup task

[Martin Kosek]

On 10/10/2013 05:45 PM, Jan Cholasta wrote:
> Hi,
> 
> On 9.10.2013 13:31, Martin Kosek wrote:
>> Change re-initialize command to consider memberOf fixup task only
>> for non-winsync replication agreements. This patch also includes
>> few fixes for DsInstance to properly set realm and fqdn properties
>> needed when connecting to LDAP.
>>
>> https://fedorahosted.org/freeipa/ticket/3854
>>
> 
> -        self.realm_name = realm_name
> +        self.realm_name = self.realm = realm_name
> 
> I think one realm property is enough. Can you please change dsinstance to use
> .realm instead of .realm_name?
> 
> Honza
> 

I did not get too much into refactoring, but this one is simple. Attaching
updating patches.

Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mkosek-426-2-winsync-re-initialize-should-not-run-memberof-fixup-.patch
Type: text/x-patch
Size: 1752 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20131011/6faa68e4/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mkosek-430-use-consistent-realm-name-in-cainstance-and-dsinstan.patch
Type: text/x-patch
Size: 7723 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20131011/6faa68e4/attachment-0001.bin>