mzidek-rh commented on a pull request
"""
The patches from PR are included in PR #18 . Labeling this one as rejected.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/16#issuecomment-245437017
___
sssd-devel mailing list
sssd-devel@lists.f
mzidek-rh's pull request #18: "[PATCHES] sss_user/groupmod fixes" label
*Pushed* has been added
See the full pull-request at https://github.com/SSSD/sssd/pull/18
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.o
mzidek-rh's pull request #16: "TOOLS: sss_groupshow did not work" label
*Rejected* has been added
See the full pull-request at https://github.com/SSSD/sssd/pull/16
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.
mzidek-rh's pull request #16: "TOOLS: sss_groupshow did not work" label
*Changes requested* has been removed
See the full pull-request at https://github.com/SSSD/sssd/pull/16
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fe
lslebodn commented on a pull request
"""
On (07/09/16 09:51), fidencio wrote:
>A few minors in the commit messages that should be fixed before pushing.
>
>Patch0001: TOOLS: Fix a typo in groupadd()
>Remove the four spaces before the link of the trac ticket.
>
>Patch0002: TOOLS: sss_groupshow did n
On Thu, Sep 01, 2016 at 02:02:40PM +0200, Pavel Březina wrote:
> https://fedorahosted.org/sssd/ticket/3131
>
> I couldn't reproduce manually so I used the second patch as a by-code
> reproducer. If you apply the patch then sssd will try to resolve meta server
> twice simultaneously and triggering
Hi,
sorry to come late, but I have one more request (last one, I promise..)
On Thu, Sep 01, 2016 at 09:36:32AM -0400, Justin Stephenson wrote:
> From f647e732c2a5b8727376dded962766fb03bb5ea8 Mon Sep 17 00:00:00 2001
> From: Justin Stephenson
> Date: Fri, 26 Aug 2016 15:15:32 -0400
> Subject: [PA
mzidek-rh's pull request #20: "sss_override fails to export" was opened
PR body:
"""
Here is a fix + CI test for https://fedorahosted.org/sssd/ticket/3179.
Michal
"""
See the full pull-request at https://github.com/SSSD/sssd/pull/20
... or pull the PR as Git branch:
git remote add ghsssd https:/
jhrozek commented on a pull request
"""
On Tue, Sep 06, 2016 at 06:09:58AM -0700, Jakub Hrozek wrote:
> good idea
ah, only when I started to implement this I realized it's already done :)
See:
https://github.com/SSSD/sssd/blob/master/src/providers/krb5/krb5_child.c#L1364
in the current mast
jhrozek's pull request #19: "KRB5: Send the output username, not internal
fqname to krb5_child" label *Accepted* has been added
See the full pull-request at https://github.com/SSSD/sssd/pull/19
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.o
fidencio commented on a pull request
"""
A few minors in the commit messages that should be fixed before pushing.
Patch0001: TOOLS: Fix a typo in groupadd()
Remove the four spaces before the link of the trac ticket.
Patch0002: TOOLS: sss_groupshow did not work
Patch0003: TESTS: sss_groupadd/grou
mzidek-rh's pull request #18: "[PATCHES] sss_user/groupmod fixes" was
synchronize
See the full pull-request at https://github.com/SSSD/sssd/pull/18
... or pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/18/head:pr18
git checkout pr18
From 0bcb9
lslebodn commented on a pull request
"""
On (07/09/16 08:49), Jakub Hrozek wrote:
>btw feel free to ping me on RH IRC for a link that shows the patch fixes the
>RH tests..
>
When I was looking into this bug
https://bugzilla.redhat.com/show_bug.cgi?id=1372753#c7
I verified in gdb that setting kr->
lslebodn commented on a pull request
"""
On (07/09/16 07:18), mzidek-rh wrote:
>Hi,
>
>This PR is blocked by PR#16 (TOOLS: sss_groupshow did not work). It includes
>fix for the ticket https://fedorahosted.org/sssd/ticket/3178.
>
>Michal
>You can view, comment on, or merge this pull request online
jhrozek commented on a pull request
"""
btw feel free to ping me on RH IRC for a link that shows the patch fixes the RH
tests..
"""
See the full comment at
https://github.com/SSSD/sssd/pull/19#issuecomment-245325705
___
sssd-devel mailing list
sssd-de
jhrozek's pull request #19: "KRB5: Send the output username, not internal
fqname to krb5_child" was opened
PR body:
"""
Resolves:
https://fedorahosted.org/sssd/ticket/3172
krb5_child calls krb5_kuserok() during the access phase which checks if
a particular user is allowed to authenticate as
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL
pointer broke sss_groupadd" was closed
See the full pull-request at https://github.com/SSSD/sssd/pull/14
... or pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/14/hea
mzidek-rh commented on a pull request
"""
Thanks for the tip Nick.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/18#issuecomment-245304406
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/admi
spbnick commented on a pull request
"""
Just a hint: if you avoid putting "PR" in front of the PR#16, then you'll get a
link to the actual pull request on the GitHub page, plus the target pull
request will have a link back. Like this: #16.
"""
See the full comment at
https://github.com/SSSD/ss
mzidek-rh's pull request #18: "[PATCHES] sss_user/groupmod fixes" was opened
PR body:
"""
Hi,
This PR is blocked by PR#16 (TOOLS: sss_groupshow did not work). It includes
fix for the ticket https://fedorahosted.org/sssd/ticket/3178.
Michal
"""
See the full pull-request at https://github.com/S
sumit-bose's pull request #17: "Improve support for gdm Smartcard support" was
opened
PR body:
"""
Those two patches try to fix two issues related to the Smartcard handling
feature of gdm. The first fixes https://fedorahosted.org/sssd/ticket/3165 and
the second fixes an issue which was introduced
On Wed, Sep 07, 2016 at 02:34:19PM +0300, Nikolai Kondrashov wrote:
> On 09/07/2016 02:18 PM, Sumit Bose wrote:
> > On Wed, Sep 07, 2016 at 01:28:12PM +0300, Nikolai Kondrashov wrote:
> > > Hi Sumit,
> > >
> > > Just wanted to tell you I still need an answer to the below.
> >
> > ah, sorry, I thi
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL
pointer broke sss_groupadd" label *rejected* has been added
See the full pull-request at https://github.com/SSSD/sssd/pull/14
___
sssd-devel mailing list
sssd-devel@lists.fed
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL
pointer broke sss_groupadd" label *rejected* has been removed
See the full pull-request at https://github.com/SSSD/sssd/pull/14
___
sssd-devel mailing list
sssd-devel@lists.f
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL
pointer broke sss_groupadd" label *Changes requested* has been removed
See the full pull-request at https://github.com/SSSD/sssd/pull/14
___
sssd-devel mailing list
sssd-deve
jhrozek's pull request #14: "Attempting to create a qualified name from a NULL
pointer broke sss_groupadd" label *rejected* has been added
See the full pull-request at https://github.com/SSSD/sssd/pull/14
___
sssd-devel mailing list
sssd-devel@lists.fed
jhrozek commented on a pull request
"""
On Tue, Sep 06, 2016 at 10:23:13AM -0700, mzidek-rh wrote:
> The PR#16 includes jhrozek's patch from this PR as well as CI test + fix for
> sss_groupshow.
OK, thank you, I will close this one as rejected (just so that we don't
track the PR any longer..)
"
On 08/29/2016 11:47 PM, Sumit Bose wrote:
Finally you call sysdb_initgroups_with_views() to get the list of groups
the user is a member of and compare them with the groups form the
session_recording configuration. Since you compare the DNs I think this
can be improved a bit. sysdb_initgroups_with
On 09/07/2016 02:18 PM, Sumit Bose wrote:
On Wed, Sep 07, 2016 at 01:28:12PM +0300, Nikolai Kondrashov wrote:
Hi Sumit,
Just wanted to tell you I still need an answer to the below.
ah, sorry, I think I missed this question while discussing the group
lookups with Simo in the other thread.
No
On 09/07/2016 01:48 PM, Pavel Březina wrote:
On 08/19/2016 06:39 PM, Nikolai Kondrashov wrote:
Hi Sumit,
Now I'm again approaching the implementation of tlog integration in
pam_sss,
and as planned, I need to get the actual user shell to put it into
TLOG_REC_SHELL environment variable upon openi
On Wed, Sep 07, 2016 at 01:28:12PM +0300, Nikolai Kondrashov wrote:
> Hi Sumit,
>
> Just wanted to tell you I still need an answer to the below.
ah, sorry, I think I missed this question while discussing the group
lookups with Simo in the other thread.
>
> Thanks!
>
> Nick
>
> On 08/19/2016 0
On 08/19/2016 06:39 PM, Nikolai Kondrashov wrote:
Hi Sumit,
Now I'm again approaching the implementation of tlog integration in
pam_sss,
and as planned, I need to get the actual user shell to put it into
TLOG_REC_SHELL environment variable upon opening of the session.
However, the get_shell_ove
Hi Simo,
On 08/29/2016 11:47 PM, Sumit Bose wrote:
Then you do a normal user lookup but if there are groups configured in
the session_recording section you always do a SSS_DP_INITGROUPS instead
of a SSS_DP_USER if the entry is expired. I think here you can add some
improvement to reduce the numb
Hi Sumit,
Just wanted to tell you I still need an answer to the below.
Thanks!
Nick
On 08/19/2016 07:39 PM, Nikolai Kondrashov wrote:
Now I'm again approaching the implementation of tlog integration in pam_sss,
and as planned, I need to get the actual user shell to put it into
TLOG_REC_SHELL
Hi folks,
while converting my workflow scripts to github [1] I decided to polish
them, give them some help and publish them on github. Feel free to use
them and improve them as you wish.
I'll gladly learn about your workflow and simplification.
[1] https://github.com/pbrezina/sssd-dev-utils
_
On 09/07/2016 09:53 AM, Jakub Hrozek wrote:
On Wed, Sep 07, 2016 at 08:45:18AM +0200, Lukas Slebodnik wrote:
On (05/09/16 16:07), Jakub Hrozek wrote:
On Mon, Sep 05, 2016 at 03:32:48PM +0200, Lukas Slebodnik wrote:
On (05/09/16 15:24), Jakub Hrozek wrote:
On Mon, Sep 05, 2016 at 02:31:31PM +0
mzidek-rh's pull request #16: "TOOLS: sss_groupshow did not work" was
synchronize
See the full pull-request at https://github.com/SSSD/sssd/pull/16
... or pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/16/head:pr16
git checkout pr16
From 0bcb9c
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 sugges
On Wed, Sep 07, 2016 at 08:45:18AM +0200, Lukas Slebodnik wrote:
> On (05/09/16 16:07), Jakub Hrozek wrote:
> >On Mon, Sep 05, 2016 at 03:32:48PM +0200, Lukas Slebodnik wrote:
> >> On (05/09/16 15:24), Jakub Hrozek wrote:
> >> >On Mon, Sep 05, 2016 at 02:31:31PM +0200, Fabiano Fidêncio wrote:
> >>
On Wed, Sep 7, 2016 at 9:03 AM, Lukas Slebodnik wrote:
> On (07/09/16 08:46), Fabiano Fidêncio wrote:
>>On Wed, Sep 7, 2016 at 8:34 AM, Lukas Slebodnik wrote:
>>> On (06/09/16 21:38), Fabiano Fidêncio wrote:
On Tue, Sep 6, 2016 at 8:49 PM, lslebodn
wrote:
> lslebodn commented on a pu
On (07/09/16 08:46), Fabiano Fidêncio wrote:
>On Wed, Sep 7, 2016 at 8:34 AM, Lukas Slebodnik wrote:
>> On (06/09/16 21:38), Fabiano Fidêncio wrote:
>>>On Tue, Sep 6, 2016 at 8:49 PM, lslebodn
>>> wrote:
lslebodn commented on a pull request
"""
IMHO, it might be better to close
41 matches
Mail list logo
