sssd-devel December 2015

sssd-devel@lists.fedorahosted.org

9 participants
48 discussions

[PATCHES] Fix warnings Wsign-compare
by Lukas Slebodnik 17 Oct '16

17 Oct '16

ehlo, I found few warnings on rawhide when I build sssd-1.13.2 https://kojipkgs.fedoraproject.org//work/tasks/8618/11918618/build.log LS

2 6

[PATCH] sss_override: improve --debug description
by Pavel Březina 24 Sep '16

24 Sep '16

https://fedorahosted.org/sssd/ticket/2813 I wanted to split include/debug_levels.xml into more files so we don't duplicate information, but I didn't figure out how to use xi:include in files that are already beeing included. I always managed to fail on dtd validation. Maybe someone more familiar with docbook may chime in.

4 12

[PATCH] Changes to allow SSSD to be used for access control with a machine account belonging to a domain controller.
by Jakub Hrozek 01 Jul '16

01 Jul '16

Hi, I'm resending a patch for ticket #2870 on behalf of the original reporter who also kindly submitted a patch. The patch looks good to me, as soon as we fix CI, I'll submit it as well and I think we can push it..

3 5

Re: DDNS: Use nsupdate keywords for all transactions
by Pavel Reichl 26 May '16

26 May '16

On 11/23/2015 04:32 PM, Petr Spacek wrote: > On 23.11.2015 13:53, Pavel Reichl wrote: >> On 11/20/2015 05:35 PM, Jakub Hrozek wrote: >>> On Fri, Nov 20, 2015 at 03:57:04PM +0100, Pavel Reichl wrote: >>>> Hello, >>>> >>>> please see attached patch. >>>> >>>> Thanks! >>> >>> Could you ask Petr Spacek to do a conceptual review? Somebody else (me >>> of noone else is interested) can then do the code-review but I don't >>> think our team has as good experience in handling nsupdate details.. >>> >> >> Hello Petr, could you do the conceptual review as Jakub wishes? >> >> I hope it should not be too hard as what we change in the patches is >> generation of a textual message for nsupdate and it's tested I dare to say >> thoroughly. > > Sure, just send me a link :-) I do not see it the message above. > Sure, hope the link will work for you: https://lists.fedorahosted.org/archives/list/sssd-devel%40lists.fedorahoste…

2 2

Running tests with different environment
by Jakub Hrozek 27 Apr '16

27 Apr '16

Hi, I'm working on pam_sss.so tests[1] and I ran into a problem that I don't know how to solve best. tl;dr, I would like to set different environment variables for different tests in order to set up cwrap libraries differently per-test. I can't use setenv() in the test itself, because that's too late, I need the variables to be set when __attribute__(constructor) is run, so normally at program startup, when the libraries are loaded. With cmake it's easy, use set(TEST_ENVIRONMENT). But with autotools, I only found two ways: - TESTS_ENVIRONMENT - this is fine, but it's per Makefile.am. So I would have to split the tests more, into pam_wrapper tests that also require uid_wrapper, tests that only require pam_wrapper, ... - LOG_COMPILER - this allows to run a wrapper script before a test that receives the test name as argv. So this is pretty much what I want except this is a feature new to automake 1.12, which would rule out both RHEL-6 and Ubuntu Trusty (which is used by Travis) So I'm really leaning towards creating a src/tests/cwrap/pwrap/Makefile.am and src/tests/cwrap/pwrap_root/Makefile.am. The downside of multiple Makefile.am files is that there is some code duplication and the build takes longer. But I still think there is enough interest (from us and from our users) to support git master on old platforms. I can file a ticket to remove this and use LOG_COMPILER when we drop support for RHEL-6 and old Ubuntu versions... If you disagree, please reply, otherwise I'm going to send a patch with per-test Makefile... [1] https://jhrozek.fedorapeople.org/cov/sssdcov/sss_client/pam_sss.c.gcov.html 42% code coverage, up from 0% last week. Woo!

4 10

[PATCHES] PAM: refactor pam_reply
by Pavel Reichl 27 Apr '16

27 Apr '16

Hello, please see attached patch set which contains some refactoring which is in my opinion quite safe. It doesn't touch the recursive call of pam_reply() as I think that changing that would be more risky change. Thanks!

3 6

[PATCH] ldap_test.py: Modify enum cache timeouts
by Michal Židek 26 Apr '16

26 Apr '16

Hi! I saw some integration tests failures recently, and I think there is a race condition between the enumeration refresh timeout and the sleeps after some operations that wait for this timeout. SSSD fails to populate changes from LDAP in time and some asserts can fail because of this. So far I saw 4 tests to fail like this, which is already quite a lot. The attached patch modifies the timeout values and hopefully removes the issue. Michal

4 20

HOWTO: Troubleshooting SUDO
by Pavel Březina 01 Apr '16

01 Apr '16

Hi, I just submitted a sudo troubleshooting guide [1]. If you find anything missing, please, let me know. [1] https://fedorahosted.org/sssd/wiki/HOWTO_Troubleshoot_SUDO

1 1

[PATCHES] util: Continue if setlocale fails
by Michal Židek 03 Mar '16

03 Mar '16

Hi, attached is patch to fix ticket: https://fedorahosted.org/sssd/ticket/2785 And one additional patch to add DEBUG message. Michal

4 16

[PATCH] Use refcount to keep track of server structures returned from failover
by Jakub Hrozek 03 Mar '16

03 Mar '16

Hi, the attached patches are my proposal to fix https://fedorahosted.org/sssd/ticket/2829 I haven't tested them past make check yet, because I'm not sure I like them myself :) but at the same time I can't see a better way to keep track of the servers and let callers set state of servers. The most ugly thing so far IMO is the fo_internal_owner member. I would prefer to instead have a fo_server_wrap structure that would be used for the server_list, but I didn't want to do a large change before we agree the refcount is a good idea at all. The other ugly side-effect is that we need to be sure that nobody calls talloc_free on the fo_server structure. Instead, only the parent context can be freed (that's also what the first patch is about).

3 10

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

sssd-devel December 2015