)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"1562894511f32b1905a7b82336ae4d075c290036","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d080773c_e20e9e1d","updated":"2024-05-23 10:25:11.000000000","message":"I wonder if we really want to backport these changes to stable branches.\n\nBeyond the fact that this eventlet debug feature would surely at some point introduce race conditions, our initial motivation behind this patch was to allow the use of the asyncio hub. A hub that won\u0027t be actionable in stable branches.\n\nOur initial motivation, the Eventlet removal, is more in front of us than behind us.\n\nOn one hand, but without not yet known issues, I\u0027d argue that that\u0027s patch would anticipate some race conditions solving. On the other hand, if we really want to that, we should first ensure that all the previous other patches related to this topic in oslo.log are already well backported to these stable branches. I don\u0027t know their states and they are possibily all already backported, at least to caracal (2024.1)","commit_id":"a9269f6c18c8d324f5043982bdbbc6527211c282"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"71549a35cbbf9c4e06ab0b08260e6aa0435823b5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7b113b7a_84e85e78","in_reply_to":"92ac0153_9f20350e","updated":"2024-05-23 15:23:08.000000000","message":"I\u0027m not against backporting it. It could solve issues. Just want to be sure that all pre-requisites are already there. I don\u0027t think that that\u0027s the case because I can easily found missing backports related to the same topic, example:\n- https://opendev.org/openstack/oslo.log/commit/142fde8224c8994b94518803fe5005cdeec6a37e\n\nIf we want to backport it, we should first backport the other related patch (those related to eventlet). Identifying missing patches could be done by comparing master and caracal:\n- https://opendev.org/openstack/oslo.log/commits/branch/master\n- https://opendev.org/openstack/oslo.log/commits/branch/stable/2024.1","commit_id":"a9269f6c18c8d324f5043982bdbbc6527211c282"},{"author":{"_account_id":31245,"name":"Daniel Bengtsson","email":"dbengt@redhat.com","username":"damani42"},"change_message_id":"4b9f4e789a6dda440ac14e14b5d9d0f285715c33","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"92ac0153_9f20350e","in_reply_to":"d080773c_e20e9e1d","updated":"2024-05-23 11:30:52.000000000","message":"Ok I understand your point. You\u0027re right about the motivation. I think it can be good to backport it, but if you think it\u0027s not a good idea, we don\u0027t need to do it.","commit_id":"a9269f6c18c8d324f5043982bdbbc6527211c282"}]}