Re: [Freeipa-devel] [PATCH 53/53] ticket 2022 - modify codebase to utilize IPALogManager, obsoletes logging

2011-11-18 Thread Rob Crittenden
Martin Kosek wrote: On Thu, 2011-11-17 at 11:17 -0500, John Dennis wrote: On 11/16/2011 07:35 AM, Martin Kosek wrote: On Tue, 2011-11-15 at 14:41 -0500, John Dennis wrote: -- John Dennis Looking to carve out IT costs? www.redhat.com/carveoutcosts/ Hi John, thanks for your patch, that was

Re: [Freeipa-devel] Putting the A in IPA

2011-11-18 Thread John Dennis
On 11/18/2011 04:02 PM, Adam Young wrote: So the crazy systemd folks are at it again: https://docs.google.com/document/pub?id=1IC9yOXj7j6cdLLxWEBAGRL6wl97tFxgjLUEHIX3MSTs&pli=1 Yup, we've been tracking it. sgrubb is supposed to do a tech talk on "standardized logging & audit" soon claiming th

Re: [Freeipa-devel] Putting the A in IPA

2011-11-18 Thread Dmitri Pal
On 11/18/2011 04:02 PM, Adam Young wrote: > So the crazy systemd folks are at it again: > > https://docs.google.com/document/pub?id=1IC9yOXj7j6cdLLxWEBAGRL6wl97tFxgjLUEHIX3MSTs&pli=1 > > > > This is a re implementation of logging much the way the systmd was a > reimplementation of init. > > Assume

[Freeipa-devel] Putting the A in IPA

2011-11-18 Thread Adam Young
So the crazy systemd folks are at it again: https://docs.google.com/document/pub?id=1IC9yOXj7j6cdLLxWEBAGRL6wl97tFxgjLUEHIX3MSTs&pli=1 This is a re implementation of logging much the way the systmd was a reimplementation of init. Assume that it is going to get implemented, does this give u

Re: [Freeipa-devel] [PATCHES] #1950 Implement CLDAP server for AD trust introperability

2011-11-18 Thread Sumit Bose
On Fri, Nov 18, 2011 at 11:50:47AM -0500, Simo Sorce wrote: > On Fri, 2011-11-18 at 16:07 +0100, Sumit Bose wrote: > > On Thu, Nov 17, 2011 at 05:00:51PM -0500, Simo Sorce wrote: > > > Attached find a series of patches that implement a CLDAP server as a > > > dirsrv plugin. > > > > > > The server

Re: [Freeipa-devel] [PATCH 53/53] ticket 2022 - modify codebase to utilize IPALogManager, obsoletes logging

2011-11-18 Thread John Dennis
On 11/18/2011 03:04 AM, Martin Kosek wrote: Ok, these are all valid arguments. I know that abusing "logging" is not right, I was just afraid that diverging master branch from our 2.x branch that much would do more harm (and extra effort whenever a patch from master is backported to 2.x branch) th

Re: [Freeipa-devel] [PATCHES] #1950 Implement CLDAP server for AD trust introperability

2011-11-18 Thread Simo Sorce
On Fri, 2011-11-18 at 16:07 +0100, Sumit Bose wrote: > On Thu, Nov 17, 2011 at 05:00:51PM -0500, Simo Sorce wrote: > > Attached find a series of patches that implement a CLDAP server as a > > dirsrv plugin. > > > > The server right now responds only to a very limited class of requests, > > as obse

Re: [Freeipa-devel] [PATCH] 314 Fixed entity definition in test cases.

2011-11-18 Thread Endi Sukma Dewata
On 11/18/2011 9:01 AM, Petr Vobornik wrote: 1) In the tests is a call of entity_init method which isn't defined anywhere. It leads to JS error and test fail. Fixed. It was defined in another patch: ACK Pushed to master. -- Endi S. Dewata ___ Fr

Re: [Freeipa-devel] [PATCH] 315 Added commands into metadata.

2011-11-18 Thread Endi Sukma Dewata
On 11/18/2011 4:58 AM, Petr Vobornik wrote: On 11/17/2011 08:27 PM, Endi Sukma Dewata wrote: The json_metadata command has been modified to return the command metadata. The API.txt has been updated as well. Needed by ticket #388 ACK from UI perspective. Ipalib changes seems fine to me, but I'

Re: [Freeipa-devel] [PATCHES] #1950 Implement CLDAP server for AD trust introperability

2011-11-18 Thread Sumit Bose
On Thu, Nov 17, 2011 at 05:00:51PM -0500, Simo Sorce wrote: > Attached find a series of patches that implement a CLDAP server as a > dirsrv plugin. > > The server right now responds only to a very limited class of requests, > as observed on the wire. But it can be easily expanded to respond to > a

