[sssd PR#5666][opened] debug: add support for tevent chain id

[pbrezina]

   URL: https://github.com/SSSD/sssd/pull/5666
Author: pbrezina
 Title: #5666: debug: add support for tevent chain id
Action: opened

PR body:
"""
This PR adds a unique identifier to the log messages of each data provider requests
so we are able to track the request form its beginning to the end.

Note: this requires functionality from
https://gitlab.com/samba-team/samba/-/merge_requests/1988
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

pbrezina commented:
"""
I need to add compile time check for presence of required tevent functions, but you can
already start testing it.
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

alexey-tikhonov commented:
"""
I think we need some configure checks for backward compatibility with `libtevent` versions
that doesn't support a feature.
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

alexey-tikhonov commented:
"""
I think we need some configure checks for backward compatibility with `libtevent` versions
that doesn't support a feature.
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[justin-stephenson]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

justin-stephenson commented:
"""
~~~
[sdap_get_generic_ext_step] (0x0400): [#11] calling ldap_search_ext with
[(&(objectClass=group)(sAMAccountName=*))][CN=Domain
Controllers,CN=Users,DC=ad,DC=vm].
[sdap_get_generic_ext_step] (0x1000): [#11] Requesting attrs: [objectClass]
~~~

Would it be possible to add some tag to allow easier parsing, similar to how the NSS
responder uses `CR #` in the logs:
~~~
[sdap_get_generic_ext_step] (0x0400): [DP #11] calling ldap_search_ext with
[(&(objectClass=group)(sAMAccountName=*))][CN=Domain
Controllers,CN=Users,DC=ad,DC=vm].
[sdap_get_generic_ext_step] (0x1000): [DP #11] Requesting attrs: [objectClass]
~~~
or `[DP_REQ #11]` for example. 
"""

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

[sssd PR#5666][synchronized] debug: add support for tevent chain id

[pbrezina]

   URL: https://github.com/SSSD/sssd/pull/5666
Author: pbrezina
 Title: #5666: debug: add support for tevent chain id
Action: synchronized

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

pbrezina commented:
"""
I added configure-time check for the required functionality. It is ready for review. 
"""

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

[sssd PR#5666][+Waiting for review] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

Label: +Waiting for review

[sssd PR#5666][synchronized] debug: add support for tevent chain id

[pbrezina]

   URL: https://github.com/SSSD/sssd/pull/5666
Author: pbrezina
 Title: #5666: debug: add support for tevent chain id
Action: synchronized

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

[sssd PR#5666][synchronized] debug: add support for tevent chain id

[pbrezina]

   URL: https://github.com/SSSD/sssd/pull/5666
Author: pbrezina
 Title: #5666: debug: add support for tevent chain id
Action: synchronized

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

[sssd PR#5666][synchronized] debug: add support for tevent chain id

[pbrezina]

   URL: https://github.com/SSSD/sssd/pull/5666
Author: pbrezina
 Title: #5666: debug: add support for tevent chain id
Action: synchronized

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

pbrezina commented:
"""
Tevent patch was merged. I fixed issues in tests and two more corner cases that I found.
This is ready for review.
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

pbrezina commented:
"""
>  ```
>  [sdap_get_generic_ext_step] (0x0400): [#11] calling ldap_search_ext with
> [(&(objectClass=group)(sAMAccountName=*))][CN=Domain
> Controllers,CN=Users,DC=ad,DC=vm].
>  [sdap_get_generic_ext_step] (0x1000): [#11] Requesting attrs: [objectClass]
>  ```
>  
>  Would it be possible to add some tag to allow easier parsing, similar to how the NSS
> responder uses `CR #` in the logs:
>  
>  ```
>  [sdap_get_generic_ext_step] (0x0400): [DP #11] calling ldap_search_ext with
> [(&(objectClass=group)(sAMAccountName=*))][CN=Domain
> Controllers,CN=Users,DC=ad,DC=vm].
>  [sdap_get_generic_ext_step] (0x1000): [DP #11] Requesting attrs: [objectClass]
>  ```
>  
>  or `[DP_REQ #11]` for example.
>  
Since this string is attached everywhere, we should make it as small as possible. But we
can add something if it will help you. Do you still need it?
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[justin-stephenson]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

justin-stephenson commented:
"""
>  > ```
>  > [sdap_get_generic_ext_step] (0x0400): [#11] calling ldap_search_ext with
> [(&(objectClass=group)(sAMAccountName=*))][CN=Domain
> Controllers,CN=Users,DC=ad,DC=vm].
>  > [sdap_get_generic_ext_step] (0x1000): [#11] Requesting attrs: [objectClass]
>  > ```
>  > 
>  > 
>  >     
>  >       
>  >     
>  > 
>  >       
>  >     
>  > 
>  >     
>  >   
>  > Would it be possible to add some tag to allow easier parsing, similar to how the NSS
> responder uses `CR #` in the logs:
>  > ```
>  > [sdap_get_generic_ext_step] (0x0400): [DP #11] calling ldap_search_ext with
> [(&(objectClass=group)(sAMAccountName=*))][CN=Domain
> Controllers,CN=Users,DC=ad,DC=vm].
>  > [sdap_get_generic_ext_step] (0x1000): [DP #11] Requesting attrs: [objectClass]
>  > ```
>  > 
>  > 
>  >     
>  >       
>  >     
>  > 
>  >       
>  >     
>  > 
>  >     
>  >   
>  > or `[DP_REQ #11]` for example.
>  
>  Since this string is attached everywhere, we should make it as small as possible. But we
> can add something if it will help you. Do you still need it?
>  
It's not needed, I am able to extract the logs without it. Disregard.
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

alexey-tikhonov commented:
"""
Mock build fails on F33 -
https://s3.eu-central-1.amazonaws.com/sssd-ci/PR-5666/6/fedora33/build.log :
```
libtool: link: gcc -Wall -Wshadow -Wstrict-prototypes -Wpointer-arith -Wcast-qual
-Wcast-align -Wwrite-strings -Wundef -Werror-implicit-function-declaration -Winit-self
-Wmissing-include-dirs -fno-strict-aliasing -std=gnu99 -O2 -flto=auto -ffat-lto-objects
-fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security
-Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS
-specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong
-specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic
-fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection
-D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wl,-z -Wl,relro
-Wl,--as-needed -Wl,-z -Wl,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -o
.libs/test_sbus_message src/tests/cmocka/sbus/test_sbus_message-test_sbus_message.o 
-lcmocka ./.libs/libsss_debug.so ./.libs/libsss_test_common.a
/builddir/build/BUILD/sssd-2.5.2/.libs/libsss_util.so -lrt -lpopt -lldb -lselinux -ltdb
-lpcre2-8 -lini_config -lbasicobjects -lref_array -lcollection
/builddir/build/BUILD/sssd-2.5.2/.libs/libsss_cert.so
/builddir/build/BUILD/sssd-2.5.2/.libs/libsss_crypt.so -lcrypto
/builddir/build/BUILD/sssd-2.5.2/.libs/libsss_child.so
/builddir/build/BUILD/sssd-2.5.2/.libs/libsss_debug.so -lsystemd ./.libs/libsss_sbus.so
-ldhash -ltevent -ltalloc -ldbus-1 -lglib-2.0 -Wl,-rpath -Wl,/usr/lib64/sssd
/usr/bin/ld: ./.libs/libsss_sbus.so: undefined reference to `sss_chain_id_set'
```
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

alexey-tikhonov commented:
"""
>  ```
>  [sdap_get_generic_ext_step] (0x0400): [#11] calling ldap_search_ext with
> [(&(objectClass=group)(sAMAccountName=*))][CN=Domain
> Controllers,CN=Users,DC=ad,DC=vm].
>  [sdap_get_generic_ext_step] (0x1000): [#11] Requesting attrs: [objectClass]
>  ```
>  
>  Would it be possible to add some tag to allow easier parsing, similar to how the NSS
> responder uses `CR #` in the logs:
>  
I would support this idea for the sake of readability. Currently it's difficult to
figure out the meaning of this marker.

Maybe `[CID#11]` (chain-id)? Or `RID` (request-id)...
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[elkoniu]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

elkoniu commented:
"""
>  Mock build fails on F33 -
> https://s3.eu-central-1.amazonaws.com/sssd-ci/PR-5666/6/fedora33/build.log :
>  
>  ```
>  libtool: link: gcc -Wall -Wshadow -Wstrict-prototypes -Wpointer-arith -Wcast-qual
> -Wcast-align -Wwrite-strings -Wundef -Werror-implicit-function-declaration -Winit-self
> -Wmissing-include-dirs -fno-strict-aliasing -std=gnu99 -O2 -flto=auto -ffat-lto-objects
> -fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security
> -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS
> -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong
> -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic
> -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection
> -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wl,-z -Wl,relro
> -Wl,--as-needed -Wl,-z -Wl,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -o
> .libs/test_sbus_message src/tests/cmocka/sbus/test_sbus_message-test_sbus_message.o 
> -lcmocka ./.libs/libsss_debug.so ./.libs/libsss_test_common.a
> /builddir/build/BUILD/sssd-2.5.2/.libs/libsss_util.so -lrt -lpopt -lldb -lselinux -ltdb
> -lpcre2-8 -lini_config -lbasicobjects -lref_array -lcollection
> /builddir/build/BUILD/sssd-2.5.2/.libs/libsss_cert.so
> /builddir/build/BUILD/sssd-2.5.2/.libs/libsss_crypt.so -lcrypto
> /builddir/build/BUILD/sssd-2.5.2/.libs/libsss_child.so
> /builddir/build/BUILD/sssd-2.5.2/.libs/libsss_debug.so -lsystemd ./.libs/libsss_sbus.so
> -ldhash -ltevent -ltalloc -ldbus-1 -lglib-2.0 -Wl,-rpath -Wl,/usr/lib64/sssd
>  /usr/bin/ld: ./.libs/libsss_sbus.so: undefined reference to `sss_chain_id_set'
>  ```
>  
This is because `sss_chain_id_set` is covered by `#ifdef BUILD_CHAIN_ID` and this mock
build does not detect patched version of tevent: `configure: Chain id support is disabled
due to missing dependencies in tevent`
This way `sss_chain_id_set` call is undefined.
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

alexey-tikhonov commented:
"""
>  > Mock build fails on F33 -
> https://s3.eu-central-1.amazonaws.com/sssd-ci/PR-5666/6/fedora33/build.log :
>  > ```
>  > libtool: link: gcc -Wall -Wshadow -Wstrict-prototypes -Wpointer-arith -Wcast-qual
> -Wcast-align -Wwrite-strings -Wundef -Werror-implicit-function-declaration -Winit-self
> -Wmissing-include-dirs -fno-strict-aliasing -std=gnu99 -O2 -flto=auto -ffat-lto-objects
> -fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security
> -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS
> -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong
> -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic
> -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection
> -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -Wl,-z -Wl,relro
> -Wl,--as-needed -Wl,-z -Wl,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -o
> .libs/test_sbus_message src/tests/cmocka/sbus/test_sbus_message-test_sbus_message.o 
> -lcmocka ./.libs/libsss_debug.so ./.libs/libsss_test_common.a
> /builddir/build/BUILD/sssd-2.5.2/.libs/libsss_util.so -lrt -lpopt -lldb -lselinux -ltdb
> -lpcre2-8 -lini_config -lbasicobjects -lref_array -lcollection
> /builddir/build/BUILD/sssd-2.5.2/.libs/libsss_cert.so
> /builddir/build/BUILD/sssd-2.5.2/.libs/libsss_crypt.so -lcrypto
> /builddir/build/BUILD/sssd-2.5.2/.libs/libsss_child.so
> /builddir/build/BUILD/sssd-2.5.2/.libs/libsss_debug.so -lsystemd ./.libs/libsss_sbus.so
> -ldhash -ltevent -ltalloc -ldbus-1 -lglib-2.0 -Wl,-rpath -Wl,/usr/lib64/sssd
>  > /usr/bin/ld: ./.libs/libsss_sbus.so: undefined reference to `sss_chain_id_set'
>  > ```
>  
>  This is because `sss_chain_id_set` is covered by `#ifdef BUILD_CHAIN_ID` and this mock
> build does not detect patched version of tevent: `configure: Chain id support is disabled
> due to missing dependencies in tevent`
>  This way `sss_chain_id_set` call is undefined.
>  
In `util/sss_chain_id.c` there is `#else /* BUILD_CHAIN_ID not defined */` section that
defines stubs in case libtevent misses required functionality. So this error is actually
odd. Even more weird that only F33 was affected.


"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

alexey-tikhonov commented:
"""
Hi @pbrezina,

I have several comments/questions, mostly cosmetic (and weird F33 mock build fail).

Otherwise looks good to me. Covscan is clean.
"""

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

[sssd PR#5666][synchronized] debug: add support for tevent chain id

[pbrezina]

   URL: https://github.com/SSSD/sssd/pull/5666
Author: pbrezina
 Title: #5666: debug: add support for tevent chain id
Action: synchronized

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

pbrezina commented:
"""
I added src/util/sss_chain_id.c to libsss_sbus.la this should fixed the mock issue.
Perhaps the linker is smarter on newer distros?
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

pbrezina commented:
"""
Also added RID prefix.
"""

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

[sssd PR#5666][synchronized] debug: add support for tevent chain id

[pbrezina]

   URL: https://github.com/SSSD/sssd/pull/5666
Author: pbrezina
 Title: #5666: debug: add support for tevent chain id
Action: synchronized

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

[sssd PR#5666][+Bugzilla] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

Label: +Bugzilla

[sssd PR#5666][synchronized] debug: add support for tevent chain id

[pbrezina]

   URL: https://github.com/SSSD/sssd/pull/5666
Author: pbrezina
 Title: #5666: debug: add support for tevent chain id
Action: synchronized

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

[sssd PR#5666][synchronized] debug: add support for tevent chain id

[pbrezina]

   URL: https://github.com/SSSD/sssd/pull/5666
Author: pbrezina
 Title: #5666: debug: add support for tevent chain id
Action: synchronized

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

pbrezina commented:
"""
See new patches.
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

alexey-tikhonov commented:
"""
Btw, previous run on F34:
```
mock-build:             success
```
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

alexey-tikhonov commented:
"""
Btw, previous run on F34:
```
mock-build:             success
```
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

alexey-tikhonov commented:
"""
>  See new patches.
>  
Thank you. All my questions were addressed.
"""

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[elkoniu]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

elkoniu commented:
"""
I was playing with this PR in various scenarios and all logs with new ID are presented as
they should. I havent observed any strange or undefined behavior. ACK from me.
"""

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

[sssd PR#5666][-Waiting for review] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

Label: -Waiting for review

[sssd PR#5666][+Accepted] debug: add support for tevent chain id

[alexey-tikhonov]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

Label: +Accepted

[sssd PR#5666][+Ready to push] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

Label: +Ready to push

[sssd PR#5666][comment] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

pbrezina commented:
"""
Pushed PR: https://github.com/SSSD/sssd/pull/5666

* `master`
    * 881a1a412eb036b8df22bc637a4bb38f10cdb9cf - debug: enable chain id in backend
    * f02ac230be3cef8fa83ee8cafea519a22268b23a - debug: add support for tevent chain id

"""

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

[sssd PR#5666][+Pushed] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

Label: +Pushed

[sssd PR#5666][-Accepted] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

Label: -Accepted

[sssd PR#5666][-Ready to push] debug: add support for tevent chain id

[pbrezina]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

Label: -Ready to push

[sssd PR#5666][closed] debug: add support for tevent chain id

[pbrezina]

   URL: https://github.com/SSSD/sssd/pull/5666
Author: pbrezina
 Title: #5666: debug: add support for tevent chain id
Action: closed

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

[sssd PR#5666][comment] debug: add support for tevent chain id

[justin-stephenson]

  URL: https://github.com/SSSD/sssd/pull/5666
Title: #5666: debug: add support for tevent chain id

justin-stephenson commented:
"""
Is it expected for this to work with ``logger=journald``. My parsing tool supports this
but I guess RID# messages are not added here.
"""

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