On Thu, Oct 06, 2016 at 12:49:30PM +0200, Sumit Bose wrote:
> Question, do we need search-and-replace at all (or at this
> stage)? Most of the interesting values from the SAN should be
> directly map-able to LDAP attributes. And processing the string
> representation of might be tricky as discuss
URL: https://github.com/SSSD/sssd/pull/39
Title: #39: RESPONDER: Enable sudoRule in case insen. domains
celestian commented:
"""
Hi,
I have question on cache_req_recv():
https://github.com/SSSD/sssd/blob/master/src/responder/common/responder_cache_req.c#L1464
```
errno_t cache_req_recv(TALLOC_C
URL: https://github.com/SSSD/sssd/pull/39
Title: #39: RESPONDER: Enable sudoRule in case insen. domains
celestian commented:
"""
Hi,
I have question on cache_req_recv():
https://github.com/SSSD/sssd/blob/master/src/responder/common/responder_cache_req.c#L1464
```
errno_t cache_req_recv(TALLOC_C
URL: https://github.com/SSSD/sssd/pull/39
Title: #39: RESPONDER: Enable sudoRule in case insen. domains
celestian commented:
"""
Hi,
I have question on cache_req_recv():
https://github.com/SSSD/sssd/blob/master/src/responder/common/responder_cache_req.c#L1464
```
errno_t cache_req_recv(TALLOC_C
On Thu, Oct 06, 2016 at 06:38:23PM +0200, Sumit Bose wrote:
> On Thu, Oct 06, 2016 at 04:41:10PM +0200, Jakub Hrozek wrote:
> > Hi,
> >
> > with Alexander's help, I wrote up a design page about how SSSD should
> > read Fleet Commander data from IPA and present them to the FC client
> > component.
URL: https://github.com/SSSD/sssd/pull/41
Title: #41: MAN: Wrong defaults for AD provider
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/40
Title: #40: TESTS: Remove a leftover debug message
celestian commented:
"""
I use one-line rule and I give ack without running tests.
=> ACK
"""
See the full comment at
https://github.com/SSSD/sssd/pull/40#issuecomment-252185048
___
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/39
Title: #39: RESPONDER: Enable sudoRule in case insen. domains
pbrezina commented:
"""
Not, it is not a bug. The output parameter _name contains a name from the
request modified according to rules of the selected domain. I.e. the name that
was used to l
Hi,
for better or worse, our milestone and release planning is not great. We
normally decide on what we want to work on for the next release and release
new versions based on Fedora or RHEL releases (mostly because there is
normally no other driver..if there are other projects or distributions
who
URL: https://github.com/SSSD/sssd/pull/41
Title: #41: MAN: Wrong defaults for AD provider
mzidek-rh commented:
"""
I did not want to explicitly state all the providers so I first special cased
AD and then just said 'other providers'... however I do not think now it was a
good choice because th
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 pull requests run the CI test
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/pull/40#issuecomment-252214390
___
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/41
Title: #41: MAN: Wrong defaults for AD provider
mzidek-rh commented:
"""
Actually I do not think it was good to talk about providers, because this is
more schema thing. I will put the schema names instead of the providers.
"""
See the full comment at
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/pull/42#issuecomment-252215676
__
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/41
Author: mzidek-rh
Title: #41: MAN: Wrong defaults for AD provider
Action: synchronized
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
From f0d4aa6e3d63c0b660da993ff5
URL: https://github.com/SSSD/sssd/pull/41
Title: #41: MAN: Wrong defaults for AD provider
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/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
On 10/07/2016 12:20 PM, Jakub Hrozek wrote:
Hi,
for better or worse, our milestone and release planning is not great. We
normally decide on what we want to work on for the next release and release
new versions based on Fedora or RHEL releases (mostly because there is
normally no other driver..if
URL: https://github.com/SSSD/sssd/pull/43
Author: celestian
Title: #43: SUDO: Adding user name alias to sudoRule filter
Action: opened
PR body:
"""
This patch adds another value to sudoUser attribute of sudoRule filter.
The value is 'user alias' which means it is cased version of user
(in doma
URL: https://github.com/SSSD/sssd/pull/43
Title: #43: SUDO: Adding user name alias to sudoRule filter
Label: +Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedo
URL: https://github.com/SSSD/sssd/pull/43
Title: #43: SUDO: Adding user name alias to sudoRule filter
celestian commented:
"""
I see it, I forgot the tests.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/43#issuecomment-252249600
___
s
URL: https://github.com/SSSD/sssd/pull/43
Author: celestian
Title: #43: SUDO: Adding user name alias to sudoRule filter
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/43/head:pr43
git checkout pr43
From a1a754852f
URL: https://github.com/SSSD/sssd/pull/43
Title: #43: SUDO: Adding user name alias to sudoRule filter
celestian commented:
"""
Fixed patch pushed.
I will add patch for testing new functionality of sudo.
And I will add patch for caching both form of user name to sudoRule, if
@pbrezina and @jhroz
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
___
sssd-devel
URL: https://github.com/SSSD/sssd/pull/43
Title: #43: SUDO: Adding user name alias to sudoRule filter
celestian commented:
"""
PR #39 is for the same issue, but for version 1-13. And I need to change the
patch. I would like to have two PR because there are two different patches, one
for each v
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/pull/41#issuecomment-252264560
On Fri, Oct 7, 2016 at 12:20 PM, Jakub Hrozek wrote:
> Hi,
>
> for better or worse, our milestone and release planning is not great. We
> normally decide on what we want to work on for the next release and release
> new versions based on Fedora or RHEL releases (mostly because there is
> normally
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. I would like to have tw
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/pull/31#issuecomment-252280250
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: scope_hint: In function
'n
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_cache -u admin
# s
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
257b66a LDAP: Removing of m
URL: https://github.com/SSSD/sssd/pull/44
Author: sumit-bose
Title: #44: libwbclient-sssd: update interface to version 0.13
Action: opened
PR body:
"""
This patch adds wbcCtxUnixIdsToSids() and wbcUnixIdsToSids() to SSSD's
libwbclient and implements the latter.
Resolves https://fedorahosted.o
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: synchronized
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: -Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-deve
51 matches
Mail list logo
