Re: [Freeipa-devel] [PATCH] 047 Add an address for a nameserver when a new zone is created during install

2011-02-01 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/01/2011 04:15 AM, Rob Crittenden wrote: Jakub Hrozek wrote: On Mon, Jan 31, 2011 at 05:52:08PM -0500, Simo Sorce wrote: On Mon, 31 Jan 2011 22:44:43 +0100 Jakub Hrozekjhro...@redhat.com wrote: https://fedorahosted.org/freeipa/ticket/881

Re: [Freeipa-devel] [PATCH] 696 fix modifying delegation

2011-02-01 Thread Martin Kosek
On Mon, 2011-01-31 at 13:12 -0500, Rob Crittenden wrote: Modifying membergroup in a delegation was failing because of an inconsnstent use of the cli name and the attribute name and also because the aci plugin was not always treating memberof as a special kind of filter. ticket 869 rob

[Freeipa-devel] [PATCH] 021 Permission rename test failing

2011-02-01 Thread Martin Kosek
This patch fixes test for Permission plugin - mainly permission-mod part. Description field that the tests expected and which was removed in ticket 792 was removed from the tests. https://fedorahosted.org/freeipa/ticket/892 From 40fe8a1cf9151de25c655c4c985905d2895662bf Mon Sep 17 00:00:00 2001

Re: [Freeipa-devel] [PATCH] 0080 Sync in both direction before changing replication agreement in replica

2011-02-01 Thread Simo Sorce
On Mon, 31 Jan 2011 23:07:24 -0500 Rob Crittenden rcrit...@redhat.com wrote: Simo Sorce wrote: On Mon, 31 Jan 2011 18:38:47 -0500 Simo Sorcesso...@redhat.com wrote: See also ticket #887 Simo. With a patch file it works better I guess :-) Simo. I wasn't entirely

Re: [Freeipa-devel] [PATCH] Added undo for permission rights.

2011-02-01 Thread Adam Young
On 02/01/2011 12:02 AM, Endi Sukma Dewata wrote: On 1/31/2011 8:53 PM, Adam Young wrote: On 01/31/2011 08:48 PM, Endi Sukma Dewata wrote: https://fedorahosted.org/freeipa/ticket/884 NACK is_dirty is broken now on permissions. Could you describe which one is broken? I tried a number of

Re: [Freeipa-devel] [PATCH] 697 Add new schema to store information about permissions.

2011-02-01 Thread Martin Kosek
On Mon, 2011-01-31 at 22:18 -0500, Rob Crittenden wrote: Rob Crittenden wrote: There are some permissions we can't display because they are stored outside of the basedn (such as the replication permissions). We are adding a new attribute to store extra information to make this clear, in

Re: [Freeipa-devel] [PATCH] Make 'ipa help' localizable.

2011-02-01 Thread Pavel Zuna
On 02/01/2011 03:08 PM, John Dennis wrote: On 02/01/2011 08:16 AM, Pavel Zuna wrote: For a long time, I was trying to find a way to localize python docstrings, that we use to generate the built-in documentation system. Unfortunately, python docstrings aren't meant to be localized and therefore

Re: [Freeipa-devel] [PATCH] 697 Add new schema to store information about permissions.

2011-02-01 Thread Martin Kosek
On Tue, 2011-02-01 at 09:07 -0500, Rob Crittenden wrote: Martin Kosek wrote: 2) In delegation.ldif: ipapermission object class is missing for removeentitlements and modifyentitlements (it has been added for addentitlements though) This was on purpose, I should have been clearer. Patch

Re: [Freeipa-devel] [PATCH] 664 entitlement support

2011-02-01 Thread Rob Crittenden
Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/31/2011 04:29 PM, Rob Crittenden wrote: Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/05/2011 04:38 PM, Rob Crittenden wrote: This patch adds a plugin and tools for managing entitlements for

Re: [Freeipa-devel] [PATCH] 0080 Sync in both direction before changing replication agreement in replica

2011-02-01 Thread Rich Megginson
On 01/31/2011 09:07 PM, Rob Crittenden wrote: Simo Sorce wrote: On Mon, 31 Jan 2011 18:38:47 -0500 Simo Sorcesso...@redhat.com wrote: See also ticket #887 Simo. With a patch file it works better I guess :-) Simo. I wasn't entirely sure how to test this so I use ipa-replica-manage

[Freeipa-devel] Changelog error after re-initialize. [WAS [PATCH] 0080 ..]

