[Freeipa-devel] [PATCH] 117 Added attrs field to permission for target=subtree

2012-04-04 Thread Petr Vobornik
Permission form was missing attrs field for target=subtree. All other target types have it. It uses multivalued text widget, same as filter, because we can't predict the target type. https://fedorahosted.org/freeipa/ticket/2592 -- Petr Vobornik From 9da9d186b2abbd633dc9dd36a1ba653a8d6f9e0f

Re: [Freeipa-devel] [PATCH] 0015 Don't try to remove auxiliary nodes from internal RBT

2012-04-04 Thread Adam Tkac
On Tue, Apr 03, 2012 at 03:06:31PM +0200, Petr Spacek wrote: Hello, this patch optimizes code for removing deleted zones from BIND instance little bit. In some cases there are auxiliary zones (= not really served zones) in internal Red-Black tree. Current code tries to remove these

Re: [Freeipa-devel] [PATCHES] 0025-26 Test improvements

2012-04-04 Thread Petr Viktorin
On 04/02/2012 05:05 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 03/26/2012 10:44 PM, Rob Crittenden wrote: Petr Viktorin wrote: Patch 25 fixes errors I found by running pylint on the testsuite. They were in code that was unused, either by error or because it only runs on errors. Patch

[Freeipa-devel] [PATCH] 118-119 DNS forward policy: checkboxes changed to radio buttons

2012-04-04 Thread Petr Vobornik
DNS forward policy fields were using mutually exclusive checkboxes. Such behavior is unusual for users. Checkboxes were changed to radios with new option 'none/default' to set empty value ''. https://fedorahosted.org/freeipa/ticket/2599 Second patch removes mutex option from checkboxes. Not

Re: [Freeipa-devel] [PATCH] 0016 Fixes for{add, set, del}attr with managed attributes

2012-04-04 Thread Petr Viktorin
On 03/30/2012 03:22 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 03/16/2012 08:01 PM, Petr Viktorin wrote: On 03/16/2012 06:35 PM, Petr Viktorin wrote: On 03/16/2012 06:33 PM, Rob Crittenden wrote: Rob Crittenden wrote: Petr Viktorin wrote: On 03/15/2012 09:24 PM, Rob Crittenden

Re: [Freeipa-devel] More types of replica in FreeIPA

2012-04-04 Thread Dmitri Pal
On 04/03/2012 12:45 PM, Ondrej Hamada wrote: On 03/13/2012 01:13 AM, Dmitri Pal wrote: On 03/12/2012 06:10 PM, Simo Sorce wrote: On Mon, 2012-03-12 at 17:40 -0400, Dmitri Pal wrote: On 03/12/2012 04:16 PM, Simo Sorce wrote: On Mon, 2012-03-12 at 20:38 +0100, Ondrej Hamada wrote: USER'S

Re: [Freeipa-devel] [PATCH 69] Use indexed format specifiers in i18n strings

2012-04-04 Thread Petr Viktorin
On 04/02/2012 03:15 PM, Rob Crittenden wrote: John Dennis wrote: Translators need to reorder messages to suit the needs of the target language. The conventional positional format specifiers (e.g. %s %d) do not permit reordering because their order is tied to the ordering of the arguments to the

Re: [Freeipa-devel] More types of replica in FreeIPA

2012-04-04 Thread Simo Sorce
On Tue, 2012-04-03 at 18:45 +0200, Ondrej Hamada wrote: On 03/13/2012 01:13 AM, Dmitri Pal wrote: On 03/12/2012 06:10 PM, Simo Sorce wrote: On Mon, 2012-03-12 at 17:40 -0400, Dmitri Pal wrote: On 03/12/2012 04:16 PM, Simo Sorce wrote: On Mon, 2012-03-12 at 20:38 +0100, Ondrej Hamada

[Freeipa-devel] [PATCH] 247 Fix installation when server hostname is not in a default domain

