)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"cf70072b66902e311ca6567dd98ccd098259497f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c6da22b7_17c2f551","updated":"2025-02-12 14:46:50.000000000","message":"Other than my nit I think this looks Ok.","commit_id":"365c76732b9710ccde67e36e34273a51987be0e1"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a267b9519c0d4bc3ccf9484ac67def66afcec892","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"25a2e7a5_3ddc61dc","updated":"2025-03-14 15:03:39.000000000","message":"-1 because this patch will also require a bump in the requirements neutron version, once merged and released.","commit_id":"d315e8b4ee1e5ffb20dc37e0dcc4a1722d57118f"},{"author":{"_account_id":34380,"name":"Maximilian Stinsky","email":"maximilian@stinsky.com","username":"mstinsky"},"change_message_id":"876a59d38e03d52ca0d1b2efb369a256992a40cd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6b14244f_8f466c14","updated":"2025-11-09 11:32:10.000000000","message":"Greetings,\nwe backported both patches required for vpn agent failover into our env and they work fine. Anything we can help to push this forward so we can get this merged?","commit_id":"d315e8b4ee1e5ffb20dc37e0dcc4a1722d57118f"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"0ff55e95bc510c7d752965f6fa25d3b0257b3e93","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"bac25302_89721a0b","updated":"2025-11-21 20:00:10.000000000","message":"recheck get new logs","commit_id":"d315e8b4ee1e5ffb20dc37e0dcc4a1722d57118f"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c56e95cf18b470a57966ac36f7b4b7d175978fd1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"24620869_5fff69ec","updated":"2025-02-14 08:40:55.000000000","message":"thanks\nlet\u0027s wait for the neutron change","commit_id":"d315e8b4ee1e5ffb20dc37e0dcc4a1722d57118f"},{"author":{"_account_id":31976,"name":"Bodo Petermann","email":"b.petermann@syseleven.de","username":"bpetermann"},"change_message_id":"24a8d9006137b238c40b5ae6a820eaf5849527f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"69b9e187_ec5bc8d9","updated":"2026-07-24 14:25:00.000000000","message":"Rebased, so now the requirements are up to date to ask for a recent-enough neutron","commit_id":"a7f48863de030ad7c8586dae92c8435dd239f223"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"72ef6af12794bba62d08b7796c1579049ee9c745","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2d3d3e70_1c774040","updated":"2026-07-31 07:59:59.000000000","message":"looks ok as depends-on is already included in 26.1","commit_id":"a7f48863de030ad7c8586dae92c8435dd239f223"}],"neutron_vpnaas/services/vpn/ovn/agent_monitor.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"cf70072b66902e311ca6567dd98ccd098259497f","unresolved":true,"context_lines":[{"line_number":49,"context_line":"              ovsdb_monitor.BaseEvent.ROW_UPDATE)"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    def __init__(self):"},{"line_number":52,"context_line":"        super().__init__(None)"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    @staticmethod"},{"line_number":55,"context_line":"    def _vpnagent_nb_cfg(row):"}],"source_content_type":"text/x-python","patch_set":1,"id":"34e04c1d_2796b04d","line":52,"updated":"2025-02-12 14:46:50.000000000","message":"Could the call on L88 just use (None)? This seems a little odd.","commit_id":"365c76732b9710ccde67e36e34273a51987be0e1"},{"author":{"_account_id":31976,"name":"Bodo Petermann","email":"b.petermann@syseleven.de","username":"bpetermann"},"change_message_id":"42c5be1faedbde4e20e3ef05a674fd436a63bb67","unresolved":false,"context_lines":[{"line_number":49,"context_line":"              ovsdb_monitor.BaseEvent.ROW_UPDATE)"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    def __init__(self):"},{"line_number":52,"context_line":"        super().__init__(None)"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"    @staticmethod"},{"line_number":55,"context_line":"    def _vpnagent_nb_cfg(row):"}],"source_content_type":"text/x-python","patch_set":1,"id":"178853d0_c0301394","line":52,"in_reply_to":"34e04c1d_2796b04d","updated":"2025-02-12 15:31:54.000000000","message":"changed it","commit_id":"365c76732b9710ccde67e36e34273a51987be0e1"}],"neutron_vpnaas/services/vpn/ovn/maintenance.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a267b9519c0d4bc3ccf9484ac67def66afcec892","unresolved":true,"context_lines":[{"line_number":17,"context_line":"from neutron.plugins.ml2.drivers.ovn.mech_driver.ovsdb import maintenance"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class VPNOVNMaintenancePeriodics:"},{"line_number":21,"context_line":"    \"\"\"Periodics to be run in the maintenance worker.\"\"\""},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"    def __init__(self, plugin, ovn_client):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9d5738c7_58dc514f","line":20,"range":{"start_line":20,"start_character":6,"end_line":20,"end_character":32},"updated":"2025-03-14 15:03:39.000000000","message":"no action: we need a shared definition for this (implemented in n-lib), that includes the lock method, the idl definitions and the plugin.","commit_id":"d315e8b4ee1e5ffb20dc37e0dcc4a1722d57118f"}]}