2011-02-01 Thread Simo Sorce
On Tue, 01 Feb 2011 09:07:58 -0700 Rich Megginson rmegg...@redhat.com wrote: On 01/31/2011 09:07 PM, Rob Crittenden wrote: Simo Sorce wrote: On Mon, 31 Jan 2011 18:38:47 -0500 Simo Sorcesso...@redhat.com wrote: See also ticket #887 Simo. With a patch file it works better I

Re: [Freeipa-devel] Changelog error after re-initialize. [WAS [PATCH] 0080 ..]

2011-02-01 Thread Rich Megginson
On 02/01/2011 09:14 AM, Simo Sorce wrote: On Tue, 01 Feb 2011 09:07:58 -0700 Rich Megginsonrmegg...@redhat.com wrote: On 01/31/2011 09:07 PM, Rob Crittenden wrote: Simo Sorce wrote: On Mon, 31 Jan 2011 18:38:47 -0500 Simo Sorcesso...@redhat.com wrote: See also ticket #887 Simo. With

Re: [Freeipa-devel] [PATCH] 697 Add new schema to store information about permissions.

2011-02-01 Thread Rob Crittenden
Martin Kosek wrote: On Tue, 2011-02-01 at 09:07 -0500, Rob Crittenden wrote: Martin Kosek wrote: 2) In delegation.ldif: ipapermission object class is missing for removeentitlements and modifyentitlements (it has been added for addentitlements though) This was on purpose, I should have been

Re: [Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-02-01 Thread Rob Crittenden
Jan Zelený wrote: Rob Crittendenrcrit...@redhat.com wrote: Jan Zelený wrote: Since some LDAP attributes have their cli_name value defined, so they can be more user friendly, it can be difficult for user to find out which attributes do the parameteres given to CLI really represent. This patch

Re: [Freeipa-devel] [PATCH] 696 fix modifying delegation

2011-02-01 Thread Rob Crittenden
Martin Kosek wrote: On Mon, 2011-01-31 at 13:12 -0500, Rob Crittenden wrote: Modifying membergroup in a delegation was failing because of an inconsnstent use of the cli name and the attribute name and also because the aci plugin was not always treating memberof as a special kind of filter.

Re: [Freeipa-devel] [PATCH] Changed dns permission types

2011-02-01 Thread Rob Crittenden
Jan Zelený wrote: Jan Zelenýjzel...@redhat.com wrote: Rob Crittendenrcrit...@redhat.com wrote: Jan Zelený wrote: Rob Crittendenrcrit...@redhat.com wrote: Jan Zelený wrote: Recent change of DNS module to version caused that dns object type was replaced by dnszone and dnsrecord. This

Re: [Freeipa-devel] [PATCH] 664 entitlement support

2011-02-01 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/01/2011 04:15 PM, Rob Crittenden wrote: Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/31/2011 04:29 PM, Rob Crittenden wrote: Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/05/2011

[Freeipa-devel] [PATCH] 048 IPv6 enhancements

2011-02-01 Thread Jakub Hrozek
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Attached is a patch that fixes the remaining IPv6 problems. Many were testable on a v4 installation, like the host plugin changes. I only verified the v6 reverse zone creation in bindinstance with ldapsearch so far.

Re: [Freeipa-devel] [PATCH] 664 entitlement support

2011-02-01 Thread Rob Crittenden
Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/01/2011 04:15 PM, Rob Crittenden wrote: Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/31/2011 04:29 PM, Rob Crittenden wrote: Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1

[Freeipa-devel] [PATCH] admiyo-0185-undo-entity-widget

2011-02-01 Thread Adam Young
From fc3537dbeb41dedbcaa3a9a26663e3ea26259c81 Mon Sep 17 00:00:00 2001 From: Adam Young ayo...@redhat.com Date: Tue, 1 Feb 2011 14:55:55 -0500 Subject: [PATCH] undo entity widget adds line level undo for the entity-select-widget https://fedorahosted.org/freeipa/ticket/886 ---

Re: [Freeipa-devel] [PATCH] 021 Permission rename test failing

2011-02-01 Thread Rob Crittenden
Martin Kosek wrote: This patch fixes test for Permission plugin - mainly permission-mod part. Description field that the tests expected and which was removed in ticket 792 was removed from the tests. https://fedorahosted.org/freeipa/ticket/892 ack, pushed to master

Re: [Freeipa-devel] [PATCH] 697 Add new schema to store information about permissions.

2011-02-01 Thread Rob Crittenden
Rob Crittenden wrote: Martin Kosek wrote: On Tue, 2011-02-01 at 09:07 -0500, Rob Crittenden wrote: Martin Kosek wrote: 2) In delegation.ldif: ipapermission object class is missing for removeentitlements and modifyentitlements (it has been added for addentitlements though) This was on

