URL: https://github.com/SSSD/sssd/pull/37
Author: jhrozek
Title: #37: Add DEBUG messages to the sssd-secrets responder
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/37/head:pr37
git checkout pr37
From
URL: https://github.com/SSSD/sssd/pull/37
Title: #37: Add DEBUG messages to the sssd-secrets responder
jhrozek commented:
"""
Thank you for the review, hopefully I addressed all comments in the patches I
just pushed. CI is pending.
"""
See the full comment at
URL: https://github.com/SSSD/sssd/pull/37
Title: #37: Add DEBUG messages to the sssd-secrets responder
Label: -Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fed
URL: https://github.com/SSSD/sssd/pull/38
Title: #38: IPA: Initialize has_changes boolean control value
jhrozek commented:
"""
On Tue, Oct 04, 2016 at 03:05:09AM -0700, fidencio wrote:
> @jhrozek: I've checked sssd-1-13 already. There refresh_has_changes is always
> i
URL: https://github.com/SSSD/sssd/pull/38
Title: #38: IPA: Initialize has_changes boolean control value
jhrozek commented:
"""
CI: http://sssd-ci.duckdns.org/logs/job/54/57/summary.html
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/38
Author: jhrozek
Title: #38: IPA: Initialize has_changes boolean control value
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/38/head:pr38
git checkout pr38
From
URL: https://github.com/SSSD/sssd/pull/38
Title: #38: IPA: Initialize has_changes boolean control value
jhrozek commented:
"""
OK, pushed a new version of the patch (just the commit message differs) and
I'll push it to upstream if there are no further comments.
"&q
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
jhrozek commented:
"""
Hey @pbrezina just making sure you didn't miss the review. About
/sbin/service...since there are fewer and fewer non-systemd systemd, I wouldn't
bother making
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
jhrozek commented:
"""
Hey @pbrezina just making sure you didn't miss the review. About
/sbin/service...since there are fewer and fewer non-systemd systems, I wouldn't
bother making
URL: https://github.com/SSSD/sssd/pull/37
Title: #37: Add DEBUG messages to the sssd-secrets responder
jhrozek commented:
"""
CI: http://sssd-ci.duckdns.org/logs/job/54/58/summary.html
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/37
Author: jhrozek
Title: #37: Add DEBUG messages to the sssd-secrets responder
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/37/head:pr37
git checkout pr37
URL: https://github.com/SSSD/sssd/pull/37
Title: #37: Add DEBUG messages to the sssd-secrets responder
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.o
URL: https://github.com/SSSD/sssd/pull/37
Title: #37: Add DEBUG messages to the sssd-secrets responder
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted
URL: https://github.com/SSSD/sssd/pull/37
Title: #37: Add DEBUG messages to the sssd-secrets responder
jhrozek commented:
"""
Pushed to master in
41cd6072648bb7a9e14e56ed38004a2947f67657
f931864f46cb19346be45f3b6f55da54c8dc7078
8fe4f98192aa23729d3607bf2
URL: https://github.com/SSSD/sssd/pull/38
Author: jhrozek
Title: #38: IPA: Initialize has_changes boolean control value
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/38/head:pr38
git checkout pr38
From
URL: https://github.com/SSSD/sssd/pull/38
Title: #38: IPA: Initialize has_changes boolean control value
jhrozek commented:
"""
On Tue, Oct 04, 2016 at 12:32:44PM -0700, fidencio wrote:
> Your text editor puts 4 spaces before the link for the bug. Please, avoid it.
> Also,
URL: https://github.com/SSSD/sssd/pull/38
Title: #38: IPA: Initialize has_changes boolean control value
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.
URL: https://github.com/SSSD/sssd/pull/38
Title: #38: IPA: Initialize has_changes boolean control value
jhrozek commented:
"""
master: 7b07f50dfdfa1e94c82d86a957ee7c9852d7a322
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/38
Title: #38: IPA: Initialize has_changes boolean control value
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahoste
URL: https://github.com/SSSD/sssd/pull/38
Author: jhrozek
Title: #38: IPA: Initialize has_changes boolean control value
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/38/head:pr38
git checkout pr38
URL: https://github.com/SSSD/sssd/pull/36
Title: #36: Partial fix for #3169
jhrozek commented:
"""
@fidencio can you rebase this patch set atop origin/master, please?
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/36
Title: #36: Partial fix for #3169
Label: +Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/36
Title: #36: Partial fix for #3169
jhrozek commented:
"""
Thank you, the patch looks good to me now and returns an error code (tested
manually).
ACK.
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/36
Title: #36: Partial fix for #3169
Label: +Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/36
Title: #36: Partial fix for #3169
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/36
Title: #36: Partial fix for #3169
jhrozek commented:
"""
* master: 65a38b8c9cabde6c46cc0e9868f54cb9bb10afbf
"""
See the full comment at
https://github.com/SSSD/sssd/pull/36#issuecomment-251634041
__
URL: https://github.com/SSSD/sssd/pull/36
Title: #36: Partial fix for #3169
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/36
Author: fidencio
Title: #36: Partial fix for #3169
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/36/head:pr36
git checkout pr36
___
sssd
URL: https://github.com/SSSD/sssd/pull/39
Title: #39: RESPONDER: Enable sudoRule in case insen. domains
jhrozek commented:
"""
On Thu, Oct 06, 2016 at 03:51:05AM -0700, Pavel Březina wrote:
> The patch that should fix this is: 61913b8f0d1ba54d82640500d7486fac5f72b030
Well, d
URL: https://github.com/SSSD/sssd/pull/40
Title: #40: TESTS: Remove a leftover debug message
jhrozek commented:
"""
On Fri, Oct 07, 2016 at 01:43:19AM -0700, celestian wrote:
> I use one-line rule and I give ack without running tests.
> => ACK
Just a note -- github
URL: https://github.com/SSSD/sssd/pull/40
Author: fidencio
Title: #40: TESTS: Remove a leftover debug message
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/40/head:pr40
git checkout pr40
___
URL: https://github.com/SSSD/sssd/pull/40
Title: #40: TESTS: Remove a leftover debug message
jhrozek commented:
"""
* master: 01022614d46c7548d3f1b2c048bf9e1b0ed094fd
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/40
Title: #40: TESTS: Remove a leftover debug message
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/40
Title: #40: TESTS: Remove a leftover debug message
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/42
Title: #42: MAN: Typo in id mapping explanation
jhrozek commented:
"""
* master: 3955667b6e5071cc1264422cb9d702534cf9bc21
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/42
Title: #42: MAN: Typo in id mapping explanation
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/42
Author: mzidek-rh
Title: #42: MAN: Typo in id mapping explanation
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/42/head:pr42
git checkout pr42
_
URL: https://github.com/SSSD/sssd/pull/42
Title: #42: MAN: Typo in id mapping explanation
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/43
Title: #43: SUDO: Adding user name alias to sudoRule filter
jhrozek commented:
"""
Can we close PR #39 ?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/43#issuecomment-252260308
URL: https://github.com/SSSD/sssd/pull/41
Title: #41: MAN: Wrong defaults for AD provider
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/41
Title: #41: MAN: Wrong defaults for AD provider
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/41
Author: mzidek-rh
Title: #41: MAN: Wrong defaults for AD provider
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/41/head:pr41
git checkout pr41
_
URL: https://github.com/SSSD/sssd/pull/41
Title: #41: MAN: Wrong defaults for AD provider
jhrozek commented:
"""
master: a5b4f865aae08d978e7bbbe1ff5c4b93ac41aa85
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/43
Title: #43: SUDO: Adding user name alias to sudoRule filter
jhrozek commented:
"""
On Fri, Oct 07, 2016 at 07:10:36AM -0700, celestian wrote:
> PR #39 is for the same issue, but for version 1-13. And I need to change the
> patch.
URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
Label: +Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@l
URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
jhrozek commented:
"""
ACK. I will push the patch once Coverity finishes.
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@l
URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
Label: +Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-deve
URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
jhrozek commented:
"""
I'm afraid Coverity found something it doesn't like:
```
Error: COMPILER_WARNING:
sssd-1.14.2/src/responder/nss/nsssrv_cmd.c
URL: https://github.com/SSSD/sssd/pull/35
Title: #35: sss_cache: improve option argument handling
Label: +Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/35
Title: #35: sss_cache: improve option argument handling
jhrozek commented:
"""
Sorry for the late review, I was hoping some of the newer developer would pick
it up :)
ACK to the patch. The code looks OK and the patch works:
```
# sss
URL: https://github.com/SSSD/sssd/pull/13
Title: #13: MEMBEROF: Don't resolve members if they are removed
Label: +Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
Label: -Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/13
Title: #13: MEMBEROF: Don't resolve members if they are removed
jhrozek commented:
"""
I think :@lslebodn is right, because I removed just the memberof patch. Now I
have:
```
ab6dee1 TESTS: Adding intg. tests on nested groups
257b6
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
jhrozek commented:
"""
Thanks, this looks good, but I wonder if we could really just detect the
service command (I know we talked about it and then decided not to..but I
forgot why).
See for
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
Label: +Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
jhrozek commented:
"""
On Mon, Oct 10, 2016 at 01:07:12AM -0700, sumit-bose wrote:
> On Fri, Oct 07, 2016 at 08:21:21AM -0700, Jakub Hrozek wrote:
>
URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
Label: +Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@l
URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
jhrozek commented:
"""
* master: dcdf292567d50e5cc527766c1944dcf6a8ecacc5
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@l
URL: https://github.com/SSSD/sssd/pull/31
Author: sumit-bose
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/31/head:pr31
git checkout pr31
__
URL: https://github.com/SSSD/sssd/pull/31
Title: #31: nss: allow UPNs in SSS_NSS_GETSIDBYNAME and SSS_NSS_GETORIGBYNAME
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lis
URL: https://github.com/SSSD/sssd/pull/35
Title: #35: sss_cache: improve option argument handling
jhrozek commented:
"""
master: 1330390c698ca0802200725df43356557aa633a2
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/35
Author: justin-stephenson
Title: #35: sss_cache: improve option argument handling
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/35/head:pr35
git checkout pr35
_
URL: https://github.com/SSSD/sssd/pull/35
Title: #35: sss_cache: improve option argument handling
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/35
Title: #35: sss_cache: improve option argument handling
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/45
Author: jhrozek
Title: #45: tests: Add unit tests for UPNs in SSS_NSS_GETSIDBYNAME and
SSS_NSS_GETORIGBYNAME
Action: opened
PR body:
"""
This is what I used to test recent Sumit's patches for ticket #3194.
"""
URL: https://github.com/SSSD/sssd/pull/45
Author: jhrozek
Title: #45: tests: Add unit tests for UPNs in SSS_NSS_GETSIDBYNAME and
SSS_NSS_GETORIGBYNAME
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/45/head:pr45
git
URL: https://github.com/SSSD/sssd/pull/46
Title: #46: sss_client: Defer thread cancellation until completion of nss/pam
operations
jhrozek commented:
"""
ok to test
"""
See the full comment at
https://github.com/SSSD/sssd/
URL: https://github.com/SSSD/sssd/pull/34
Title: #34: cache_req: move from switch to plugins
jhrozek commented:
"""
Well, it's still a lot of code :) but at least I started the review now. For
starters, Coverity found some warnings:
```
Error: FORWARD_NULL (CWE-476):
sssd-
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
jhrozek commented:
"""
On Mon, Oct 10, 2016 at 03:04:35AM -0700, Pavel Březina wrote:
> Hi, your patch looks good to me. There was no specific reason, see
> https://github.com/SSSD/sssd
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
Label: +Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
Label: -Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/45
Title: #45: tests: Add unit tests for UPNs in SSS_NSS_GETSIDBYNAME and
SSS_NSS_GETORIGBYNAME
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd
URL: https://github.com/SSSD/sssd/pull/45
Title: #45: tests: Add unit tests for UPNs in SSS_NSS_GETSIDBYNAME and
SSS_NSS_GETORIGBYNAME
jhrozek commented:
"""
* master:
* eb9bc1c590b8c3b3b58574c70d9fe5357ef3e901
* 03713a6444fdecd01465b9d5fbfead9601adce6e
"""
URL: https://github.com/SSSD/sssd/pull/45
Author: jhrozek
Title: #45: tests: Add unit tests for UPNs in SSS_NSS_GETSIDBYNAME and
SSS_NSS_GETORIGBYNAME
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/45/head:pr45
git
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
jhrozek commented:
"""
* master: 03713a6..761515e
"""
See the full comment at
https://github.com/SSSD/sssd/pull/30#issuecomment-252905006
_
URL: https://github.com/SSSD/sssd/pull/30
Author: pbrezina
Title: #30: sssctl: use systemd D-Bus API
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/30/head:pr30
git checkout pr30
URL: https://github.com/SSSD/sssd/pull/30
Title: #30: sssctl: use systemd D-Bus API
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
URL: https://github.com/SSSD/sssd/pull/47
Author: jhrozek
Title: #47: BUILD: Fix build without /sbin/service installed on the build host
Action: opened
PR body:
"""
There were some issues in the sssctl-related patches that we pushed
recently. First, the build failed if no ser
URL: https://github.com/SSSD/sssd/pull/47
Title: #47: BUILD: Fix build without /sbin/service installed on the build host
jhrozek commented:
"""
Apparently our CI, see the master build status :). And I guess especially
Debian-based systems or other distributions might rely on t
URL: https://github.com/SSSD/sssd/pull/49
Author: jhrozek
Title: #49: Try to match multiple results from an AD initgroups request
against domain's search bases, too
Action: opened
PR body:
"""
This is a proposed way of solving
https://fedorahosted.org/sssd/ticket/3199
URL: https://github.com/SSSD/sssd/pull/44
Title: #44: libwbclient-sssd: update interface to version 0.13
jhrozek commented:
"""
On Thu, Oct 13, 2016 at 02:35:57AM -0700, fidencio wrote:
> Okay, please, fire our CI and feel free to push the patches after CI passes.
Unfortunat
URL: https://github.com/SSSD/sssd/pull/44
Title: #44: libwbclient-sssd: update interface to version 0.13
jhrozek commented:
"""
On Thu, Oct 13, 2016 at 04:10:04AM -0700, fidencio wrote:
> @jhrozek: Okay, okay. I'm running CI on #47 and we can get back to this one
>
URL: https://github.com/SSSD/sssd/pull/47
Author: jhrozek
Title: #47: BUILD: Fix build without /sbin/service installed on the build host
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/47/head:pr47
git checkout pr47
URL: https://github.com/SSSD/sssd/pull/13
Title: #13: MEMBEROF: Don't resolve members if they are removed
jhrozek commented:
"""
So I wanted to write a test for the memberof patch, but I couldn't reproduce
the ticket #2940 even without the memberof patch. I tried
URL: https://github.com/SSSD/sssd/pull/44
Title: #44: libwbclient-sssd: update interface to version 0.13
jhrozek commented:
"""
btw what the library exports also looks good to me:
```
objdump -T /usr/lib64/sssd/modules/libwbclient.so.0.13.0 | grep WBCLIENT_0.13
5e7
URL: https://github.com/SSSD/sssd/pull/47
Title: #47: BUILD: Fix build without /sbin/service installed on the build host
jhrozek commented:
"""
@lslebodn suggested on IRC to only search for service in /sbin and /usr/sbin. I
added another patch that does that..
CI: http://sssd
URL: https://github.com/SSSD/sssd/pull/47
Author: jhrozek
Title: #47: BUILD: Fix build without /sbin/service installed on the build host
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/47/head:pr47
git checkout pr47
URL: https://github.com/SSSD/sssd/pull/13
Title: #13: MEMBEROF: Don't resolve members if they are removed
jhrozek commented:
"""
btw since the memberof patch we couldn't test is gone, I'm fine with pushing
these two.
"""
See the full com
URL: https://github.com/SSSD/sssd/pull/13
Title: #13: MEMBEROF: Don't resolve members if they are removed
Label: +Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahos
URL: https://github.com/SSSD/sssd/pull/48
Title: #48: sssctl: Flags for commadn initialization
Label: -Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahoste
URL: https://github.com/SSSD/sssd/pull/34
Title: #34: cache_req: move from switch to plugins
jhrozek commented:
"""
Apart from the nitpicks, the code looks better to me (I like that the lookup
functionality is abstracted, so adding new lookups will be easy).
I tested several k
URL: https://github.com/SSSD/sssd/pull/34
Title: #34: cache_req: move from switch to plugins
Label: +Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.
URL: https://github.com/SSSD/sssd/pull/13
Title: #13: MEMBEROF: Don't resolve members if they are removed
jhrozek commented:
"""
yes, if we don't know how to trigger the code, we shouldn't push it.
OK to push the two commits now?
"""
See the full
URL: https://github.com/SSSD/sssd/pull/34
Title: #34: cache_req: move from switch to plugins
Label: -Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.
URL: https://github.com/SSSD/sssd/pull/43
Title: #43: RESPONDER: Enable sudoRule in case insen. domains (1.14)
Label: +Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@
URL: https://github.com/SSSD/sssd/pull/47
Title: #47: BUILD: Fix build without /sbin/service installed on the build host
Label: +Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-dev
URL: https://github.com/SSSD/sssd/pull/47
Title: #47: BUILD: Fix build without /sbin/service installed on the build host
jhrozek commented:
"""
OK, new patches pushed
"""
See the full comment at
https://github.com/SSSD/sssd/
201 - 300 of 3879 matches
Mail list logo