URL: https://github.com/SSSD/sssd/pull/5766
Title: #5766: Sources cleanup - part 3.
alexey-tikhonov commented:
"""
> `test_kcm_renewals` is broken.
This PR built without last patch fails this test the same way. Built without 2
last patches - `test_kcm.py::test_kcm_renewals PASSED`.
So looks
URL: https://github.com/SSSD/sssd/pull/5712
Title: #5712: Health and Support Analyzer - Add request log parsing utility
justin-stephenson commented:
"""
Latest push includes change to declare Reader as an abstract base class with
required methods, source Files and Journald classes are
URL: https://github.com/SSSD/sssd/pull/5712
Title: #5712: Health and Support Analyzer - Add request log parsing utility
justin-stephenson commented:
"""
Latest push includes change to declare Reader as an abstract base class with
required methods, Files() and Journald() readers are subclassing
URL: https://github.com/SSSD/sssd/pull/5712
Author: justin-stephenson
Title: #5712: Health and Support Analyzer - Add request log parsing utility
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5712/head:pr5712
git
URL: https://github.com/SSSD/sssd/pull/5817
Author: shridhargadekar
Title: #5817: Tests: pam_sss_gss.so doesn't work with large kerberos tickets
#5815
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5817/head:pr5817
git
URL: https://github.com/SSSD/sssd/pull/5803
Title: #5803: CONF: removed unused 'sbus_timeout' option
Label: -Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5803
Title: #5803: CONF: removed unused 'sbus_timeout' option
Label: +Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5803
Title: #5803: CONF: removed unused 'sbus_timeout' option
ikerexxe commented:
"""
> But the options was NOT available. I mean, putting this option in sssd.conf
> did NOT have any effect. This is merely removal of code that did nothing, so
> I don't
URL: https://github.com/SSSD/sssd/pull/5712
Title: #5712: Health and Support Analyzer - Add request log parsing utility
justin-stephenson commented:
"""
> Can't run:
>
> ```
> [pbrezina /dev/shm/sssd]$ sudo sssctl analyze
> Traceback (most recent call last):
> File
URL: https://github.com/SSSD/sssd/pull/5712
Author: justin-stephenson
Title: #5712: Health and Support Analyzer - Add request log parsing utility
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5712/head:pr5712
git
URL: https://github.com/SSSD/sssd/pull/5816
Title: #5816: debug: fix unused variable warnings
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/5816
Title: #5816: debug: fix unused variable warnings
alexey-tikhonov commented:
"""
Thank you, ACK.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5816#issuecomment-936282402
___
sssd-devel
URL: https://github.com/SSSD/sssd/pull/5803
Title: #5803: CONF: removed unused 'sbus_timeout' option
alexey-tikhonov commented:
"""
> The changes look good to me but I think that you should link an upstream
> ticket both in the commit message and in the PR description.
>
> Moreover, I wonder
URL: https://github.com/SSSD/sssd/pull/5784
Title: #5784: proxy: allow removing group members
Label: -Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5784
Title: #5784: proxy: allow removing group members
Label: +Waiting for review
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5784
Title: #5784: proxy: allow removing group members
ikerexxe commented:
"""
> Wouldn't it make sense to instead lookup GID of SYSDB_MEMBEROF DN and compare
> GIDs (numerical values)?
Yes, so I've updated the PR accordingly. Nowadays, a list of GIDs is
URL: https://github.com/SSSD/sssd/pull/5784
Author: ikerexxe
Title: #5784: proxy: allow removing group members
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5784/head:pr5784
git checkout pr5784
From
URL: https://github.com/SSSD/sssd/pull/5803
Title: #5803: CONF: removed unused 'sbus_timeout' option
Label: +Changes requested
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5803
Title: #5803: CONF: removed unused 'sbus_timeout' option
Label: -Waiting for review
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5803
Title: #5803: CONF: removed unused 'sbus_timeout' option
ikerexxe commented:
"""
The changes look good to me but I think that you should link an upstream ticket
both in the commit message and in the PR description.
Moreover, I wonder if we should
URL: https://github.com/SSSD/sssd/pull/5712
Title: #5712: Health and Support Analyzer - Add request log parsing utility
pbrezina commented:
"""
Can't run:
```
[pbrezina /dev/shm/sssd]$ sudo sssctl analyze
Traceback (most recent call last):
File
URL: https://github.com/SSSD/sssd/pull/5817
Title: #5817: Tests: pam_sss_gss.so doesn't work with large kerberos tickets
#5815
Label: +Tests
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5817
Author: shridhargadekar
Title: #5817: Tests: pam_sss_gss.so doesn't work with large kerberos tickets
#5815
Action: opened
PR body:
"""
Verifies: #5568
Bug: https://bugzilla.redhat.com/show_bug.cgi?id=1948657
"""
To pull the PR as Git branch:
git
URL: https://github.com/SSSD/sssd/pull/5815
Author: shridhargadekar
Title: #5815: Tests: pam_sss_gss.so doesn't work with large kerberos tickets
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5815/head:pr5815
git checkout
URL: https://github.com/SSSD/sssd/pull/5816
Author: pbrezina
Title: #5816: debug: fix unused variable warnings
Action: opened
PR body:
"""
```
/home/pbrezina/workspace/sssd/src/util/debug.c: In function ‘sss_vdebug_fn’:
/home/pbrezina/workspace/sssd/src/util/debug.c:274:11: error: unused
URL: https://github.com/SSSD/sssd/pull/5815
Author: shridhargadekar
Title: #5815: Tests: pam_sss_gss.so doesn't work with large kerberos tickets
Action: opened
PR body:
"""
Verifies: #5568
Bug: https://bugzilla.redhat.com/show_bug.cgi?id=1948657
"""
To pull the PR as Git branch:
git remote
URL: https://github.com/SSSD/sssd/pull/5814
Author: aborah-sudo
Title: #5814: Tests: sss_override does not take precedence over
override_homedir di…
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5814/head:pr5814
URL: https://github.com/SSSD/sssd/pull/5814
Author: aborah-sudo
Title: #5814: Tests: sss_override does not take precedence over
override_homedir di…
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5814/head:pr5814
URL: https://github.com/SSSD/sssd/pull/5814
Title: #5814: Tests: sss_override does not take precedence over
override_homedir di…
Label: +Tests
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5814
Author: aborah-sudo
Title: #5814: Tests: sss_override does not take precedence over
override_homedir di…
Action: opened
PR body:
"""
…rective
bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1919942
"""
To pull the PR as Git branch:
git
URL: https://github.com/SSSD/sssd/pull/5813
Title: #5813: Tests: files provider automation
Label: +Tests
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora
URL: https://github.com/SSSD/sssd/pull/5813
Author: shridhargadekar
Title: #5813: Tests: files provider automation
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5813/head:pr5813
git checkout pr5813
From
URL: https://github.com/SSSD/sssd/pull/5813
Author: shridhargadekar
Title: #5813: Tests: files provider automation
Action: opened
PR body:
"""
verifies: #4295
"""
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5813/head:pr5813
git
URL: https://github.com/SSSD/sssd/pull/5812
Title: #5812: Tests: Modify ported AD tests marker to tier1_2 and fix a restore
error
Label: +Tests
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5812
Title: #5812: Tests: Modify ported AD tests marker to tier1_2 and fix a restore
error
Label: +Waiting for review
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email
URL: https://github.com/SSSD/sssd/pull/5766
Title: #5766: Sources cleanup - part 3.
alexey-tikhonov commented:
"""
`test_kcm_renewals` is broken.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5766#issuecomment-935981231
___
URL: https://github.com/SSSD/sssd/pull/5766
Title: #5766: Sources cleanup - part 3.
Label: -Waiting for review
___
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/5766
Title: #5766: Sources cleanup - part 3.
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/5801
Author: alexey-tikhonov
Title: #5801: util/inotify: fixed bug in inotify event processing - 1.16
backport
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5801/head:pr5801
git
URL: https://github.com/SSSD/sssd/pull/5801
Title: #5801: util/inotify: fixed bug in inotify event processing - 1.16
backport
Label: -Ready to push
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5801
Title: #5801: util/inotify: fixed bug in inotify event processing - 1.16
backport
pbrezina commented:
"""
Pushed PR: https://github.com/SSSD/sssd/pull/5801
* `sssd-1-16`
* 758b99590a8e1f69b4487fad9cf343525797e05f - util/inotify: fixed bug in
URL: https://github.com/SSSD/sssd/pull/5801
Title: #5801: util/inotify: fixed bug in inotify event processing - 1.16
backport
Label: +Ready to push
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5801
Title: #5801: util/inotify: fixed bug in inotify event processing - 1.16
backport
Label: +Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5801
Title: #5801: util/inotify: fixed bug in inotify event processing - 1.16
backport
Label: -Waiting for review
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5801
Title: #5801: util/inotify: fixed bug in inotify event processing - 1.16
backport
thalman commented:
"""
Thank you for the patch, LGTM, ACK
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5801#issuecomment-935825772
URL: https://github.com/SSSD/sssd/pull/5809
Author: justin-stephenson
Title: #5809: PR CI Test - Do not merge
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5809/head:pr5809
git checkout pr5809
URL: https://github.com/SSSD/sssd/pull/5809
Title: #5809: PR CI Test - Do not merge
pbrezina commented:
"""
Works now. Closing.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5809#issuecomment-935801507
___
sssd-devel mailing list --
URL: https://github.com/SSSD/sssd/pull/5722
Author: grawity
Title: #5722: NSS client: avoid using NETDB_INTERNAL if daemon is not available
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5722/head:pr5722
git checkout
URL: https://github.com/SSSD/sssd/pull/5722
Title: #5722: NSS client: avoid using NETDB_INTERNAL if daemon is not available
Label: -Ready to push
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5722
Title: #5722: NSS client: avoid using NETDB_INTERNAL if daemon is not available
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5722
Title: #5722: NSS client: avoid using NETDB_INTERNAL if daemon is not available
Label: +Pushed
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5722
Title: #5722: NSS client: avoid using NETDB_INTERNAL if daemon is not available
pbrezina commented:
"""
Pushed PR: https://github.com/SSSD/sssd/pull/5722
* `master`
* 1a1e914b95c6415533f318f32da58a04015fa912 - NSS client: avoid using
URL: https://github.com/SSSD/sssd/pull/5795
Title: #5795: debug: Add chain ID support for journald logger
Label: -Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5795
Author: justin-stephenson
Title: #5795: debug: Add chain ID support for journald logger
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5795/head:pr5795
git checkout pr5795
URL: https://github.com/SSSD/sssd/pull/5795
Title: #5795: debug: Add chain ID support for journald logger
Label: -Ready to push
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5795
Title: #5795: debug: Add chain ID support for journald logger
pbrezina commented:
"""
Pushed PR: https://github.com/SSSD/sssd/pull/5795
* `master`
* 3d8dd1282ffb7d0188e36d0109340ce622745717 - debug: Add chain ID support
for journald logger
"""
URL: https://github.com/SSSD/sssd/pull/5804
Author: aborah-sudo
Title: #5804: Tests: Fix Failure of sssctl_local test
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5804/head:pr5804
git checkout pr5804
URL: https://github.com/SSSD/sssd/pull/5804
Title: #5804: Tests: Fix Failure of sssctl_local test
Label: -Ready to push
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5804
Title: #5804: Tests: Fix Failure of sssctl_local test
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/5804
Title: #5804: Tests: Fix Failure of sssctl_local test
pbrezina commented:
"""
Pushed PR: https://github.com/SSSD/sssd/pull/5804
* `master`
* 51eaed9d0627465ee90142dd092816abb84db3e9 - Tests: Fix Failure of
sssctl_local test
"""
See the full
URL: https://github.com/SSSD/sssd/pull/5804
Title: #5804: Tests: Fix Failure of sssctl_local test
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/5806
Author: shridhargadekar
Title: #5806: Tests: files provider testcases
Action: closed
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5806/head:pr5806
git checkout pr5806
URL: https://github.com/SSSD/sssd/pull/5806
Title: #5806: Tests: files provider testcases
shridhargadekar commented:
"""
Code will be rearranged in the alltests instead of a separate dir.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5806#issuecomment-935790326
URL: https://github.com/SSSD/sssd/pull/5722
Title: #5722: NSS client: avoid using NETDB_INTERNAL if daemon is not available
Label: +Ready to push
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5795
Title: #5795: debug: Add chain ID support for journald logger
Label: +Ready to push
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5804
Title: #5804: Tests: Fix Failure of sssctl_local test
Label: +Ready to push
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5812
Author: sgoveas
Title: #5812: Tests: Modify ported AD tests marker to tier1_2 and fix a
restore error
Action: opened
PR body:
"""
Use tier1_2 marker for newly ported ad parameters tests to reduce runtime to 1
hour and fix and ldap.conf backup
URL: https://github.com/SSSD/sssd/pull/5808
Title: #5808: Tests: Remove shadow-utils test cases from sssd repo
aborah-sudo commented:
"""
> I think you shouldn't have merged the PR. @pbrezina has a script that does it
> in a fancy way.
By mistake its was merged by me .
"""
See the full
URL: https://github.com/SSSD/sssd/pull/5808
Title: #5808: Tests: Remove shadow-utils test cases from sssd repo
ikerexxe commented:
"""
I think you shouldn't have merged the PR. @pbrezina has a script that does it
in a fancy way.
"""
See the full comment at
*WARNING: this pull request has been merged!*
This is only mirrored repo thus any changes will be erased. Please push
commit(s) to authoritative repository.
URL: https://github.com/SSSD/sssd/pull/5808
Author: aborah-sudo
Title: #5808: Tests: Remove shadow-utils test cases from sssd repo
URL: https://github.com/SSSD/sssd/pull/5808
Title: #5808: Tests: Remove shadow-utils test cases from sssd repo
Label: +Accepted
___
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to
URL: https://github.com/SSSD/sssd/pull/5808
Title: #5808: Tests: Remove shadow-utils test cases from sssd repo
ikerexxe commented:
"""
LGTM. Thanks for moving it to the shadow-utils test repository.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5808#issuecomment-935645839
URL: https://github.com/SSSD/sssd/pull/5806
Author: shridhargadekar
Title: #5806: Tests: files provider testcases
Action: synchronized
To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/5806/head:pr5806
git checkout pr5806
From
73 matches
Mail list logo