2012-04-04 Thread Martin Kosek
When IPA server is configured with DNS and its hostname is not located in a default domain, SRV records are not valid. Additionally, httpd does not serve XMLRPC interface because it IPA server domain-realm mapping is missing in krb5.conf. All CLI commands were then failing. This patch amends this

Re: [Freeipa-devel] [PATCH] 0016 Fixes for{add, set, del}attr with managed attributes

2012-04-04 Thread Rob Crittenden
Petr Viktorin wrote: On 03/30/2012 03:22 PM, Rob Crittenden wrote: Petr Viktorin wrote: On 03/16/2012 08:01 PM, Petr Viktorin wrote: On 03/16/2012 06:35 PM, Petr Viktorin wrote: On 03/16/2012 06:33 PM, Rob Crittenden wrote: Rob Crittenden wrote: Petr Viktorin wrote: On 03/15/2012 09:24

[Freeipa-devel] [PATCH] 0033 Pass make-test arguments through to Nose + Test coverage

2012-04-04 Thread Petr Viktorin
Currently, our test script forwards a select few command line arguments to nosetests. This patch removes the filtering, passing all arguments through. This allows things like disabling output redirection (--nocapture), dropping into a debugger (--pdb, --pdb-failures), coverage reporting

Re: [Freeipa-devel] [PATCH] 246 Configure SELinux for httpd during upgrades

2012-04-04 Thread Rob Crittenden
Martin Kosek wrote: SELinux configuration for httpd instance was set for new installations only. Upgraded IPA servers (namely 2.1.x - 2.2.x upgrade) missed the configuration. This lead to AVCs when httpd tries to contact ipa_memcached and user not being able to log in. This patch updates

Re: [Freeipa-devel] More types of replica in FreeIPA

2012-04-04 Thread Ondrej Hamada
On 04/04/2012 03:02 PM, Simo Sorce wrote: On Tue, 2012-04-03 at 18:45 +0200, Ondrej Hamada wrote: On 03/13/2012 01:13 AM, Dmitri Pal wrote: On 03/12/2012 06:10 PM, Simo Sorce wrote: On Mon, 2012-03-12 at 17:40 -0400, Dmitri Pal wrote: On 03/12/2012 04:16 PM, Simo Sorce wrote: On Mon,

Re: [Freeipa-devel] [PATCH] 73 Check whether the default user group is POSIX when adding new user with --noprivate

2012-04-04 Thread Jan Cholasta
On 3.4.2012 13:04, Martin Kosek wrote: On Tue, 2012-04-03 at 13:02 +0200, Martin Kosek wrote: On Tue, 2012-04-03 at 11:58 +0200, Jan Cholasta wrote: https://fedorahosted.org/freeipa/ticket/2572 Honza NACK. This creates a regression: # ipa group-show foogroup Group name: foogroup

[Freeipa-devel] [PATCH] 1002 make revocation_reason required

2012-04-04 Thread Rob Crittenden
When revoking a certificate passing in an empty revocation reason caused an Internal Error. It already sets a default so making it required prevents empty values and it still operates the same way. rob From 669879148bd0c98911327661007e18906dd5499d Mon Sep 17 00:00:00 2001 From: Rob Crittenden

[Freeipa-devel] [PATCH 71] improve handling of ds instances during uninstall

2012-04-04 Thread John Dennis
-- John Dennis jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 032ebced279d32bb3e6d51ef23fecaca36774ec7 Mon Sep 17 00:00:00 2001 From: John Dennis jden...@redhat.com Date: Wed, 4 Apr 2012 15:19:29 -0400 Subject: [PATCH 71] improve handling of ds instances

Re: [Freeipa-devel] [PATCH 71] improve handling of ds instances during uninstall

2012-04-04 Thread Alexander Bokovoy
On Wed, 04 Apr 2012, John Dennis wrote: Ticket #2502 * remove the running flag from backup_state in cainstance.py and dsinstance.py because it does not provide the correct information. In cainstance the running flag was never referenced because restarting dirsrv instances occurs later in