)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ada5f9ddaefa97a3f0286407b4d52e20b0e80f7b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a7c0b91e_768e5ab3","updated":"2024-11-11 08:52:15.000000000","message":"Some router\u0027s tests failed in the integration suit, TODO: fix them :-)","commit_id":"7d7151dee4a5c7e187344f85676b093c399cc77c"},{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"39e85eb9970156a2cfe224df1aaf7cca8380a212","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ce310d48_a49f802f","updated":"2024-11-19 15:23:12.000000000","message":"Commented on some suspect code, not sure if it\u0027s important.","commit_id":"75c44dd3a3d96fa7e942e70440959a619968c323"},{"author":{"_account_id":35133,"name":"Jan Jasek","email":"jjasek@redhat.com","username":"janjasek"},"change_message_id":"ad9d81a75808690b3cb00b2a4be1c1e4d3e621b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"9bee74a0_f9f1cd71","updated":"2024-12-06 17:26:57.000000000","message":"LGTM, thanks!","commit_id":"1d00997f50936976c75f10c95fb489549ca0023e"}],"openstack_dashboard/api/neutron.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"bb8ac41598be3c2560f9f2ee1d2628837fc21780","unresolved":true,"context_lines":[{"line_number":1977,"context_line":"                                              subnet_id\u003dsubnet_id)"},{"line_number":1978,"context_line":"    if port_id:"},{"line_number":1979,"context_line":"        return client.add_interface_to_router(router\u003drouter_id,"},{"line_number":1980,"context_line":"                                              port_id\u003dport_id)"},{"line_number":1981,"context_line":""},{"line_number":1982,"context_line":""},{"line_number":1983,"context_line":"@profiler.trace"}],"source_content_type":"text/x-python","patch_set":3,"id":"c90e2a3b_2b9149a5","line":1980,"updated":"2024-11-19 17:51:37.000000000","message":"Same comment as below - should this be done in a single call with both values? The add and remove calls support both simultaineously.","commit_id":"75c44dd3a3d96fa7e942e70440959a619968c323"},{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"39e85eb9970156a2cfe224df1aaf7cca8380a212","unresolved":true,"context_lines":[{"line_number":1984,"context_line":"def router_remove_interface(request, router_id, subnet_id\u003dNone, port_id\u003dNone):"},{"line_number":1985,"context_line":"    client \u003d networkclient(request)"},{"line_number":1986,"context_line":"    if subnet_id:"},{"line_number":1987,"context_line":"        return client.remove_interface_from_router(router\u003drouter_id,"},{"line_number":1988,"context_line":"                                                   subnet_id\u003dsubnet_id)"},{"line_number":1989,"context_line":"    if port_id:"},{"line_number":1990,"context_line":"        return client.remove_interface_from_router(router\u003drouter_id,"}],"source_content_type":"text/x-python","patch_set":3,"id":"d9ca96c9_428070ec","line":1987,"updated":"2024-11-19 15:23:12.000000000","message":"What if both subdnet_id and port_id are passed? This will ignore the port_id.","commit_id":"75c44dd3a3d96fa7e942e70440959a619968c323"}],"openstack_dashboard/dashboards/project/routers/views.py":[{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"39e85eb9970156a2cfe224df1aaf7cca8380a212","unresolved":true,"context_lines":[{"line_number":223,"context_line":"                   \u0027tenant_id\u0027: router[\u0027tenant_id\u0027],"},{"line_number":224,"context_line":"                   \u0027name\u0027: router[\u0027name\u0027],"},{"line_number":225,"context_line":"                   \u0027admin_state\u0027: router[\u0027is_admin_state_up\u0027]}"},{"line_number":226,"context_line":"        if hasattr(router, \u0027distributed\u0027):"},{"line_number":227,"context_line":"            initial[\u0027mode\u0027] \u003d (\u0027distributed\u0027 if router.is_distributed"},{"line_number":228,"context_line":"                               else \u0027centralized\u0027)"},{"line_number":229,"context_line":"        if hasattr(router, \u0027ha\u0027):"}],"source_content_type":"text/x-python","patch_set":3,"id":"0aac57d6_190071ec","line":226,"updated":"2024-11-19 15:23:12.000000000","message":"should this be \"is_distributed\"?","commit_id":"75c44dd3a3d96fa7e942e70440959a619968c323"},{"author":{"_account_id":8648,"name":"Radomir Dopieralski","email":"openstack@dopieralski.pl","username":"thesheep"},"change_message_id":"39e85eb9970156a2cfe224df1aaf7cca8380a212","unresolved":true,"context_lines":[{"line_number":226,"context_line":"        if hasattr(router, \u0027distributed\u0027):"},{"line_number":227,"context_line":"            initial[\u0027mode\u0027] \u003d (\u0027distributed\u0027 if router.is_distributed"},{"line_number":228,"context_line":"                               else \u0027centralized\u0027)"},{"line_number":229,"context_line":"        if hasattr(router, \u0027ha\u0027):"},{"line_number":230,"context_line":"            initial[\u0027ha\u0027] \u003d router.is_ha"},{"line_number":231,"context_line":"        return initial"}],"source_content_type":"text/x-python","patch_set":3,"id":"230e9de2_6925ea73","line":229,"updated":"2024-11-19 15:23:12.000000000","message":"ditto","commit_id":"75c44dd3a3d96fa7e942e70440959a619968c323"}]}