Re: [Freeipa-devel] [PATCHES] 91-92 Add support for RFC 6594 SSHFP DNS records

2013-01-10 Thread Jan Cholasta
On 10.1.2013 05:56, Jan Cholasta wrote: Hi, Patch 91 removes module ipapython.compat. The code that uses it doesn't work with ancient Python versions anyway, so there's no need to keep it around. Patch 92 adds support for automatic generation of RFC 6594 SSHFP DNS records to ipa-client-install

[Freeipa-devel] [PATCH] 346 permission-find no longer crashes with --targetgroup

2013-01-10 Thread Martin Kosek
Target Group parameter was not processed correctly which caused permission-find to always crash when this search parameter was used. Fix the crash and create a unit test case to avoid future regression. https://fedorahosted.org/freeipa/ticket/3335 From 461a2096c3b18331cd5fabdc8c56525b66f54c7d Mon

[Freeipa-devel] CSV support in IPA administration tools - to be, or not to be?

2013-01-10 Thread Petr Spacek
Hello, is there any user of CSV support built-in to IPA administration tools (ipa command)? Do you consider it sane or even useful? Please reply. I wanted to add single TXT record with double quotation marks () inside the TXT data. I spent some time figuring out how it is supposed to work

[Freeipa-devel] [PATCH] 347 Avoid CRL migration error message

2013-01-10 Thread Martin Kosek
When CRL files are being migrated to a new directory, the upgrade log may contain an error message raised during MasterCRL.bin symlink migration. This is actually being caused by `chown' operation which tried to chown a symlinked file that was not migrated yet. Sort migrated files before the

Re: [Freeipa-devel] CSV support in IPA administration tools - to be, or not to be?

2013-01-10 Thread Jan Cholasta
Hi, On 10.1.2013 14:14, Petr Spacek wrote: Hello, is there any user of CSV support built-in to IPA administration tools (ipa command)? Do you consider it sane or even useful? Please reply. No and no. We should disable CSV in new installs. As you pointed out, shell does a better job than

[Freeipa-devel] [PATCH] 1080 fix migration of uniqueMember

2013-01-10 Thread Rob Crittenden
We were asserting that the uniqueMember contain DN objects but weren't actually making them DN objects. A sample entry looks like: dn: cn=Group1,ou=Groups,dc=example,dc=com gidNumber: 1001 objectClass: top objectClass: groupOfUniqueNames objectClass: posixGroup cn: Group1 uniqueMember:

Re: [Freeipa-devel] [PATCH] 346 permission-find no longer crashes with --targetgroup

2013-01-10 Thread Rob Crittenden
Martin Kosek wrote: Target Group parameter was not processed correctly which caused permission-find to always crash when this search parameter was used. Fix the crash and create a unit test case to avoid future regression. https://fedorahosted.org/freeipa/ticket/3335 ACK

Re: [Freeipa-devel] [PATCH] 347 Avoid CRL migration error message

2013-01-10 Thread Rob Crittenden
Martin Kosek wrote: When CRL files are being migrated to a new directory, the upgrade log may contain an error message raised during MasterCRL.bin symlink migration. This is actually being caused by `chown' operation which tried to chown a symlinked file that was not migrated yet. Sort migrated