[Freeipa-devel] [PATCHES 0318 - 0320, 0323] installer: allow to modify dse.ldif during installation
Martin Basti
mbasti at redhat.com
Tue Oct 13 16:38:07 UTC 2015
On 12.10.2015 12:30, Martin Babinsky wrote:
> On 10/08/2015 05:58 PM, Martin Basti wrote:
>> The attached patches fix following tickets:
>> https://fedorahosted.org/freeipa/ticket/4949
>> https://fedorahosted.org/freeipa/ticket/4048
>> https://fedorahosted.org/freeipa/ticket/1930
>>
>> With these patches, an administrator can specify LDIF file that contains
>> modifications to be applied to dse.ldif right after creation of DS
>> instance.
>>
>>
> Hi,
>
> Functionally the paches work as expected. However I have following
> nitpicks:
>
> in patch 318:
>
> 1.) there is a typo in ModifyLDIF class docstring:
>
> + Operations keep the order in whihc were specified per DN.
>
> in patch 320:
>
> 1.) you should use 'os.path.join' to construct FS paths:
>
>
> - dse_filename = '%s/%s' % (
> + dse_filename = os.path.join(
> paths.ETC_DIRSRV_SLAPD_INSTANCE_TEMPLATE % self.serverid,
> - 'dse.ldif',
> + 'dse.ldif'
> )
>
> 2.) IIUC the 'config_ldif_file' knob in BaseServer holds the path to
> LDIF containing the mod operations to dse.ldif. However, the knob name
> sounds like the option accepts the path of dse.ldif itself. I propose
> to rename the knob to something more in-line with the supposed
> function, like 'dse_mods_file'.
>
Updated patches + CI test attached
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0318.2-Make-offline-LDIF-modify-more-robust.patch
Type: text/x-patch
Size: 10755 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151013/3a4f292f/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0319.2-Add-method-to-read-changes-from-LDIF.patch
Type: text/x-patch
Size: 2957 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151013/3a4f292f/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0320.2-Add-option-to-specify-LDIF-file-that-contains-DS-con.patch
Type: text/x-patch
Size: 9376 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151013/3a4f292f/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0323.2-CI-installation-with-customized-DS-config.patch
Type: text/x-patch
Size: 5389 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151013/3a4f292f/attachment-0003.bin>
More information about the Freeipa-devel
mailing list