On 03/19/2015 12:20 PM, David Kupka wrote:
On 03/13/2015 03:08 PM, Martin Basti wrote:
On 12/03/15 16:21, Rob Crittenden wrote:
Martin Basti wrote:
The patchset ensure, the upgrade order will respect ordering of entries
in *.update files.

Required for: https://fedorahosted.org/freeipa/ticket/4904

Patch 205 also fixes https://fedorahosted.org/freeipa/ticket/3560

Required patch mbasti-0203

Patches attached.



Just reading the patches, untested.

I think ordered should default to True in the update() method of
ldapupdater to keep in spirit with the design.

Otherwise LGTM that it implements what was designed.

rob


New patch that switch default value for ordered to True attached.



Looks and works as expected, ACK.

Pushed to master: a42fcfc18bb94fbf97ec310dbb920e045b0473a5

--
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to