[SSSD] [sssd PR#5830][synchronized] Tests: Port the old ns_account_lock.sh script to pytest

2021-10-18 Thread aborah-sudo
URL: https://github.com/SSSD/sssd/pull/5830 Author: aborah-sudo Title: #5830: Tests: Port the old ns_account_lock.sh script to pytest Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5830/head:pr5830 git checkout pr5830

[SSSD] [sssd PR#5830][synchronized] Tests: Port the old ns_account_lock.sh script to pytest

2021-10-18 Thread aborah-sudo
URL: https://github.com/SSSD/sssd/pull/5830 Author: aborah-sudo Title: #5830: Tests: Port the old ns_account_lock.sh script to pytest Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5830/head:pr5830 git checkout pr5830

[SSSD] [sssd PR#5831][opened] Fixes a couple of compilations warnings

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5831 Author: alexey-tikhonov Title: #5831: Fixes a couple of compilations warnings Action: opened PR body: """ None """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5831/head:pr5831 git checkout p

[SSSD] [sssd PR#5827][+Waiting for review] A number of fixes around `strto*()` usage

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5827 Title: #5827: A number of fixes around `strto*()` usage Label: +Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedor

[SSSD] [sssd PR#5827][-Changes requested] A number of fixes around `strto*()` usage

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5827 Title: #5827: A number of fixes around `strto*()` usage Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedora

[SSSD] [sssd PR#5827][comment] A number of fixes around `strto*()` usage

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5827 Title: #5827: A number of fixes around `strto*()` usage alexey-tikhonov commented: """ > > src/providers/ldap/ldap_id_services.c > > 266:port = strtouint16(state->name, NULL, 10); > > Again, internal stuff. But ok, to be consistent I will

[SSSD] [sssd PR#5827][synchronized] A number of fixes around `strto*()` usage

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5827 Author: alexey-tikhonov Title: #5827: A number of fixes around `strto*()` usage Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5827/head:pr5827 git checkout pr5827 From 4354

[SSSD] [sssd PR#5829][comment] DP: Resolve intermediate groups prior to SR overlay

2021-10-18 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/5829 Title: #5829: DP: Resolve intermediate groups prior to SR overlay justin-stephenson commented: """ This is on me to fix the intg test failures, test_initgroups_with_mc is failing, likely due to ``sbus_call_nss_memcache_UpdateInitgroups_send`` being m

[SSSD] [sssd PR#5827][comment] A number of fixes around `strto*()` usage

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5827 Title: #5827: A number of fixes around `strto*()` usage alexey-tikhonov commented: """ > I found call for `strtoul()` in one more place: > > ``` > src/providers/ldap/sdap_sudo_shared.c > > 194:st

[SSSD] [sssd PR#5827][+Changes requested] A number of fixes around `strto*()` usage

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5827 Title: #5827: A number of fixes around `strto*()` usage Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedora

[SSSD] [sssd PR#5827][comment] A number of fixes around `strto*()` usage

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5827 Title: #5827: A number of fixes around `strto*()` usage alexey-tikhonov commented: """ > There are few more places where `strtouint*()` is called with 2nd parameter > set to NULL: I don't have very strong justification but will try my best to explain

[SSSD] [sssd PR#5827][comment] A number of fixes around `strto*()` usage

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5827 Title: #5827: A number of fixes around `strto*()` usage alexey-tikhonov commented: """ > There are few more places where `strtouint*()` is called with 2nd parameter > set to NULL: I don't have very strong justification but will try my best to explain

[SSSD] [sssd PR#5827][comment] A number of fixes around `strto*()` usage

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5827 Title: #5827: A number of fixes around `strto*()` usage alexey-tikhonov commented: """ > There are few more places where `strtouint*()` is called with 2nd parameter > set to NULL: I don't have very strong justification but will try my best to explain

[SSSD] [sssd PR#5830][synchronized] Tests: Port the old ns_account_lock.sh script to pytest

2021-10-18 Thread aborah-sudo
URL: https://github.com/SSSD/sssd/pull/5830 Author: aborah-sudo Title: #5830: Tests: Port the old ns_account_lock.sh script to pytest Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5830/head:pr5830 git checkout pr5830

[SSSD] [sssd PR#5830][synchronized] Tests: Port the old ns_account_lock.sh script to pytest

2021-10-18 Thread aborah-sudo
URL: https://github.com/SSSD/sssd/pull/5830 Author: aborah-sudo Title: #5830: Tests: Port the old ns_account_lock.sh script to pytest Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5830/head:pr5830 git checkout pr5830

[SSSD] [sssd PR#5830][synchronized] Tests: Port the old ns_account_lock.sh script to pytest

2021-10-18 Thread aborah-sudo
URL: https://github.com/SSSD/sssd/pull/5830 Author: aborah-sudo Title: #5830: Tests: Port the old ns_account_lock.sh script to pytest Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5830/head:pr5830 git checkout pr5830

[SSSD] [sssd PR#5830][opened] Tests: Port the old ns_account_lock.sh script to pytest

2021-10-18 Thread aborah-sudo
URL: https://github.com/SSSD/sssd/pull/5830 Author: aborah-sudo Title: #5830: Tests: Port the old ns_account_lock.sh script to pytest Action: opened PR body: """ Port the old ns_account_lock.sh script to pytest """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/ss

[SSSD] [sssd PR#5829][+Bugzilla] DP: Resolve intermediate groups prior to SR overlay

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5829 Title: #5829: DP: Resolve intermediate groups prior to SR overlay Label: +Bugzilla ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedor

[SSSD] [sssd PR#5829][synchronized] DP: Resolve intermediate groups prior to SR overlay

2021-10-18 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/5829 Author: justin-stephenson Title: #5829: DP: Resolve intermediate groups prior to SR overlay Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5829/head:pr5829 git checkout pr58

[SSSD][sssd PR#5814][-Changes requested] Tests: sss_override does not take precedence over override_homedir di…

2021-10-18 Thread sgoveas
URL: https://github.com/SSSD/sssd/pull/5814 Title: #5814: Tests: sss_override does not take precedence over override_homedir di… Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD][sssd PR#5814][+Accepted] Tests: sss_override does not take precedence over override_homedir di…

2021-10-18 Thread sgoveas
URL: https://github.com/SSSD/sssd/pull/5814 Title: #5814: Tests: sss_override does not take precedence over override_homedir di… Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-dev

[SSSD] [sssd PR#5829][comment] DP: Resolve intermediate groups prior to SR overlay

2021-10-18 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/5829 Title: #5829: DP: Resolve intermediate groups prior to SR overlay justin-stephenson commented: """ https://bugzilla.redhat.com/show_bug.cgi?id=1968340 """ See the full comment at https://github.com/SSSD/sssd/pull/5829#issuecomment-945898763 _

[SSSD] [sssd PR#5829][opened] DP: Resolve intermediate groups prior to SR overlay

2021-10-18 Thread justin-stephenson
URL: https://github.com/SSSD/sssd/pull/5829 Author: justin-stephenson Title: #5829: DP: Resolve intermediate groups prior to SR overlay Action: opened PR body: """ SSSD SR exclude_groups checking can fail when only intermediate groups are fetched during the login process. Add a step to resolve

[SSSD] [sssd PR#5782][-Changes requested] CONFDB: Change ownership of config.ldb

2021-10-18 Thread thalman
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahos

[SSSD] [sssd PR#5782][+Waiting for review] CONFDB: Change ownership of config.ldb

2021-10-18 Thread thalman
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb Label: +Waiting for review ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedoraho

[SSSD] [sssd PR#5784][+Waiting for review] proxy: allow removing group members

2021-10-18 Thread ikerexxe
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 sssd-devel-le...@lists.fedorahoste

[SSSD] [sssd PR#5784][-Changes requested] proxy: allow removing group members

2021-10-18 Thread ikerexxe
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 sssd-devel-le...@lists.fedorahosted

[SSSD] [sssd PR#5784][comment] proxy: allow removing group members

2021-10-18 Thread ikerexxe
URL: https://github.com/SSSD/sssd/pull/5784 Title: #5784: proxy: allow removing group members ikerexxe commented: """ I've updated the patch taking into account your suggestions. """ See the full comment at https://github.com/SSSD/sssd/pull/5784#issuecomment-945777100 _

[SSSD] [sssd PR#5784][synchronized] proxy: allow removing group members

2021-10-18 Thread ikerexxe
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 be0584db2c5ff898b

[SSSD] [sssd PR#5826][closed] DEBUG: fix missing "va_end"

2021-10-18 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/5826 Author: alexey-tikhonov Title: #5826: DEBUG: fix missing "va_end" Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/5826/head:pr5826 git checkout pr5826 _

[SSSD] [sssd PR#5826][-Ready to push] DEBUG: fix missing "va_end"

2021-10-18 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/5826 Title: #5826: DEBUG: fix missing "va_end" Label: -Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora

[SSSD] [sssd PR#5826][+Pushed] DEBUG: fix missing "va_end"

2021-10-18 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/5826 Title: #5826: DEBUG: fix missing "va_end" Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of

[SSSD] [sssd PR#5826][-Accepted] DEBUG: fix missing "va_end"

2021-10-18 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/5826 Title: #5826: DEBUG: fix missing "va_end" Label: -Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code

[SSSD] [sssd PR#5826][comment] DEBUG: fix missing "va_end"

2021-10-18 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/5826 Title: #5826: DEBUG: fix missing "va_end" pbrezina commented: """ Pushed PR: https://github.com/SSSD/sssd/pull/5826 * `master` * 625274738b5f68418608be99b68d35c43079e2a1 - DEBUG: fix missing "va_end" """ See the full comment at https://github.c

[SSSD] [sssd PR#5826][+Ready to push] DEBUG: fix missing "va_end"

2021-10-18 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/5826 Title: #5826: DEBUG: fix missing "va_end" Label: +Ready to push ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora

[SSSD] [sssd PR#5828][opened] sbus: maintain correct refcount before sending a reply

2021-10-18 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/5828 Author: pbrezina Title: #5828: sbus: maintain correct refcount before sending a reply Action: opened PR body: """ `sbus_reply` decreases the refcount of `@reply`. This usuall means that refcount drops to zero and the message is freed. However, under

[SSSD] [sssd PR#5782][comment] CONFDB: Change ownership of config.ldb

2021-10-18 Thread alexey-tikhonov
URL: https://github.com/SSSD/sssd/pull/5782 Title: #5782: CONFDB: Change ownership of config.ldb alexey-tikhonov commented: """ > By the way, I haven't found why `debian10` is failing. Isn't relevant to this PR (failed on attempt to setup machine before multihost tests that aren't run on this