)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"fdf71db6bb386bc8e3a87a5ac9d3ab486c5bb684","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c8e2241a_25deda30","updated":"2025-04-28 13:00:10.000000000","message":"If you do not mind I set the topic to eventlet-removal to have the networking-baremetal patch in the eventlet removal dashboard","commit_id":"b132342b5868160633e606d3d0a6afbb804cbfd2"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"6d9f5860d4b84a15cfdfa6e2aea56bd16edbc9cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"62dc7690_825801b6","updated":"2025-04-07 21:32:37.000000000","message":"This change should fix the CI errors:\nhttps://review.opendev.org/c/openstack/networking-generic-switch/+/946089","commit_id":"b132342b5868160633e606d3d0a6afbb804cbfd2"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"bcd6d44f63f4b8891a4f49bdaecc46a80119d5e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a66a7ddc_3e7a008f","updated":"2025-04-08 17:43:22.000000000","message":"This is review feedback from the PTG review jam and reflects feedback from multiple cores.\n\nThe code that\u0027s written looks good, but there are a few things we\u0027d like done as well before this lands:\n- Ensure all the requirements.txt entires do not require eventlet where possible (at a minimum, this will require you to raise minimum supported oslo.* library versions). A good way to help us understand this in code review would be the output of a pip freeze from a fresh tox unit test venv (tox -repy312; . .tox/py312/bin/pip freeze). If some downstream package/library pulls in eventlet; that may be OK but we\u0027ll need to do due diligence to ensure it\u0027s not monkey patching our code anymore.\n- Please provide a release node","commit_id":"b132342b5868160633e606d3d0a6afbb804cbfd2"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"0c22cee709968bd613bafa637ef6dccd276dbb1e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0ce2ee83_8e034403","updated":"2025-04-08 07:50:35.000000000","message":"recheck","commit_id":"b132342b5868160633e606d3d0a6afbb804cbfd2"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"5471aee70c663b93b456ecbb872c4dba2a3b173f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f8fa1473_b14fc029","in_reply_to":"6f20b6d2_bc26a233","updated":"2025-04-24 14:37:15.000000000","message":"The other patch was made simply because I forgot about this one, didn\u0027t meant to stomp on this! I would\u0027ve looked at NGS if I remembered this. Either way, it\u0027ll be a bit until the oslo.service stuff lands I suspect.","commit_id":"b132342b5868160633e606d3d0a6afbb804cbfd2"},{"author":{"_account_id":24245,"name":"Harald Jensås","email":"hjensas@redhat.com","username":"harald.jensas"},"change_message_id":"4d8727b6ce329ac98c5029bdd63a546fe1388684","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6f20b6d2_bc26a233","in_reply_to":"a66a7ddc_3e7a008f","updated":"2025-04-24 09:35:41.000000000","message":"Hey Jay, the intent here was to stop using eventlet for the notification listener. It will still be eventlet behind other oslo stuff, I saw on a different review that you tested the in progress oslo.service change to use threading/cotelaydon?(probably misspelled that). I think (and hope) that is the way forward too.\n\nOn looking at this again, I think removing the eventlet monkey_patching should not be part of this. I\u0027ll investigate and update this with a better commit message and reno.","commit_id":"b132342b5868160633e606d3d0a6afbb804cbfd2"}]}