Re: [Freeipa-devel] make-lint failures

2011-11-30 Thread Martin Kosek
On Wed, 2011-11-30 at 00:12 +0200, Alexander Bokovoy wrote: > On Tue, 29 Nov 2011, Rob Crittenden wrote: > > Seeing some new make-lint failures after the reworking of ipaldap > > function signatures; > > > > $ ./make-lint > > ipaserver/install/adtrustinstance.py:101: [E1120, > > ADTRUSTInstance.__

Re: [Freeipa-devel] make-lint failures

2011-11-30 Thread Alexander Bokovoy
On Wed, 30 Nov 2011, Martin Kosek wrote: > > The original LDAPObject.add_s() has modlist as non-optional argument: > > def add_s(self,dn,modlist): > > > > I don't think it is wise to break that API assumption. > > > > In all the cases above it should get .add_s(entry) replaced by > > .addEnt

Re: [Freeipa-devel] [PATCH] Add ipasam samba passdb backend

2011-11-30 Thread Sumit Bose
On Tue, Nov 29, 2011 at 11:25:41PM +0200, Alexander Bokovoy wrote: > On Tue, 29 Nov 2011, Sumit Bose wrote: > > @@ -199,10 +216,11 @@ class ADTRUSTInstance(service.Service): > > self.admin_conn.addEntry(entry) > > > > entry = ipaldap.Entry(self.smb_dom_dn) > > -e

Re: [Freeipa-devel] [PATCH] Add ipasam samba passdb backend

2011-11-30 Thread Martin Kosek
On Wed, 2011-11-30 at 10:11 +0100, Sumit Bose wrote: > On Tue, Nov 29, 2011 at 11:25:41PM +0200, Alexander Bokovoy wrote: > > On Tue, 29 Nov 2011, Sumit Bose wrote: > > > @@ -199,10 +216,11 @@ class ADTRUSTInstance(service.Service): > > > self.admin_conn.addEntry(entry) > > > > >

Re: [Freeipa-devel] make-lint failures

2011-11-30 Thread John Dennis
On 11/30/2011 04:06 AM, Alexander Bokovoy wrote: We need to review all .add_s() calls to ensure their correctness. .modify_s() are simpler because they always were used with two arguments (dn and list of modifications) but in case of .add_s() we were abusing it with Entry as a single argument and

[Freeipa-devel] Samba package name change samba-4.0 -> samba4

2011-11-30 Thread Sumit Bose
Hi, we recently changed the name of the samba packages in the ipa-devel respository. The packages are now called samba4-* and libsmbclient4-* instead of samba-4.0-* and libsmbclient-4.0-* . The name was changed because the samba packages will updated the samba4 packages which are currently availa

Re: [Freeipa-devel] Samba package name change samba-4.0 -> samba4

2011-11-30 Thread Stephen Gallagher
On Wed, 2011-11-30 at 14:40 +0100, Sumit Bose wrote: > Hi, > > we recently changed the name of the samba packages in the ipa-devel > respository. The packages are now called samba4-* and libsmbclient4-* > instead of samba-4.0-* and libsmbclient-4.0-* . > > The name was changed because the samba p

Re: [Freeipa-devel] Samba package name change samba-4.0 -> samba4

2011-11-30 Thread Sumit Bose
On Wed, Nov 30, 2011 at 08:46:04AM -0500, Stephen Gallagher wrote: > On Wed, 2011-11-30 at 14:40 +0100, Sumit Bose wrote: > > Hi, > > > > we recently changed the name of the samba packages in the ipa-devel > > respository. The packages are now called samba4-* and libsmbclient4-* > > instead of sam

Re: [Freeipa-devel] [PATCH] 55 Parse comma-separated lists of values in all parameter types

2011-11-30 Thread Martin Kosek
On Wed, 2011-11-30 at 15:19 +0100, Jan Cholasta wrote: > Dne 23.11.2011 16:46, Martin Kosek napsal(a): > > On Wed, 2011-11-23 at 11:14 +0100, Jan Cholasta wrote: > >> Dne 23.11.2011 09:52, Martin Kosek napsal(a): > > ... > >> > >> Well, the crash is obviously not related to the patch. Fedora 16 use

[Freeipa-devel] [Package: freeipa-2.1.3-7.fc16 Tag: f16-updates-candidate Status: complete Built by: abbra]

2011-11-30 Thread Alexander Bokovoy
Hi, and update for migrating from F15 to F16 (SysV to systemd) is ready for testing. I'll push it through bodhi soon. This package upon package update should notice inactive ipa.service and trigger upgrade procedure. If there is no FreeIPA install found, the upgrade will do nothing. If there

Re: [Freeipa-devel] [Package: freeipa-2.1.3-7.fc16 Tag: f16-updates-candidate Status: complete Built by: abbra]

2011-11-30 Thread Alexander Bokovoy
On Wed, 30 Nov 2011, Alexander Bokovoy wrote: > > Hi, > > and update for migrating from F15 to F16 (SysV to systemd) is ready > for testing. I'll push it through bodhi soon. > > This package upon package update should notice inactive ipa.service > and trigger upgrade procedure. If there is no

[Freeipa-devel] [PATCH 55/55] Restore default log level in server to INFO

2011-11-30 Thread John Dennis
The default log level for server messages captured by httpd's error_log historically was INFO. The log_manager patch had it set to ERROR, this patch resets it back to INFO. Although it would have been trival to set the default_level to INFO in IPALogManager.configure_from_env() that is not logical

Re: [Freeipa-devel] [PATCH 55/55] Restore default log level in server to INFO

2011-11-30 Thread John Dennis
Found & fixed minor issue with doc on default_level property, please use this version of the patch instead. -- John Dennis Looking to carve out IT costs? www.redhat.com/carveoutcosts/ >From 4203a6aa6e9768d576daf80fc686e7e96aa397f3 Mon Sep 17 00:00:00 2001 From: John Dennis Date: Wed, 30 Nov 2

[Freeipa-devel] Session design document

2011-11-30 Thread John Dennis
-- John Dennis Looking to carve out IT costs? www.redhat.com/carveoutcosts/ [ Note: This document is formatted in restructuredText and may be viewed in HTML at this link: http://jdennis.fedorapeople.org/doc/ipa_session.html ] Session Support for IPA John Dennis Goals = Provide per-user

Re: [Freeipa-devel] [PATCH 55/55] Restore default log level in server to INFO

2011-11-30 Thread Martin Kosek
On Wed, 2011-11-30 at 12:17 -0500, John Dennis wrote: > Found & fixed minor issue with doc on default_level property, please use > this version of the patch instead. > ACK. Pushed to master. Martin ___ Freeipa-devel mailing list Freeipa-devel@redhat.