)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"cd92305886dc57d44b69029e4de2d6a1d78c69b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b8e4a9ca_c382cde5","updated":"2022-05-13 14:26:47.000000000","message":"2 cents from testing..\n\nWe have tried cherry-pick of this patch with tripleo-ci wallaby jobs(where we are hitting bug[2]) in testproject[3].\n\nWith this patch neutron-openvswitch-agent on undercloud is not starting correctly.\n\nneutron_ovs_agent container is unhealthy\n\nhttps://logserver.rdoproject.org/40/36140/49/check/periodic-tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001-wallaby/aceb20b/logs/undercloud/var/log/extra/podman/podman_allinfo.log.txt.gz\n\n6a4a8ba8ef7e 192.168.24.1:8787/tripleowallabycentos9/openstack-neutron-openvswitch-agent:f83994cb11ef42e4c94ee49af211c863-updated-20220513064107 kolla_start 45 minutes ago Up 45 minutes ago (unhealthy) neutron_ovs_agent\n\nNo openvswitch entry in \"openstack network agent list\"\n\n+--------------------------------------+----------------+--------------------------------------+-------------------+-------+-------+----------------------+\n| ID | Agent Type | Host | Availability Zone | Alive | State | Binary |\n+--------------------------------------+----------------+--------------------------------------+-------------------+-------+-------+----------------------+\n| 385461b2-c04d-4bb3-8f5e-cf6d2e3bd524 | Baremetal Node | f2c6882b-285f-494a-86b7-01c24fbe1206 | None | :-) | UP | ironic-neutron-agent |\n| 72bb93d0-8465-4eff-a1a3-18983fa72328 | Baremetal Node | 820119f8-6432-4883-a171-363c99f6fc53 | None | :-) | UP | ironic-neutron-agent |\n| 82f73f9b-31be-42c0-98e0-2e52c7e15472 | Baremetal Node | 4022af37-9502-412f-9df2-e94af29a93aa | None | :-) | UP | ironic-neutron-agent |\n| b78283ec-a9a2-41fb-a8fa-248499eee23f | Baremetal Node | 6c8c2707-5d5f-49f8-b3b4-b3b4719321c9 | None | :-) | UP | ironic-neutron-agent |\n| e5dbf620-8d9f-40c3-9319-3d2c9848f17b | DHCP agent | undercloud.localdomain | nova | :-) | UP | neutron-dhcp-agent |\n| fb42b407-25ff-456a-b772-c279a0d8e362 | L3 agent | undercloud.localdomain | nova | :-) | UP | neutron-l3-agent |\n+--------------------------------------+----------------+--------------------------------------+-------------------+-------+-------+----------------------+\n\nopenvswitch-agent.log at [4]\n\n\n[1] https://review.opendev.org/c/openstack/ovsdbapp/+/841429\n[2] Bug #1964940 “Compute tests are failing with failed to reach ACT...” : Bugs : tripleo\n[3] https://review.rdoproject.org/r/c/testproject/+/36140\n[4] https://logserver.rdoproject.org/40/36140/49/check/periodic-tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001-wallaby/aceb20b/logs/undercloud/var/log/containers/neutron/openvswitch-agent.log.txt.gz","commit_id":"643d3cbd7989d127f5b0947bc46856361b9c7922"}],"ovsdbapp/event.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"30a93f152867c75d1de4966657d5af55ed4a7efa","unresolved":true,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    # pylint: disable\u003dimport-outside-toplevel"},{"line_number":47,"context_line":"    from eventlet import patcher"},{"line_number":48,"context_line":"    gt_modules \u003d getattr(patcher, \u0027_green_%s_modules\u0027 % \u0027thread\u0027)"},{"line_number":49,"context_line":"    if not gt_modules:"},{"line_number":50,"context_line":"        # The system is using eventlet but the \"threading\" module has not been"},{"line_number":51,"context_line":"        # patched (e.g.: Windows)."}],"source_content_type":"text/x-python","patch_set":4,"id":"529fd7c7_b16e69f5","line":48,"range":{"start_line":48,"start_character":35,"end_line":48,"end_character":64},"updated":"2022-05-19 09:30:12.000000000","message":"why this string operation is needed here?","commit_id":"643d3cbd7989d127f5b0947bc46856361b9c7922"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"1f6289773762efbbe75bcf53772c17d03c7815b6","unresolved":true,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    for mod_name, module in gt_modules():"},{"line_number":55,"context_line":"        if mod_name \u003d\u003d \u0027threading\u0027:"},{"line_number":56,"context_line":"            return module"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    return threading"},{"line_number":59,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5eb1e21c_f3ba8487","line":56,"updated":"2022-05-12 16:22:19.000000000","message":"It looks like it\u0027s gonna return the greenthread threading instead of the original python threading: https://github.com/eventlet/eventlet/blob/master/eventlet/patcher.py#L480","commit_id":"643d3cbd7989d127f5b0947bc46856361b9c7922"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"1f6289773762efbbe75bcf53772c17d03c7815b6","unresolved":true,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"    # pylint: disable\u003dimport-outside-toplevel"},{"line_number":47,"context_line":"    from eventlet import patcher"},{"line_number":48,"context_line":"    gt_modules \u003d getattr(patcher, \u0027_green_%s_modules\u0027 % \u0027thread\u0027)"},{"line_number":49,"context_line":"    if not gt_modules:"},{"line_number":50,"context_line":"        # The system is using eventlet but the \"threading\" module has not been"},{"line_number":51,"context_line":"        # patched (e.g.: Windows)."},{"line_number":52,"context_line":"        return threading"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    for mod_name, module in gt_modules():"},{"line_number":55,"context_line":"        if mod_name \u003d\u003d \u0027threading\u0027:"},{"line_number":56,"context_line":"            return module"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    return threading"},{"line_number":59,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7e46c5c7_b6b732ae","line":56,"range":{"start_line":48,"start_character":0,"end_line":56,"end_character":25},"updated":"2022-05-12 16:22:19.000000000","message":"try:\n    for mod_name, module in patches._green_thread_modules():\n        if mode_name \u003d\u003d \u0027threading\u0027:\n            return module\nexcept TypeError:\n    # _green_thread_modules is set to None\n    return threading","commit_id":"643d3cbd7989d127f5b0947bc46856361b9c7922"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"087cd06af56daa42685ed72deebfcc521d80b147","unresolved":true,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    for mod_name, module in gt_modules():"},{"line_number":55,"context_line":"        if mod_name \u003d\u003d \u0027threading\u0027:"},{"line_number":56,"context_line":"            return module"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    return threading"},{"line_number":59,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"fd6691ca_7ca83fcd","line":56,"in_reply_to":"5eb1e21c_f3ba8487","updated":"2022-05-12 16:24:08.000000000","message":"I think what you want to call is original(): https://github.com/eventlet/eventlet/blob/ef36eb703de60b899f1e16a744d1dfaf33928906/eventlet/patcher.py#L183","commit_id":"643d3cbd7989d127f5b0947bc46856361b9c7922"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a74bf03d3c14478d7ac7b7dfef62a3485971eca7","unresolved":true,"context_lines":[{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    for mod_name, module in gt_modules():"},{"line_number":55,"context_line":"        if mod_name \u003d\u003d \u0027threading\u0027:"},{"line_number":56,"context_line":"            return module"},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    return threading"},{"line_number":59,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"362d9db3_55d7e4e7","line":56,"in_reply_to":"fd6691ca_7ca83fcd","updated":"2022-06-03 07:26:19.000000000","message":"Check oslo.privsep. That was implemented time ago.","commit_id":"643d3cbd7989d127f5b0947bc46856361b9c7922"}]}