[sssd PR#283][opened] KCM: Fix the per-client serialization queue

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/283
Author: jhrozek
 Title: #283: KCM: Fix the per-client serialization queue
Action: opened

PR body:
"""
Resolves:
   https://pagure.io/SSSD/sssd/issue/3372

Fixes a race condition between one client request adding an operation to
the hash table value, which was previously a linked list of operations,
while another concurrent operation would remove the last remaining linked
list element through its callback.

Instead, the hash table value is now a separate 'queue head' structure
which is only changed in a tevent request to make sure is is not processes
concurrently with adding to the queue (which is also a tevent request).
"""

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/283/head:pr283
git checkout pr283

[sssd PR#283][synchronized] KCM: Fix the per-client serialization queue

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/283
Author: jhrozek
 Title: #283: KCM: Fix the per-client serialization queue
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/283/head:pr283
git checkout pr283

[sssd PR#283][synchronized] KCM: Fix the per-client serialization queue

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/283
Author: jhrozek
 Title: #283: KCM: Fix the per-client serialization queue
Action: synchronized

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/283/head:pr283
git checkout pr283

[sssd PR#283][comment] KCM: Fix the per-client serialization queue

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/283
Title: #283: KCM: Fix the per-client serialization queue

jhrozek commented:
"""
retest this please
"""

See the full comment at https://github.com/SSSD/sssd/pull/283#issuecomment-303628124

[sssd PR#283][comment] KCM: Fix the per-client serialization queue

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/283
Title: #283: KCM: Fix the per-client serialization queue

jhrozek commented:
"""
CI: http://sssd-ci.duckdns.org/logs/job/69/96/summary.html

the failure on rawhide is: http://sssd-ci.duckdns.org/logs/job/69/96/fedora_rawhide/ci-build-debug/ci-m...

(tl;dr find-uid tests)
"""

See the full comment at https://github.com/SSSD/sssd/pull/283#issuecomment-303628236

[sssd PR#283][comment] KCM: Fix the per-client serialization queue

[pbrezina]

URL: https://github.com/SSSD/sssd/pull/283
Title: #283: KCM: Fix the per-client serialization queue

pbrezina commented:
"""
I can't reproduce with the patch anymore. Ack.
"""

See the full comment at https://github.com/SSSD/sssd/pull/283#issuecomment-303699270

[sssd PR#283][+Accepted] KCM: Fix the per-client serialization queue

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/283
Title: #283: KCM: Fix the per-client serialization queue

Label: +Accepted

[sssd PR#283][comment] KCM: Fix the per-client serialization queue

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/283
Title: #283: KCM: Fix the per-client serialization queue

jhrozek commented:
"""
* master:
 * b31f75f44a9e1dc0521ec73176f89e05db4973ba
 * 7930ee12093eae1e1ab9422c4f4f9f8c5661fcb9
"""

See the full comment at https://github.com/SSSD/sssd/pull/283#issuecomment-303733876

[sssd PR#283][+Pushed] KCM: Fix the per-client serialization queue

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/283
Title: #283: KCM: Fix the per-client serialization queue

Label: +Pushed

[sssd PR#283][closed] KCM: Fix the per-client serialization queue

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/283
Author: jhrozek
 Title: #283: KCM: Fix the per-client serialization queue
Action: closed

To pull the PR as Git branch:
git remote add ghsssd https://github.com/SSSD/sssd
git fetch ghsssd pull/283/head:pr283
git checkout pr283

[sssd PR#283][comment] KCM: Fix the per-client serialization queue

[jhrozek]

URL: https://github.com/SSSD/sssd/pull/283
Title: #283: KCM: Fix the per-client serialization queue

jhrozek commented:
"""
Sorry, wrong hashes:
 * fb51bb68e62de7bb8542f5d224994eb7143040a6
 * 274489b092bba5fc81cb0f803843d56b267c5aaf
"""

See the full comment at https://github.com/SSSD/sssd/pull/283#issuecomment-303734428