[autobackport: sssd-2-11] SBUS: increase SBUS_MESSAGE_TIMEOUT to 5 mins #8382
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request is an automatic backport of two commits. The first commit increases the SBUS_MESSAGE_TIMEOUT from 2 minutes to 5 minutes to better handle long-running operations, such as initgroups for users with a very large number of groups. The second commit fixes a bug in the client idle timer logic, ensuring that connections are terminated correctly at the specified timeout. The changes are generally good, but I've noted one area for improvement where a similar bug in the responder idle timer was not addressed.
sumit-bose
left a comment
sumit-bose
left a comment
There was a problem hiding this comment.
Hi,
backport looks like the original, ACK.
bye,
Sumit
Handling BE_REQ_INITGROUPS for LDAP user with 10k groups takes longer than 2 mins. Reviewed-by: Pavel Březina <pbrezina@redhat.com> Reviewed-by: Sumit Bose <sbose@redhat.com> (cherry picked from commit ee081e1)
As it was implemented previously, effective period was 1.5*client_idle_timeout instead of `client_idle_timeout` as documented. Log with default value - 60 - before a fix: ``` (:49:12): [nss] [setup_client_idle_timer] (0x4000): Idle timer re-set for client [0x557af16f31b0][22] (:49:42): [nss] [setup_client_idle_timer] (0x4000): Idle timer re-set for client [0x557af16f31b0][22] (:50:12): [nss] [setup_client_idle_timer] (0x4000): Idle timer re-set for client [0x557af16f31b0][22] (:50:42): [nss] [client_idle_handler] (0x2000): Terminating idle client [0x557af16f31b0][22] ``` Reviewed-by: Pavel Březina <pbrezina@redhat.com> Reviewed-by: Sumit Bose <sbose@redhat.com> (cherry picked from commit 7762901)
|
The pull request was accepted by @alexey-tikhonov with the following PR CI status: 🟢 CodeQL (success) There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging. |
sssd-bot
force-pushed
the
SSSD-sssd-backport-pr8367-to-sssd-2-11
branch
from
42461aa to
7e56f39
Compare
4 participants
This is an automatic backport of PR#8367 SBUS: increase SBUS_MESSAGE_TIMEOUT to 5 mins to branch sssd-2-11, created by @alexey-tikhonov.
Please make sure this backport is correct.
Note
The commits were cherry-picked without conflicts.
You can push changes to this pull request
Original commits
ee081e1 - SBUS: increase SBUS_MESSAGE_TIMEOUT to 5 mins
7762901 - RESPONDER: fixed an issue with 'client_idle_timer'
Backported commits
Original Pull Request Body
... and fixed an issue with 'client_idle_timer'