)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3789884150d6579f1361113a9ff9e237034f0c87","unresolved":true,"context_lines":[{"line_number":18,"context_line":"thread sending RPC updates to neutron-server will never"},{"line_number":19,"context_line":"start."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"This also needs to be backported to stable/2025.1 as the"},{"line_number":22,"context_line":"original change was merged there."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"[0] https://review.opendev.org/c/openstack/neutron/+/942916"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bcd0afbf_ce8ff710","line":22,"range":{"start_line":21,"start_character":0,"end_line":22,"end_character":33},"updated":"2025-06-13 06:04:29.000000000","message":"We have a problem with this: oslo.service 4.2.0 is not available in 2025.1. We can, however, implement ``FixedIntervalLoopingCall`` and ``LoopingCallBase`` in 2025.1 in Neutron, to avoid this issue. I would recommend this alternative.\n\nWe can also revert in 2025.1 the eventlet removal for the metadata agents, but that will be a mess in the code.\n\nThis is a high priority issue, let\u0027s discuss it today.","commit_id":"e28b9169bee62192a1eb4c4f10f902c28a80bd32"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"d6f921be5da12245311ef7ab4e7cdb80d2da05e2","unresolved":false,"context_lines":[{"line_number":18,"context_line":"thread sending RPC updates to neutron-server will never"},{"line_number":19,"context_line":"start."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"This also needs to be backported to stable/2025.1 as the"},{"line_number":22,"context_line":"original change was merged there."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"[0] https://review.opendev.org/c/openstack/neutron/+/942916"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3d0f0ba8_92e093b7","line":22,"range":{"start_line":21,"start_character":0,"end_line":22,"end_character":33},"in_reply_to":"38304607_ed059986","updated":"2025-06-13 13:49:30.000000000","message":"Done","commit_id":"e28b9169bee62192a1eb4c4f10f902c28a80bd32"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4f5a439cca8a308fa54a5d39125d7a1e2c4f87f1","unresolved":true,"context_lines":[{"line_number":18,"context_line":"thread sending RPC updates to neutron-server will never"},{"line_number":19,"context_line":"start."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"This also needs to be backported to stable/2025.1 as the"},{"line_number":22,"context_line":"original change was merged there."},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"[0] https://review.opendev.org/c/openstack/neutron/+/942916"},{"line_number":25,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"38304607_ed059986","line":22,"range":{"start_line":21,"start_character":0,"end_line":22,"end_character":33},"in_reply_to":"bcd0afbf_ce8ff710","updated":"2025-06-13 09:51:07.000000000","message":"I think for 2025.1 we can do something like this: https://review.opendev.org/c/openstack/neutron/+/952561","commit_id":"e28b9169bee62192a1eb4c4f10f902c28a80bd32"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3789884150d6579f1361113a9ff9e237034f0c87","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"59ded236_1fda81fe","updated":"2025-06-13 06:04:29.000000000","message":"-1 for visibility. The code is OK. I would change the commit message only.","commit_id":"e28b9169bee62192a1eb4c4f10f902c28a80bd32"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"31528a1f3eb6dcd0f65cdb2a66efc10f9979d35e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"200f9a1b_fb6b1ff6","updated":"2025-06-13 15:29:02.000000000","message":"recheck neutron-functional","commit_id":"3ff9cfd026d29392b049d4e9a51664a86d5bead1"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"c714cdd71012a1ea6167753ccc04e36085091897","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"372d8c0b_b3b4cbf3","updated":"2025-06-13 17:57:40.000000000","message":"recheck unrelated customize image failure","commit_id":"3ff9cfd026d29392b049d4e9a51664a86d5bead1"}],"neutron/agent/metadata/agent.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"68e7c050914dd4f16cd1942ac186434721dc24c6","unresolved":true,"context_lines":[{"line_number":235,"context_line":"        self._server \u003d socketserver.ThreadingUnixStreamServer("},{"line_number":236,"context_line":"            file_socket, MetadataProxyHandler)"},{"line_number":237,"context_line":"        MetadataProxyHandler._conf \u003d self.conf"},{"line_number":238,"context_line":"        self._init_state_reporting()"},{"line_number":239,"context_line":"        self._server.serve_forever()"}],"source_content_type":"text/x-python","patch_set":1,"id":"d966999c_0a585e40","line":238,"updated":"2025-06-11 19:56:26.000000000","message":"While this looks like it should work, my testing shows self._report_state is never actually called from L210. I think it\u0027s because the ThreadingUnixStreamServer is not actually continuously running (?) but called on-demand as requests arrive.\n\nIf I put a direct call to self._report_state() here I do see :-) but it only lasts about 30 seconds, then back to XXX, showing there is no thread in the background.\n\nWe will need to find another way to make this work, I just wanted to post this change so it\u0027s obvious this init call here was dropped.","commit_id":"f1e0bda214dfe0c8333e21f99d4f6a58eb87ed14"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"39c8588e7c95d587abd56aabac325d02fc782f4d","unresolved":false,"context_lines":[{"line_number":235,"context_line":"        self._server \u003d socketserver.ThreadingUnixStreamServer("},{"line_number":236,"context_line":"            file_socket, MetadataProxyHandler)"},{"line_number":237,"context_line":"        MetadataProxyHandler._conf \u003d self.conf"},{"line_number":238,"context_line":"        self._init_state_reporting()"},{"line_number":239,"context_line":"        self._server.serve_forever()"}],"source_content_type":"text/x-python","patch_set":1,"id":"40059d5b_94e5c2d6","line":238,"in_reply_to":"d966999c_0a585e40","updated":"2025-06-12 16:37:08.000000000","message":"Ignore this comment, it was more related to oslo.service than anything.","commit_id":"f1e0bda214dfe0c8333e21f99d4f6a58eb87ed14"}]}