URL: https://github.com/SSSD/sssd/pull/5863
Title: #5863: Responder and Child process tevent chain id improvements

alexey-tikhonov commented:
"""
> This introduced an issue where `[ldb]` messages are tagged with CID/RID, 
> which makes the log analyzer output quite unfriendly/ugly.

But this requires log with `SSSDBG_TRACE_LDB` enabled - something that should 
not really happen much in the wild, right?
Probably we don't have to do anything with it at all?

```
>      if (debug_chain_id > 0) {
> -        sss_debug_backtrace_printf(level, debug_chain_id_fmt, 
> debug_chain_id);
> +        if (strcmp(function, "ldb") != 0) {
```

Doesn't look appealing... Imo, even check for `level` would be better, but see 
above.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/5863#issuecomment-992890844
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/sssd-devel@lists.fedorahosted.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure

Reply via email to