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,

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 .addEntry(entry).

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) -

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

[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

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

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 samba-4.0-*

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 uses a newer

[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

[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

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 jden...@redhat.com Looking to carve out IT costs? www.redhat.com/carveoutcosts/ From 4203a6aa6e9768d576daf80fc686e7e96aa397f3 Mon Sep 17 00:00:00 2001 From: John Dennis

[Freeipa-devel] Session design document

2011-11-30 Thread John Dennis
-- John Dennis jden...@redhat.com 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

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.com