Re: [Freeipa-devel] [PATCH] 314 Fixed entity definition in test cases.

2011-11-18 Thread Petr Vobornik
On 11/18/2011 03:57 PM, Endi Sukma Dewata wrote: On 11/18/2011 4:29 AM, Petr Vobornik wrote: 1) In the tests is a call of entity_init method which isn't defined anywhere. It leads to JS error and test fail. Fixed. It was defined in another patch: ACK -- Petr Vobornik __

Re: [Freeipa-devel] [PATCH] 314 Fixed entity definition in test cases.

2011-11-18 Thread Endi Sukma Dewata
On 11/18/2011 4:29 AM, Petr Vobornik wrote: 1) In the tests is a call of entity_init method which isn't defined anywhere. It leads to JS error and test fail. Fixed. It was defined in another patch: diff --git a/install/ui/entity.js b/install/ui/entity.js index f7bf992..4e5c724 100644 --- a/ins

Re: [Freeipa-devel] [PATCH] Skip invalid records

2011-11-18 Thread Adam Tkac
On 11/18/2011 02:18 PM, Jiri Kuncar wrote: Bind-dyndb-ldap: Don't fail to load zone when it contains invalid records. Your patch seems fine for me, thanks. Please check my comments inside the patch. Regards, Adam Needed by ticket #36 https://fedorahosted.org/bind-dyndb-ldap/ticket/36 0001-

[Freeipa-devel] [PATCH] 4 ipa-client-install fails when not run as root

2011-11-18 Thread Ondrej Hamada
https://fedorahosted.org/freeipa/ticket/2123 ipa-client-install was failing and returning traceback when wasn't run by root. It was caused by logging initialization that was taking part before the root privileges check. To correct it, the privileges check was moved before the logging initializati

[Freeipa-devel] [PATCH] Skip invalid records

2011-11-18 Thread Jiri Kuncar
Bind-dyndb-ldap: Don't fail to load zone when it contains invalid records. Needed by ticket #36 https://fedorahosted.org/bind-dyndb-ldap/ticket/36From 37a8430b4ad33fc727a6b6a19998710b37da Mon Sep 17 00:00:00 2001 From: Jiri Kuncar Date: Fri, 18 Nov 2011 08:04:53 -0500 Subject: [PATCH] Skip ba

Re: [Freeipa-devel] [PATCH] 315 Added commands into metadata.

2011-11-18 Thread Alexander Bokovoy
On Thu, 17 Nov 2011, Endi Sukma Dewata wrote: > The json_metadata command has been modified to return the command > metadata. The API.txt has been updated as well. > > Needed by ticket #388 ... > --- a/ipalib/plugins/internal.py > +++ b/ipalib/plugins/internal.py > @@ -49,15 +52,25 @@ class json_m

Re: [Freeipa-devel] [PATCH] 315 Added commands into metadata.

2011-11-18 Thread Petr Vobornik
On 11/17/2011 08:27 PM, Endi Sukma Dewata wrote: The json_metadata command has been modified to return the command metadata. The API.txt has been updated as well. Needed by ticket #388 ACK from UI perspective. Ipalib changes seems fine to me, but I'm not 100% sure. Note: this patch depends

Re: [Freeipa-devel] [PATCH] 314 Fixed entity definition in test cases.

2011-11-18 Thread Petr Vobornik
On 11/17/2011 08:25 PM, Endi Sukma Dewata wrote: The test cases have been updated to use the new extensible mechanism for defining and registering entities. Ticket #2043 1) In the tests is a call of entity_init method which isn't defined anywhere. It leads to JS error and test fail. -- Pet

Re: [Freeipa-devel] [PATCH 53/53] ticket 2022 - modify codebase to utilize IPALogManager, obsoletes logging

2011-11-18 Thread Martin Kosek
On Thu, 2011-11-17 at 11:17 -0500, John Dennis wrote: > On 11/16/2011 07:35 AM, Martin Kosek wrote: > > On Tue, 2011-11-15 at 14:41 -0500, John Dennis wrote: > >> > >> -- > >> John Dennis > >> > >> Looking to carve out IT costs? > >> www.redhat.com/carveoutcosts/ > > > > Hi John, > > > > thanks for

Re: [Freeipa-devel] [PATCH 53/53] ticket 2022 - modify codebase to utilize IPALogManager, obsoletes logging

2011-11-18 Thread Martin Kosek
On Thu, 2011-11-17 at 14:00 -0500, John Dennis wrote: > On 11/17/2011 11:17 AM, John Dennis wrote: > > On 11/16/2011 07:35 AM, Martin Kosek wrote: > >> 1) Patch 53 will need rebasing > > > > O.K. will rebase. > > O.K. I'm confused. This is a 3.0 feature and the patch is against > origin/master wh