[autobackport: sssd-2-13] sdap: Reduce log level when get_naming_context() fails#8699
Conversation
There was a problem hiding this comment.
Code Review
This pull request downgrades the log level from SSSDBG_CRIT_FAILURE to SSSDBG_MINOR_FAILURE in src/providers/ldap/sdap.c when get_naming_context fails, reflecting that this failure is non-fatal. I have no feedback to provide as there were no review comments.
Signed-off-by: Samuel Cabrero <[email protected]> Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Justin Stephenson <[email protected]> (cherry picked from commit edf4a0f)
|
The pull request was accepted by @alexey-tikhonov with the following PR CI status: 🟢 rpm-build:centos-stream-10-x86_64:upstream (success) There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging. |
3d94983 to
78e8cd5
Compare
This is an automatic backport of PR#8697 sdap: Reduce log level when get_naming_context() fails to branch sssd-2-13, created by @scabrero.
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
edf4a0f - sdap: Reduce log level when get_naming_context() fails
Backported commits
Original Pull Request Body
When a plain LDAP server hosts multiple naming contexts but there is no
defaultNamingContextin the rootdse, the following message is logged periodically:[be[LDAP]] [sdap_set_config_options_with_rootdse] (0x0020): get_naming_context failed.The message is logged at
SSSDBG_CRIT_FAILURElevel, which seems too high for a non-fatal failure: