On 08/31/2016 10:28 AM, Jakub Hrozek wrote:
Hi,
I documented workflow that we could use for submitting PRs:
https://fedorahosted.org/sssd/wiki/GithubWorkflow
It's quite similar to what the FreeIPA team uses (although I don't think
they publicly document it yet).
Comments or edits welcome.
On 08/31/2016 07:49 PM, Stephen Gallagher wrote:
On 08/31/2016 01:24 PM, Simo Sorce wrote:
On Wed, 2016-08-31 at 17:41 +0200, Michal Židek wrote:
Hi,
here is patch for ticket #3161.
See more in the ticket description.
I was thinking why we originally replaced
the lists and I think it comes f
fidencio's pull request #10: "UTIL: Remove support to libini older than 1.0.0"
was synchronize
See the full pull-request at https://github.com/SSSD/sssd/pull/10
... or pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/10/head:pr10
git checkout pr1
On Thu, Sep 01, 2016 at 09:49:50AM +0200, Petr Cech wrote:
> On 08/31/2016 10:28 AM, Jakub Hrozek wrote:
> > Hi,
> >
> > I documented workflow that we could use for submitting PRs:
> > https://fedorahosted.org/sssd/wiki/GithubWorkflow
> >
> > It's quite similar to what the FreeIPA team uses (
On Thu, Sep 01, 2016 at 09:56:33AM +0200, Michal Židek wrote:
> On 08/31/2016 07:49 PM, Stephen Gallagher wrote:
> > On 08/31/2016 01:24 PM, Simo Sorce wrote:
> > > On Wed, 2016-08-31 at 17:41 +0200, Michal Židek wrote:
> > > > Hi,
> > > >
> > > > here is patch for ticket #3161.
> > > >
> > > > S
On 09/01/2016 10:31 AM, Jakub Hrozek wrote:
On Thu, Sep 01, 2016 at 09:56:33AM +0200, Michal Židek wrote:
On 08/31/2016 07:49 PM, Stephen Gallagher wrote:
On 08/31/2016 01:24 PM, Simo Sorce wrote:
On Wed, 2016-08-31 at 17:41 +0200, Michal Židek wrote:
Hi,
here is patch for ticket #3161.
See
On Thu, 01 Sep 2016, Michal Židek wrote:
On 09/01/2016 10:31 AM, Jakub Hrozek wrote:
On Thu, Sep 01, 2016 at 09:56:33AM +0200, Michal Židek wrote:
On 08/31/2016 07:49 PM, Stephen Gallagher wrote:
On 08/31/2016 01:24 PM, Simo Sorce wrote:
On Wed, 2016-08-31 at 17:41 +0200, Michal Židek wrote:
fidencio's pull request #10: "UTIL: Remove support to libini older than 1.0.0"
was edited
See the full pull-request at https://github.com/SSSD/sssd/pull/10
... or pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/10/head:pr10
git checkout pr10
___
On 08/31/2016 10:16 PM, Jakub Hrozek wrote:
On Wed, Aug 31, 2016 at 07:40:32PM +0200, Fabiano Fidêncio wrote:
Howdy!
Taking a look on https://fedorahosted.org/sssd/ticket/2395 seems that
there are a few ways to achieve what's proposed by Simo and I'd like
to discuss one of those before start im
On (01/09/16 10:30), Jakub Hrozek wrote:
>On Thu, Sep 01, 2016 at 09:49:50AM +0200, Petr Cech wrote:
>> On 08/31/2016 10:28 AM, Jakub Hrozek wrote:
>> > Hi,
>> >
>> > I documented workflow that we could use for submitting PRs:
>> > https://fedorahosted.org/sssd/wiki/GithubWorkflow
>> >
>> > I
On Thu, Sep 01, 2016 at 12:10:15PM +0200, Lukas Slebodnik wrote:
> On (01/09/16 10:30), Jakub Hrozek wrote:
> >On Thu, Sep 01, 2016 at 09:49:50AM +0200, Petr Cech wrote:
> >> On 08/31/2016 10:28 AM, Jakub Hrozek wrote:
> >> > Hi,
> >> >
> >> > I documented workflow that we could use for submitting
On (01/09/16 08:23), Fabiano Fidêncio wrote:
>On Thu, Sep 1, 2016 at 8:16 AM, Lukas Slebodnik wrote:
>> ehlo,
>>
>> @see
>> (Wed Aug 31 14:12:18 2016) [[sssd[selinux_child[7088 [libsemanage]
>> (0x0020): could not query record value(Wed Aug 31 14:12:18 2016)
>> [[sssd[selinux_child[7088
On (01/09/16 12:29), Jakub Hrozek wrote:
>On Thu, Sep 01, 2016 at 12:10:15PM +0200, Lukas Slebodnik wrote:
>> On (01/09/16 10:30), Jakub Hrozek wrote:
>> >On Thu, Sep 01, 2016 at 09:49:50AM +0200, Petr Cech wrote:
>> >> On 08/31/2016 10:28 AM, Jakub Hrozek wrote:
>> >> > Hi,
>> >> >
>> >> > I docu
Hi,
see the attached ding-libs patch for ticket #3166.
This is how sssctl config-check prints the merging
issues without this patch:
Messages generated during configuration merging: 2
File blaa did not match provided patterns. Skipping.
File /etc/sssd/conf.d/blaa.conf did not pass access check.
jhrozek commented on a pull request
"""
CI: http://sssd-ci.duckdns.org/logs/job/52/87/summary.html
"""
See the full comment at
https://github.com/SSSD/sssd/pull/8#issuecomment-244052362
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
http
jhrozek commented on a pull request
"""
On Wed, Aug 31, 2016 at 07:15:10AM -0700, lslebodn wrote:
> On (31/08/16 01:47), sumit-bose wrote:
> >On Wed, Aug 31, 2016 at 01:30:12AM -0700, Jakub Hrozek wrote:
> >> On Wed, Aug 31, 2016 at 12:36:37AM -0700, sumit-bose wrote:
> >> > On Tue, Aug 30, 2016 a
jhrozek's pull request #8: "MAN: Document the ldap_user_gid_number option" was
closed
See the full pull-request at https://github.com/SSSD/sssd/pull/8
... or pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/8/head:pr8
git checkout pr8
___
jhrozek commented on a pull request
"""
* master: 6f59bb822d1e54e178207be45e382f4ee173c434
"""
See the full comment at
https://github.com/SSSD/sssd/pull/8#issuecomment-244054299
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://list
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 the problematic code path.
You can search for RESOLV SRV DONE in logs
jhrozek commented on a pull request
"""
* master: 5bd3bef4a655fdfacd2f5df8a2343fe7bc68a771
"""
See the full comment at
https://github.com/SSSD/sssd/pull/7#issuecomment-244058146
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://list
sumit-bose's pull request #7: "Fix initgroups with nested groups" was closed
See the full pull-request at https://github.com/SSSD/sssd/pull/7
... or pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/7/head:pr7
git checkout pr7
_
jhrozek's pull request #8: "MAN: Document the ldap_user_gid_number option"
label *pushed* has been added
See the full pull-request at https://github.com/SSSD/sssd/pull/8
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorah
sumit-bose's pull request #7: "Fix initgroups with nested groups" label
*pushed* has been added
See the full pull-request at https://github.com/SSSD/sssd/pull/7
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org
celestian's pull request #9: "SYSDB: Removing of unused parametr" label
*pushed* has been added
See the full pull-request at https://github.com/SSSD/sssd/pull/9
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org
jhrozek commented on a pull request
"""
* master: 0d1d9d8001232f74eca63cbba6c400d507b33823
"""
See the full comment at
https://github.com/SSSD/sssd/pull/9#issuecomment-244058945
___
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://list
celestian's pull request #9: "SYSDB: Removing of unused parametr" was closed
See the full pull-request at https://github.com/SSSD/sssd/pull/9
... or pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/9/head:pr9
git checkout pr9
_
On (01/09/16 13:15), Michal Židek wrote:
>Hi,
>
>see the attached ding-libs patch for ticket #3166.
>
>This is how sssctl config-check prints the merging
>issues without this patch:
>
>Messages generated during configuration merging: 2
>File blaa did not match provided patterns. Skipping.
>File /et
On 09/01/2016 02:13 PM, Lukas Slebodnik wrote:
On (01/09/16 13:15), Michal Židek wrote:
Hi,
see the attached ding-libs patch for ticket #3166.
This is how sssctl config-check prints the merging
issues without this patch:
Messages generated during configuration merging: 2
File blaa did not mat
On (01/09/16 14:28), Michal Židek wrote:
>On 09/01/2016 02:13 PM, Lukas Slebodnik wrote:
>> On (01/09/16 13:15), Michal Židek wrote:
>> > Hi,
>> >
>> > see the attached ding-libs patch for ticket #3166.
>> >
>> > This is how sssctl config-check prints the merging
>> > issues without this patch:
>
On (26/08/16 13:16), Victor Tapia wrote:
>Hi,
>
>Sorry for the late response, I've been out for nearly two weeks. I'm
>attaching a new version of the patch to review.
>
>> Does upstart/init script need to be updated as well?
>> We have some generic in src/sysv/sssd.in
>
>Upstart does need to keep t
On 08/31/2016 01:40 PM, Fabiano Fidêncio wrote:
> Howdy!
>
> Taking a look on https://fedorahosted.org/sssd/ticket/2395 seems that
> there are a few ways to achieve what's proposed by Simo and I'd like
> to discuss one of those before start implementing it.
>
> As far as I understand the bug, SSS
On 08/30/2016 03:54 AM, Jakub Hrozek wrote:
On Sat, Aug 27, 2016 at 12:54:53PM -0400, Justin Stephenson wrote:
Hello,
The attached patches resolve https://fedorahosted.org/sssd/ticket/3142
However, I am having difficult with the man page addition to
'src/man/sssd.conf.5.xml' for this new optio
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 check
On 09/01/2016 02:32 PM, Lukas Slebodnik wrote:
On (01/09/16 14:28), Michal Židek wrote:
On 09/01/2016 02:13 PM, Lukas Slebodnik wrote:
On (01/09/16 13:15), Michal Židek wrote:
Hi,
see the attached ding-libs patch for ticket #3166.
This is how sssctl config-check prints the merging
issues wit
On (30/08/16 17:07), Lukas Slebodnik wrote:
>On (30/08/16 16:59), Fabiano Fidêncio wrote:
>>Lukaš,
>>
>>On Tue, Aug 30, 2016 at 4:54 PM, Lukas Slebodnik wrote:
>>> ehlo,
>>>
>>> Clang static analyzer assume that ldb_search can found
>>> 0 entries in the tree "cn=sysdb". Thenvariable version
>>> co
On Thu, Sep 1, 2016 at 5:12 PM, Lukas Slebodnik wrote:
> On (30/08/16 17:07), Lukas Slebodnik wrote:
>>On (30/08/16 16:59), Fabiano Fidêncio wrote:
>>>Lukaš,
>>>
>>>On Tue, Aug 30, 2016 at 4:54 PM, Lukas Slebodnik wrote:
ehlo,
Clang static analyzer assume that ldb_search can found
Hello,
I do not like either of the versions of the patch.
It is OK to use path_concat instead of snprintf. The whole point of not
using it was to simplify the code and not have to check yet another
error clause. But using path_concat is fine.
The thing that I do not like is that in the current cod
On 09/01/2016 05:26 PM, Dmitri Pal wrote:
Hello,
I do not like either of the versions of the patch.
It is OK to use path_concat instead of snprintf. The whole point of not
using it was to simplify the code and not have to check yet another
error clause. But using path_concat is fine.
The thing t
lslebodn commented on a pull request
"""
On (01/09/16 08:35), fidencio wrote:
>On Thu, Sep 1, 2016 at 2:10 PM, lslebodn 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
>> wrote:
>>
On (01/09/16 17:22), Fabiano Fidêncio wrote:
>On Thu, Sep 1, 2016 at 5:12 PM, Lukas Slebodnik wrote:
>> On (30/08/16 17:07), Lukas Slebodnik wrote:
>>>On (30/08/16 16:59), Fabiano Fidêncio wrote:
Lukaš,
On Tue, Aug 30, 2016 at 4:54 PM, Lukas Slebodnik
wrote:
> ehlo,
>
>>
On Thu, Sep 1, 2016 at 6:02 PM, Lukas Slebodnik wrote:
> On (01/09/16 17:22), Fabiano Fidêncio wrote:
>>On Thu, Sep 1, 2016 at 5:12 PM, Lukas Slebodnik wrote:
>>> On (30/08/16 17:07), Lukas Slebodnik wrote:
On (30/08/16 16:59), Fabiano Fidêncio wrote:
>Lukaš,
>
>On Tue, Aug 30, 20
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 check
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 p
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 p
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 check
45 matches
Mail list logo