Update git submodules

* Update oslo.log from branch 'master'
  to 8705f67bfbd733edf923003c0a5850e7a4a5735b
  - Remove the usage of the Eventlet debug feature from oslo.log.
    
    We initially started to work on this topic with the intention to solve
    the problem of using the Eventlet hub_prevent_multiple_readers debug feature.
    This feature is a debug convenience. This feature is not supported by the new
    asyncio hub[1] of Eventlet. You can also check the documentation here[2] from eventlet. The problem with disabling this procedure is that
    it exposes you to risk. Deactivation applies to the entire stack. If a project
    uses oslo.log, for example nova, then it exposes all threads to concurrent access
    on the process file descriptors.
    As explained in the function's documentation[3], when several greenlets are reading from
    the same socket, it's difficult to predict which greenlet will receive which data.
    You really need to be sure of what you're doing to use this function. Also explained
    in the raise condition added by this commit[4] and we have a reproducer[5] that exposes
    the problem. According to our tests, removing this line does not change the fix provided.
    It also seems that cinder no longer uses[6] logging native thread.
    
    Related-Bug: #1983863
    
    [1] https://review.opendev.org/c/openstack/governance/+/902585/17..18
    [2] https://eventlet.readthedocs.io/en/latest/asyncio/migration.html#known-limitations-and-work-in-progress
    [3] https://github.com/eventlet/eventlet/blob/master/eventlet/debug.py#L149
    [4] cb7c8c0196
    [5] https://github.com/4383/oslo.log/blob/reproducer2/reproducer.py
    [6] https://bugs.launchpad.net/oslo.log/+bug/1983863/comments/18
    
    Co-authored-by: Hervé Beraud <hberaud@redhat.com>
    Change-Id: Ia4c8e82f957c4ef3c176246fea0bc0adab4f4dc4
This commit is contained in:
damani42 2024-05-22 05:02:14 +02:00 committed by Gerrit Code Review
parent fd579b2fb7
commit 77583d1312
1 changed files with 1 additions and 1 deletions

@ -1 +1 @@
Subproject commit f05a8521420a99d426130ac5e0c48459e42d7512
Subproject commit 8705f67bfbd733edf923003c0a5850e7a4a5735b