Re: [Freeipa-devel] [PATCH] admiyo-0184-use-entity-select-widget-for-permissions

2011-02-01 Thread Endi Sukma Dewata
On 2/1/2011 12:08 PM, Adam Young wrote: ACK and pushed to master. -- Endi S. Dewata ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] admiyo-0185-undo-entity-widget

2011-02-01 Thread Adam Young
On 02/01/2011 02:57 PM, Adam Young wrote: ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel NOw shows undo link if the filter changes From 419b755871b1de7256f7ca540bf6058ef5075f8f Mon Sep

Re: [Freeipa-devel] [PATCH] 697 Add new schema to store information about permissions.

2011-02-01 Thread Martin Kosek
On Tue, 2011-02-01 at 14:57 -0500, Rob Crittenden wrote: Rob Crittenden wrote: Martin Kosek wrote: On Tue, 2011-02-01 at 09:07 -0500, Rob Crittenden wrote: Martin Kosek wrote: 2) In delegation.ldif: ipapermission object class is missing for removeentitlements and modifyentitlements (it

Re: [Freeipa-devel] [PATCH] 697 Add new schema to store information about permissions.

2011-02-01 Thread Rob Crittenden
Martin Kosek wrote: On Tue, 2011-02-01 at 14:57 -0500, Rob Crittenden wrote: Rob Crittenden wrote: Martin Kosek wrote: On Tue, 2011-02-01 at 09:07 -0500, Rob Crittenden wrote: Martin Kosek wrote: 2) In delegation.ldif: ipapermission object class is missing for removeentitlements and

Re: [Freeipa-devel] [PATCH] 664 entitlement support

2011-02-01 Thread Jakub Hrozek
On 02/01/2011 08:25 PM, Rob Crittenden wrote: Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 02/01/2011 04:15 PM, Rob Crittenden wrote: Jakub Hrozek wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 01/31/2011 04:29 PM, Rob Crittenden wrote: Jakub Hrozek wrote:

Re: [Freeipa-devel] [PATCH] admiyo-0185-undo-entity-widget

2011-02-01 Thread Adam Young
On 02/01/2011 03:49 PM, Adam Young wrote: On 02/01/2011 02:57 PM, Adam Young wrote: ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel NOw shows undo link if the filter changes

Re: [Freeipa-devel] [PATCH] admiyo-0185-undo-entity-widget

2011-02-01 Thread Endi Sukma Dewata
On 2/1/2011 3:25 PM, Adam Young wrote: NOw shows undo link if the filter changes edewata noticed that the the search wasn't working under the following sequence: double click to highlight all, then backspace to remove it: the undo shows up, but the list is not updated That was due to

Re: [Freeipa-devel] [PATCH] Changed dns permission types

2011-02-01 Thread Jan Zelený
Rob Crittenden rcrit...@redhat.com wrote: Jan Zelený wrote: Jan Zelenýjzel...@redhat.com wrote: Rob Crittendenrcrit...@redhat.com wrote: Jan Zelený wrote: Rob Crittendenrcrit...@redhat.com wrote: Jan Zelený wrote: Recent change of DNS module to version caused that dns object type

Re: [Freeipa-devel] [PATCH] Provide a way to display CLI-LDAP attribute relation

2011-02-01 Thread Jan Zelený
Ok, I'm sending updated patch in attachment Should I change it in class help then? That's where I copied this from. I think so. Ok, I'll send another patch, so me don't mix it together with this patch. I'll do a review of the code in cli.py, maybe the same issue is elsewhere as well.

[Freeipa-devel] [PATCH] Fixed type of argument in class help

2011-02-01 Thread Jan Zelený
At Rob's suggestion I changed the argument type in class help, this is only oneliner, I think it can be pushed directly. Jan From f7ca20a79bf06832c20262582f5ca83ec48ff056 Mon Sep 17 00:00:00 2001 From: Jan Zeleny jzel...@redhat.com Date: Wed, 2 Feb 2011 02:49:57 -0500 Subject: [PATCH] Fixed type