Re: [SSSD] [PATCH] Monitor and sbus changes for running SSSD as a non-privileged user

2014-10-17 Thread Simo Sorce
On Fri, 17 Oct 2014 18:47:28 +0200 Jakub Hrozek wrote: > On Thu, Oct 16, 2014 at 10:25:12AM +0200, Jakub Hrozek wrote: > > On Wed, Oct 15, 2014 at 06:19:49PM -0400, Simo Sorce wrote: > > > On Wed, 15 Oct 2014 22:24:04 +0200 > > > Jakub Hrozek wrote: > > > > > > > Attached are patches that perfo

Re: [SSSD] [PATCH] Monitor and sbus changes for running SSSD as a non-privileged user

2014-10-17 Thread Jakub Hrozek
On Thu, Oct 16, 2014 at 10:25:12AM +0200, Jakub Hrozek wrote: > On Wed, Oct 15, 2014 at 06:19:49PM -0400, Simo Sorce wrote: > > On Wed, 15 Oct 2014 22:24:04 +0200 > > Jakub Hrozek wrote: > > > > > Attached are patches that perform changes in the monitor process and > > > the low-level sbus and sy

Re: [SSSD] [PATCH] UTIL: Add a function to convert id_t from a number or a name

2014-10-17 Thread Jakub Hrozek
On Fri, Oct 17, 2014 at 02:17:21PM +0200, Jakub Hrozek wrote: > On Thu, Oct 16, 2014 at 04:37:25PM +0200, Pavel Reichl wrote: > > > > On 10/15/2014 09:35 PM, Jakub Hrozek wrote: > > >0001-UTIL-Add-a-function-to-convert-id_t-from-a-number-or.patch > > > > > > > > > From 9ef055cef1d109e0e932ddb0c43e

Re: [SSSD] [PATCH] Packaging fixes related to non-root SSSD

2014-10-17 Thread Jakub Hrozek
On Wed, Oct 15, 2014 at 09:55:07PM +0200, Jakub Hrozek wrote: > Hi, > > attached are two patches that change the file ownership when SSSD runs > as as a non-root user. Please see the commit messages for more details. > > I have one question -- is it wise to also set the permissions of > directori

Re: [SSSD] [PATCHES] Make nss responder aware of views and overrides

2014-10-17 Thread Sumit Bose
On Fri, Oct 17, 2014 at 02:05:44PM +0200, Pavel Březina wrote: > On 10/15/2014 01:08 PM, Sumit Bose wrote: > >On Tue, Oct 14, 2014 at 05:57:36PM +0200, Sumit Bose wrote: > >>Hi, > >> > >>this patches make the nss responder view aware. A number of new sysdb > >>calls is added as drop in replacements

Re: [SSSD] [PATCH] IPA: Handle NULL members in process_members()

2014-10-17 Thread Jakub Hrozek
On Tue, Oct 14, 2014 at 05:28:36PM +0200, Lukas Slebodnik wrote: > On (14/10/14 14:27), Jakub Hrozek wrote: > >Hi, > > > >I stumbled upon a crash when reviewing Pavel's recent subdomain fix against > >an IPA 3.3 server. I'm not sure if it's the right way to fix the problem, > >normally I would pref

[SSSD] SUDO: support views

2014-10-17 Thread Pavel Březina
SUDO part of views. From 351c2848b6866e05eb83944257f91a7b0ad1971e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= Date: Fri, 10 Oct 2014 16:50:47 +0200 Subject: [PATCH 1/2] sysdb: add sss_view_ldb_msg_find_attr_as_int() --- src/db/sysdb.h | 5 + src/db/sysdb_views.c |

Re: [SSSD] [PATCH] UTIL: Add a function to convert id_t from a number or a name

2014-10-17 Thread Jakub Hrozek
On Thu, Oct 16, 2014 at 04:37:25PM +0200, Pavel Reichl wrote: > > On 10/15/2014 09:35 PM, Jakub Hrozek wrote: > >0001-UTIL-Add-a-function-to-convert-id_t-from-a-number-or.patch > > > > > > From 9ef055cef1d109e0e932ddb0c43e6a59b9b29e0f Mon Sep 17 00:00:00 2001 > >From: Jakub Hrozek > >Date: Tue, 23

[SSSD] [PATCHES] IFP: support views

2014-10-17 Thread Pavel Březina
Since the NSS and PAM parts of the views are in the review process and I don't expect any big changes there, I'm sending IFP part. It depends on Sumit's patches. From fca02ff716215f952f84191be73cced61b151024 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Pavel=20B=C5=99ezina?= Date: Thu, 2 Oct 2014 16

Re: [SSSD] [PATCHES] Make nss responder aware of views and overrides

2014-10-17 Thread Sumit Bose
On Wed, Oct 15, 2014 at 01:08:29PM +0200, Sumit Bose wrote: > On Tue, Oct 14, 2014 at 05:57:36PM +0200, Sumit Bose wrote: > > Hi, > > > > this patches make the nss responder view aware. A number of new sysdb > > calls is added as drop in replacements from existing ones. I preferred > > this instea

Re: [SSSD] [PATCHES] Make nss responder aware of views and overrides

2014-10-17 Thread Pavel Březina
On 10/15/2014 01:08 PM, Sumit Bose wrote: On Tue, Oct 14, 2014 at 05:57:36PM +0200, Sumit Bose wrote: Hi, this patches make the nss responder view aware. A number of new sysdb calls is added as drop in replacements from existing ones. I preferred this instead of modifying the existing ones to n

Re: [SSSD] [PATCH] CI: Remove Clang analyzer

2014-10-17 Thread Nikolai Kondrashov
On 10/17/2014 01:11 PM, Nikolai Kondrashov wrote: The attached patch removes Clang analyzer run from CI as it takes a long time and its results are ignored. And here is the result of a successfull CI run with it: http://sssd-ci.duckdns.org/logs/job/1/91/summary.html Nick __

[SSSD] [PATCH] CI: Remove Clang analyzer

2014-10-17 Thread Nikolai Kondrashov
Hi everyone, The attached patch removes Clang analyzer run from CI as it takes a long time and its results are ignored. I initially expected it to be taken into use quickly, but as that is not happening, we'd better remove it to save valuable CI server time. We can always add it back when a sol