Re: [SSSD] [PATCH] AD: Add initial gpo-smb implementation.

2014-06-11 Thread Sumit Bose
On Fri, May 30, 2014 at 12:35:32PM -0400, Yassir Elley wrote: > > > - Original Message - > > On (29/05/14 09:06), Yassir Elley wrote: > > >- Original Message - > > >> On Thu, May 29, 2014 at 04:09:21AM -0400, Yassir Elley wrote: > > >> > Also note that some (but not all) of the ch

Re: [SSSD] [PATCH] IPA: Rename label in users_get_send/groups_get_send

2014-06-11 Thread Simo Sorce
On Wed, 2014-06-11 at 14:58 +0200, Pavel Reichl wrote: > Hello, > > please see attached simple patch. Personally I do not like two labels, the idea of having 'done/fail' is to have one place that terminates and cleans up in all cases. Simo. -- Simo Sorce * Red Hat, Inc * New York

[SSSD] [PATCH] Fix return value of attr_name_val_split()

2014-06-11 Thread Sumit Bose
Hi, I found this wrong return value. I've checked the other recent posting and didn't found a patch for this. If someone already send a patch for this please point me to the right thread so that I can ack it. bye, Sumit From 9c3d4f637d80448922779b093fc4a2da909054e4 Mon Sep 17 00:00:00 2001 From:

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Nikolai Kondrashov
On 06/11/2014 08:14 PM, Pavel Reichl wrote: On Wed, 2014-06-11 at 19:34 +0300, Nikolai Kondrashov wrote: On 06/11/2014 05:54 PM, Pavel Reichl wrote: On Wed, 2014-06-11 at 17:28 +0300, Nikolai Kondrashov wrote: Sure, there are various ways to make Valgrind happy, although none of them are ver

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Pavel Reichl
On Wed, 2014-06-11 at 19:34 +0300, Nikolai Kondrashov wrote: > On 06/11/2014 05:54 PM, Pavel Reichl wrote: > > On Wed, 2014-06-11 at 17:28 +0300, Nikolai Kondrashov wrote: > >> > Sure, there are various ways to make Valgrind happy, although none of > them are > very easy. The import

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Nikolai Kondrashov
On 06/11/2014 06:19 PM, Lukas Slebodnik wrote: On (11/06/14 17:28), Nikolai Kondrashov wrote: It will expand macros, which can include Lua scripts, and those can do basically anything. See http://www.rpm.org/wiki/PackagerDocs/RpmLua Try putting %{lua: os.execute('echo "Hello, world!"')} into a

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Nikolai Kondrashov
On 06/11/2014 05:54 PM, Pavel Reichl wrote: On Wed, 2014-06-11 at 17:28 +0300, Nikolai Kondrashov wrote: Sure, there are various ways to make Valgrind happy, although none of them are very easy. The important thing is that we would have a way to track it now. This way, or another we should arr

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Lukas Slebodnik
On (11/06/14 17:28), Nikolai Kondrashov wrote: >On 06/11/2014 03:46 PM, Lukas Slebodnik wrote: >>On (11/06/14 14:52), Nikolai Kondrashov wrote: >>>On 06/11/2014 10:52 AM, Lukas Slebodnik wrote: I would like to avoind maintain two different lists with build dependency for fedora/rhel ba

Re: [SSSD] [PATCH] SSH: Allow newline at the end of public key values in LDAP

2014-06-11 Thread Jan Cholasta
On 5.6.2014 16:34, Pavel Reichl wrote: On Wed, 2014-06-04 at 17:05 +0200, Jan Cholasta wrote: Hi, the attached patch fixes . Honza ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Pavel Reichl
On Wed, 2014-06-11 at 17:28 +0300, Nikolai Kondrashov wrote: > > >> Sure, there are various ways to make Valgrind happy, although none of them > >> are > >> very easy. The important thing is that we would have a way to track it now. > >> This way, or another we should arrive to zero Valgrind erro

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Nikolai Kondrashov
On 06/11/2014 05:28 PM, Nikolai Kondrashov wrote: A package missing from CI dependencies would be immediately obvious as CI builds would fail and it would be easy to then update the lists. RPM dependencies are subset of CI dependencies. you can install CI dependencies with yum install and sssd

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Nikolai Kondrashov
On 06/11/2014 03:46 PM, Lukas Slebodnik wrote: On (11/06/14 14:52), Nikolai Kondrashov wrote: On 06/11/2014 10:52 AM, Lukas Slebodnik wrote: I would like to avoind maintain two different lists with build dependency for fedora/rhel based OSes. We already have listed all build dependencies in sss

Re: [SSSD] [PATCH] CONFIGURE: Prefer python2

2014-06-11 Thread Pavel Reichl
On Tue, 2014-06-10 at 16:16 +0200, Pavel Reichl wrote: > On Mon, 2014-06-09 at 09:15 +0200, Lukas Slebodnik wrote: > > ehlo, > > > > The configure script failed with python3 > > > > checking for python... /usr/bin/python > > checking for python version... 3.3 > > checking for python platform... l

[SSSD] [PATCH] IPA: Rename label in users_get_send/groups_get_send

2014-06-11 Thread Pavel Reichl
Hello, please see attached simple patch. PR >From 217d21393ecff8e72fc815b5acb54a9ee8e0b6cc Mon Sep 17 00:00:00 2001 From: Pavel Reichl Date: Wed, 11 Jun 2014 13:11:19 +0100 Subject: [PATCH] IPA: Rename label in users_get_send/groups_get_send Resolves: https://fedorahosted.org/sssd/ticket/2209 -

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Lukas Slebodnik
On (11/06/14 14:52), Nikolai Kondrashov wrote: >Hi Lukas, > >On 06/11/2014 10:52 AM, Lukas Slebodnik wrote: >>>[1] >>>http://sssd-ci.idm.lab.eng.brq.redhat.com:8080/job/new_private_master_fedora20/214/console >>The host is not accesible. It should not be in commit message. >> >>$ ping sssd-ci.idm.

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Nikolai Kondrashov
Hi Lukas, On 06/11/2014 10:52 AM, Lukas Slebodnik wrote: [1] http://sssd-ci.idm.lab.eng.brq.redhat.com:8080/job/new_private_master_fedora20/214/console The host is not accesible. It should not be in commit message. $ ping sssd-ci.idm.lab.eng.brq.redhat.com ping: unknown host sssd-ci.idm.lab.e

Re: [SSSD] sssd-ad uses wrong key to verify tgt at login time

2014-06-11 Thread Sumit Bose
On Sat, Jun 07, 2014 at 01:20:24PM +0200, Jurjen Bokma wrote: > On 06/06/2014 10:24 AM, Sumit Bose wrote: > >On Thu, Jun 05, 2014 at 04:54:50PM +0200, Jurjen Bokma wrote: > >>Hi, > >> > >>I believe the closing of > >>https://fedorahosted.org/sssd/ticket/1871#comment:2 > >>to be wrong. Jakub Hrozek

Re: [SSSD] [RFC] Add basic support for CI test execution

2014-06-11 Thread Lukas Slebodnik
On (10/06/14 19:28), Nikolai Kondrashov wrote: >Hi everyone, > >I would like to collect feedback about the attached patch adding support for >basic CI test runs. > >Please ask any questions, give any suggestions and voice any objections you >think are appropriate. > >Thank you. > >Nick >From dbb98