sssd-devel September 2016

sssd-devel@lists.fedorahosted.org

18 participants
95 discussions

[sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (comment)
by fidencio 07 Sep '16

07 Sep '16

jhrozek commented on a pull request """ On Tue, Sep 06, 2016 at 11:49:09AM -0700, lslebodn wrote: > IMHO, it might be better to close this PR. > If we decide to dor support for libini_config < 1.1 or 1.2 > then it will be a different patch anyway. @see my previous comment The only reason I suggest deferred over closing is that if we close this PR, we will never remember to ressurect it. If it's going to be deferred, it will keep coming up in the PR list. """ See the full comment at https://github.com/SSSD/sssd/pull/10#issuecomment-245204278

1 0

[sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (comment)
by fidencio 07 Sep '16

07 Sep '16

lslebodn commented on a pull request """ IMHO, it might be better to close this PR. If we decide to dor support for libini_config < 1.1 or 1.2 then it will be a different patch anyway. @see my previous comment """ See the full comment at https://github.com/SSSD/sssd/pull/10#issuecomment-245050796

4 5

[sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (+deferred)
by fidencio 06 Sep '16

06 Sep '16

fidencio's pull request #10: "UTIL: Remove support to libini older than 1.0.0" label *deferred* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/10

1 0

[sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (comment)
by fidencio 06 Sep '16

06 Sep '16

jhrozek commented on a pull request """ On Tue, Sep 06, 2016 at 05:16:00AM -0700, fidencio wrote: > The distributions that would break with this patch are: > - RHEL/CentOS 5 and older I don't think we care about RHEL-5 with master and I'm not sure sssd master even builds there. > - Debian Wheezy (from 2013) and older OK, this one I think we care about for the basic functionality: 7.0 Wheezy May 4th 2013 April 26th 2016 (full) / May 2018 (LTS) But LTS means bug fixes and for those I think sssd-1-13 would be OK. > - Ubuntu 12.04 LTS and older 12.04 is often used (for example travis-ci still offers only this distro) and ends its life in 2017. On one hand, it's unlikely that users of LTS distributions will run master, they will probably only run sssd-1-13 or some PPAs, on the other hand, I don't see too much value except for cleaner code. So all in all I suggest we nack this patchset for now and push it when Ubuntu 12.04 goes EOL. Does that sound like a reasonable compromise? """ See the full comment at https://github.com/SSSD/sssd/pull/10#issuecomment-244935329

2 1

[sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (comment)
by fidencio 06 Sep '16

06 Sep '16

lslebodn commented on a pull request """ On (06/09/16 05:16), fidencio wrote: >The distributions that would break with this patch are: >- RHEL/CentOS 5 and older >- Debian Wheezy (from 2013) and older >- Ubuntu 12.04 LTS and older > >I was not able to find what's the version of the package on SLES > * libini_config-0.7.0 is feature wise the same as libini_config-1.0 sssd uses it for basic parsing of ini files * libini_config-1.1 is an optional feature used in ad provider for parsing gpo files * libini_config-1.2 provides optional feature for config snippets in /etc/sssd/conf.d * libini_config-1.3 provides optional feature for validation of ini files After this patch we still have 3 optional features provided by different versions of libini_config. If we want to drop support for older version of libini_config and siplify the wrapper then I would drop support for libini_config < 1.3 and remove wrappers in src/util/sss_ini.c However, libini_config-1.3 is only in fedora. LS """ See the full comment at https://github.com/SSSD/sssd/pull/10#issuecomment-244937445

1 0

[sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (comment)
by fidencio 06 Sep '16

06 Sep '16

fidencio commented on a pull request """ The distributions that would break with this patch are: - RHEL/CentOS 5 and older - Debian Wheezy (from 2013) and older - Ubuntu 12.04 LTS and older I was not able to find what's the version of the package on SLES """ See the full comment at https://github.com/SSSD/sssd/pull/10#issuecomment-244932095

1 0

[sssd PR#10] UTIL: Remove support to libini older than 1.0.0 (comment)
by fidencio 06 Sep '16

06 Sep '16

jhrozek commented on a pull request """ On Thu, Sep 01, 2016 at 09:01:54AM -0700, lslebodn wrote: > On (01/09/16 08:35), fidencio wrote: > >On Thu, Sep 1, 2016 at 2:10 PM, lslebodn <notifications(a)github.com> wrote: > > > >> On (01/09/16 04:32), Jakub Hrozek wrote: > >> >On Thu, Sep 01, 2016 at 03:21:06AM -0700, fidencio wrote: > >> >> On Thu, Sep 1, 2016 at 11:54 AM, lslebodn <notifications(a)github.com> > >> wrote: > >> >> > > >> >> > On (31/08/16 23:18), fidencio wrote: > >> >> > >libini 1.0.0 is part of ding-libs 0.3.0 and has been around since > >> 2013. > >> >> > >Even the old systems that we have to support already have a newer > >> >> > >version of the library. RHEL6, for instance, has ding-libs 0.4.0 > >> which > >> >> > >provides libinit 1.1.0. > >> >> > > > >> >> > >By removing this code we also can stop depending on libcollection. > >> >> > > > >> >> > >Signed-off-by: Fabiano Fidêncio <fidencio(a)redhat.com> > >> >> > >You can view, comment on, or merge this pull request online at: > >> >> > > > >> >> > > https://github.com/SSSD/sssd/pull/10 > >> >> > > > >> >> > >-- Commit Summary -- > >> >> > > > >> >> > > * UTIL: Remove support to libini older than 1.0.0 > >> >> > > > >> >> > >-- File Changes -- > >> >> > > > >> >> > > M configure.ac (1) > >> >> > > M contrib/ci/deps.sh (1) > >> >> > > M contrib/sssd.spec.in (1) > >> >> > > D src/external/libcollection.m4 (9) > >> >> > > M src/util/sss_ini.c (97) > >> >> > > > >> >> > >-- Patch Links -- > >> >> > > > >> >> > >https://github.com/SSSD/sssd/pull/10.patch > >> >> > >https://github.com/SSSD/sssd/pull/10.diff > >> >> > > > >> >> > OpenSUSE LEAP has just ding-libs 0.3.0.1 in official repositories. > >> >> > http://software.opensuse.org/package/ding-libs?search_term=ding-libs > >> >> > >> >> Yep. OpenSUSE LEAP has 0.3.0.1, Debian Stable (Jessie) has 0.4.0, > >> >> latest Ubuntu LTS has 0.5.0. > >> >> And all of them would be able to build SSSD with my patch without any > >> issues. > >> > > >> >Two questions: > >> > 1) how long until the distributions with too old ding-libs go out of > >> > support? > >> > > > >Hmm. Unfortunately I don't have an answer for you. > >What are the major distributions that we want to support? Debian, Ubuntu > >LTS, RHEL, SLES ...? > > > I would prefer if limited version of sssd (ldap + krb5 provider) > could be compiled almost anywhere. (even old distributions) > It is not only about major distributions. > > > > >> > 2) since we will be (likely) supporting sssd-1-13 for the lifetime > >> > of RHEL-6, can we say that the old distributions just use sssd-1-13? > >> > > >> libini_config-1.0 does not provide any new functionality > >> which is not in libini_config < 1.0 > >> > > > >> They have just a different API (and moreover libini_config-1.0 > >> still provides old API) (at least from sssd POV) > >> > > > >There's no new functionality, true. But there's a quite good API > >simplification > I agree that API is better. > But feature wise are the same. So if we wanted to drop > support for libini_config < 1.0 then we could > drop support for libini_config-1.0. > It would simplify more things in sssd > > However, OpenSUSE LEAP has just a libini_config-1.0 This PR has stalled somehow. Can we either move it forward or reject? I admit I got a bit lost in the comments, which distributions would we break with this patch? Are any of them supported/will be supported in th near future? """ See the full comment at https://github.com/SSSD/sssd/pull/10#issuecomment-244928774

1 0

[PATCH SYSDB: Fix handling of version in sysdb_cache_connect_helper
by Lukas Slebodnik 02 Sep '16

02 Sep '16

ehlo, Clang static analyzer assume that ldb_search can found 0 entries in the tree "cn=sysdb". Thenvariable version could be used uninitialized. We cannot get to such state in sssd but we already handle a case for more then one entry. LS

3 7

[sssd PR#11] SECRETS: Don't remove a container when it has children (edited)
by fidencio 02 Sep '16

02 Sep '16

fidencio's pull request #11: "SECRETS: Don't remove a container when it has children" was edited See the full pull-request at https://github.com/SSSD/sssd/pull/11 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/11/head:pr11 git checkout pr11

1 0

[sssd PR#11] SECRETS: Don't remove a container when it has children (synchronize)
by fidencio 02 Sep '16

02 Sep '16

fidencio's pull request #11: "SECRETS: Don't remove a container when it has children" was synchronize See the full pull-request at https://github.com/SSSD/sssd/pull/11 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/11/head:pr11 git checkout pr11

1 0

← Newer
1
2
3
4
5
6
7
8
9
10
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

sssd-devel September 2016