URL: https://github.com/SSSD/sssd/pull/246
Title: #246: filter_users and filter_groups stop working properly in v 1.15
fidencio commented:
"""
New patch set pushed and in the end the changes were not intrusive at all.
@pbrezina, I've partially taken your idea (with some changes).
Please, let
URL: https://github.com/SSSD/sssd/pull/246
Author: fidencio
Title: #246: filter_users and filter_groups stop working properly in v 1.15
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/246/head:pr246
git checkout pr246
URL: https://github.com/SSSD/sssd/pull/248
Author: justin-stephenson
Title: #248: IPA: Improve s2n debug message for missing ipaNTSecurityIdentifier
Action: edited
Changed field: body
Original value:
"""
This patch is piggyback
This patch improves the log message to be more information for
URL: https://github.com/SSSD/sssd/pull/248
Title: #248: IPA: Improve s2n debug message for missing ipaNTSecurityIdentifier
centos-ci commented:
"""
Can one of the admins verify this patch?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/248#issuecomment-297100901
URL: https://github.com/SSSD/sssd/pull/248
Title: #248: IPA: Improve s2n debug message for missing ipaNTSecurityIdentifier
centos-ci commented:
"""
Can one of the admins verify this patch?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/248#issuecomment-297100893
URL: https://github.com/SSSD/sssd/pull/248
Author: justin-stephenson
Title: #248: IPA: Improve s2n debug message for missing ipaNTSecurityIdentifier
Action: opened
PR body:
"""
This patch is piggyback
This patch improves the log message to be more information for
the SSSD user
URL: https://github.com/SSSD/sssd/pull/136
Title: #136: Tlog integration
spbnick commented:
"""
BTW, should I perhaps include an update to the sssd.conf man page?
"""
See the full comment at
https://github.com/SSSD/sssd/pull/136#issuecomment-297017382
URL: https://github.com/SSSD/sssd/pull/136
Title: #136: Tlog integration
spbnick commented:
"""
Hi Pavel, thank you for your review! I'll be addressing your comments soon, but
for now here is how to test this.
The patches add support for a new section in sssd.conf: `session_recording`.
The
URL: https://github.com/SSSD/sssd/pull/235
Title: #235: Allow using the "shortnames" feature without requiring any
configuration from the client side
fidencio commented:
"""
On Tue, Apr 25, 2017 at 1:56 PM, Pavel Březina
wrote:
> Yes, I think we should log it so we
URL: https://github.com/SSSD/sssd/pull/235
Author: fidencio
Title: #235: Allow using the "shortnames" feature without requiring any
configuration from the client side
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd
URL: https://github.com/SSSD/sssd/pull/235
Title: #235: Allow using the "shortnames" feature without requiring any
configuration from the client side
pbrezina commented:
"""
Yes, I think we should log it so we know what's used when we will debug issues
:-)
"""
See the full comment at
URL: https://github.com/SSSD/sssd/pull/136
Title: #136: Tlog integration
pbrezina commented:
"""
Hi, here are some comments. Mostly nitpicks.
Can you also share some link how to enable tlog and test this code?
* **CACHE_REQ: Rename search_domains_done to search_domains_next_done```**
As far
URL: https://github.com/SSSD/sssd/pull/246
Title: #246: filter_users and filter_groups stop working properly in v 1.15
lslebodn commented:
"""
On (25/04/17 03:59), fidencio wrote:
>On Tue, Apr 25, 2017 at 12:16 PM, Pavel Březina
>wrote:
>
>> Hi, we should solve this
URL: https://github.com/SSSD/sssd/pull/246
Title: #246: filter_users and filter_groups stop working properly in v 1.15
fidencio commented:
"""
On Tue, Apr 25, 2017 at 12:16 PM, Pavel Březina
wrote:
> Hi, we should solve this on cache_req level so we get the same
URL: https://github.com/SSSD/sssd/pull/235
Title: #235: Allow using the "shortnames" feature without requiring any
configuration from the client side
fidencio commented:
"""
On Tue, Apr 25, 2017 at 12:37 PM, Pavel Březina
wrote:
> Thank you. Last trivial change, you
URL: https://github.com/SSSD/sssd/pull/235
Title: #235: Allow using the "shortnames" feature without requiring any
configuration from the client side
pbrezina commented:
"""
Thank you. Last trivial change, you have typo in commit message (domaiN):
`RESPONDER_COMMON: Improve
URL: https://github.com/SSSD/sssd/pull/247
Title: #247: Subdomain inherit
lslebodn commented:
"""
This patch set modified the function `check_subdom_config_file` which is
called from the function `new_subdomain`. And this function is already tested
in unitests.
```
sh$ git grep
URL: https://github.com/SSSD/sssd/pull/246
Title: #246: filter_users and filter_groups stop working properly in v 1.15
pbrezina commented:
"""
This is really from top of my mind, there may be another, easier way to do it.
But we should do it on cache_req level.
"""
See the full comment at
URL: https://github.com/SSSD/sssd/pull/246
Title: #246: filter_users and filter_groups stop working properly in v 1.15
pbrezina commented:
"""
Hi, we should solve this on `cache_req` level so we get the same resultt in
`nss` and `ifp` (and others) responders. What I had in mind was to create
URL: https://github.com/SSSD/sssd/pull/247
Title: #247: Subdomain inherit
lslebodn commented:
"""
This patch set modified the function `check_subdom_config_file` which is
called from the function `new_subdomain`. And this function is already tested
in unitests.
```
sh$ git grep
URL: https://github.com/SSSD/sssd/pull/247
Title: #247: Subdomain inherit
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/247
Author: mzidek-rh
Title: #247: Subdomain inherit
Action: opened
PR body:
"""
I tested if the options that work in subdomain inherit also work in trusted
domain section in sssd.conf. Most seem to work without any changes in the code
except for two.
22 matches
Mail list logo