Re: [Freeipa-devel] [PATCH] 1031 run cleanallruv task

2012-07-09 Thread Martin Kosek
On 07/05/2012 08:39 PM, Rob Crittenden wrote: > Martin Kosek wrote: >> On 07/03/2012 04:41 PM, Rob Crittenden wrote: >>> Deleting a replica can leave a replication vector (RUV) on the other >>> servers. >>> This can confuse things if the replica is re-added, and it also causes the >>> server to ca

Re: [Freeipa-devel] [PATCH] 168 Password policy measurement units

2012-07-09 Thread Endi Sukma Dewata
On 7/9/2012 10:21 AM, Petr Vobornik wrote: Note: I think we should improve handling of measurement units in server plugins. Label and measurement unit should be separated and send in metadata. Client - Web UI or CLI would then decide when and where to display the units. I will create an enhanceme

Re: [Freeipa-devel] [PATCH] 169 Web UI: kerberos ticket policy measurement units

2012-07-09 Thread Endi Sukma Dewata
On 7/9/2012 10:32 AM, Petr Vobornik wrote: Added measurement units for kerberos ticket policy. https://fedorahosted.org/freeipa/ticket/2444 Note: patch depends on pvoborni-0168. ACK. -- Endi S. Dewata ___ Freeipa-devel mailing list Freeipa-devel@

Re: [Freeipa-devel] [PATCH] 167 Add and remove dns per-domain permission in Web UI

2012-07-09 Thread Endi Sukma Dewata
On 7/9/2012 6:55 AM, Petr Vobornik wrote: Patch functionality depends on not yet posted pviktori's patch which adds error_code (in case of command error) to batch response. Patch description: This patch adds support for new per-domain permissions to Web UI. User with assigned permission (throu

Re: [Freeipa-devel] [PATCH] 165 Display loginas information only after login

2012-07-09 Thread Endi Sukma Dewata
On 7/4/2012 8:08 AM, Petr Vobornik wrote: On 07/02/2012 05:49 PM, Endi Sukma Dewata wrote: ACK. Some more comments below. Feel free to fix before push or later separately. Implemented most of the issues, look bellow. Update patch attached. ACK. -- Endi S. Dewata __

[Freeipa-devel] [PATCH] 169 Web UI: kerberos ticket policy measurement units

2012-07-09 Thread Petr Vobornik
Added measurement units for kerberos ticket policy. https://fedorahosted.org/freeipa/ticket/2444 Note: patch depends on pvoborni-0168. -- Petr Vobornik From 7bfff70733b061ebd832c706d98cbb5f5521a4dc Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Mon, 9 Jul 2012 17:28:08 +0200 Subject: [PATC

[Freeipa-devel] [PATCH] 168 Password policy measurement units

2012-07-09 Thread Petr Vobornik
Note: I think we should improve handling of measurement units in server plugins. Label and measurement unit should be separated and send in metadata. Client - Web UI or CLI would then decide when and where to display the units. I will create an enhancement ticket for it. Patch description: Wh

[Freeipa-devel] [PATCH] 284 Do not change LDAPObject objectclass list

2012-07-09 Thread Martin Kosek
This one caused dnszone_find command in the Web UI to sometimes return an empty list. Pushed to master as a one-liner patch. Martin >From e1af6c5db4423babddcdd3c9f02fcde4fabf335b Mon Sep 17 00:00:00 2001 From: Martin Kosek Date: Mon, 9 Jul 2012 14:27:07 +0200 Subject: [PATCH] Do not change LDAPO

Re: [Freeipa-devel] [PATCH] Fix typo

2012-07-09 Thread Simo Sorce
On Mon, 2012-07-09 at 14:20 +0200, Sumit Bose wrote: > On Tue, Jun 26, 2012 at 10:29:00AM +0200, Sumit Bose wrote: > > Hi, > > > > this patch fixes a small typo and silences a compiler warning. I think > > it is right to use authdata instead of &authdata here, but I have to > > admit that I cannot

Re: [Freeipa-devel] [PATCH] Fix typo

2012-07-09 Thread Sumit Bose
On Tue, Jun 26, 2012 at 10:29:00AM +0200, Sumit Bose wrote: > Hi, > > this patch fixes a small typo and silences a compiler warning. I think > it is right to use authdata instead of &authdata here, but I have to > admit that I cannot say why we have not seen any issues before. > > bye, > Sumit I

[Freeipa-devel] [PATCH] 167 Add and remove dns per-domain permission in Web UI

2012-07-09 Thread Petr Vobornik
Patch functionality depends on not yet posted pviktori's patch which adds error_code (in case of command error) to batch response. Patch description: This patch adds support for new per-domain permissions to Web UI. User with assigned permission (through role,priviledge) can edit DNS zone. Th