[Freeipa-devel] [PATCHES] 59-65 SSH public key management

Jan Cholasta jcholast at redhat.com
Tue Feb 7 13:52:20 UTC 2012


Dne 7.2.2012 00:04, Rob Crittenden napsal(a):
> Jan Cholasta wrote:
>> Updated & rebased the patches.
>>
>> I have also attached a patch that Rob made:
>>
>>
>> [PATCH] Don't use sets when calculating the modlist so order is
>> preserved.
>>
>> This is for the LDAP updater in particular. When adding new schema order
>> can be important when one objectclass depends on another via SUP.
>>
>>
>> Without this patch updates won't work.
>>
>> Dne 25.1.2012 17:19, Rob Crittenden napsal(a):
>>>>>
>>>>> Patch 61 you can drop the md5 and sha1 imports and import them from
>>>>> ipalib.compat instead.
>>>>
>>>> Is this OK in ipapython?
>>>
>>> It should be, ipa-python and ipalib should be packaged together so I
>>> think it is safe.
>>
>> Turns out this change breaks ipa-upgradeconfig.
>
> The problem is the package initializer, ipalib/__init__.py. Just
> importing ipalib.compat also imports a bunch of other stuff.
>
> I wonder if moving the decode function to ipalib.util would resolve
> this. I'm not a big fan of duplicating that import code.

I have created a patch that moves compat.py to ipapython - IMO ipapython 
is the right place for such module.

>
> Otherwise this works ok. I tested in the ipa-2-2 branch so had to do a
> couple of merges, not sure if this applies cleanly to current master or
> not.
>
> rob

I have rebased the patches on top of current master and added patch 68, 
which moves the compat module. See attachments.

Honza

-- 
Jan Cholasta
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-59.3-ssh-ldap-schema.patch
Type: text/x-patch
Size: 5497 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120207/52831c03/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-60.3-ssh-ldap-aci.patch
Type: text/x-patch
Size: 8791 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120207/52831c03/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-61.3-ssh-host-user-plugins.patch
Type: text/x-patch
Size: 24540 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120207/52831c03/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-62.3-ipa-client-install-api.patch
Type: text/x-patch
Size: 4114 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120207/52831c03/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-63.3-ipa-client-install-nsupdate.patch
Type: text/x-patch
Size: 2215 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120207/52831c03/attachment-0004.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-64.3-ssh-install-update-keys.patch
Type: text/x-patch
Size: 9726 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120207/52831c03/attachment-0005.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-65.4-ssh-install-config-sshd.patch
Type: text/x-patch
Size: 11279 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120207/52831c03/attachment-0006.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-66.2-parameter-bytes-base64.patch
Type: text/x-patch
Size: 3732 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120207/52831c03/attachment-0007.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-67.2-ssh-platform-service.patch
Type: text/x-patch
Size: 4049 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120207/52831c03/attachment-0008.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jcholast-68-move-compat-to-ipapython.patch
Type: text/x-patch
Size: 8209 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120207/52831c03/attachment-0009.bin>


More information about the Freeipa-devel mailing list