)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"32407d729fa635b24129c18f65d5c0f9eb163b41","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0aa88a46_eaa7a5dc","updated":"2023-01-23 06:47:23.000000000","message":"nice catch! wondering if this needs a launchpad bug in case a backport to previous branches is needed","commit_id":"c3e5718f7fb9db372f88eeafe2c6ab797e319adf"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"1fd59b9f3b1d7a9b6cf4d6a49cc647f8f279811a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b8c3ec04_516f055f","updated":"2023-01-21 19:18:57.000000000","message":"recheck neutron-ovn-rally-task TIMED_OUT","commit_id":"c3e5718f7fb9db372f88eeafe2c6ab797e319adf"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"6dca441758a8f5b7494dbe1b58a14544d18e56d3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dfc3daeb_d4b104a6","updated":"2023-01-24 12:50:15.000000000","message":"Nice to refresh/make deeper my not so perfect knowledge of these things :-)","commit_id":"55af3f34e59f37432886e7b71455f1c6d355a686"}],"neutron/agent/linux/ip_lib.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ce43df6cd1ebadf8911327a33c3dc50f0689add0","unresolved":true,"context_lines":[{"line_number":599,"context_line":"                    address\u003daddress,"},{"line_number":600,"context_line":"                    reason\u003d_(\u0027Address not present on interface\u0027))"},{"line_number":601,"context_line":"            # Since both \u0027dadfailed\u0027 and \u0027tentative\u0027 will be set if DAD fails,"},{"line_number":602,"context_line":"            # check \u0027dadfailed\u0027 first so we don\u0027t loop unneccessarily"},{"line_number":603,"context_line":"            if addr_info[\u0027dadfailed\u0027]:"},{"line_number":604,"context_line":"                raise AddressNotReady("},{"line_number":605,"context_line":"                    address\u003daddress, reason\u003d_(\u0027Duplicate address detected\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"b93243cf_775ada7e","line":602,"range":{"start_line":602,"start_character":38,"end_line":602,"end_character":69},"updated":"2023-01-23 19:39:20.000000000","message":"Thanks for the review Luis, but I realize this part of my comment is not true, as the \"not tentative\" check below won\u0027t match in this case. I think it\u0027s still useful to change the order to make it obvious, and to add tests, but I\u0027ll update this and the commit message accordingly.","commit_id":"c3e5718f7fb9db372f88eeafe2c6ab797e319adf"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"59aa2e2771ade4330bb93f9679aac5b138511a2c","unresolved":true,"context_lines":[{"line_number":599,"context_line":"                    address\u003daddress,"},{"line_number":600,"context_line":"                    reason\u003d_(\u0027Address not present on interface\u0027))"},{"line_number":601,"context_line":"            # Since both \u0027dadfailed\u0027 and \u0027tentative\u0027 will be set if DAD fails,"},{"line_number":602,"context_line":"            # check \u0027dadfailed\u0027 first so we don\u0027t loop unneccessarily"},{"line_number":603,"context_line":"            if addr_info[\u0027dadfailed\u0027]:"},{"line_number":604,"context_line":"                raise AddressNotReady("},{"line_number":605,"context_line":"                    address\u003daddress, reason\u003d_(\u0027Duplicate address detected\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"54d020fd_c82e226d","line":602,"range":{"start_line":602,"start_character":38,"end_line":602,"end_character":69},"in_reply_to":"25628ff9_b761f9da","updated":"2023-01-24 14:45:24.000000000","message":"I guess I might as well since a job failed...","commit_id":"c3e5718f7fb9db372f88eeafe2c6ab797e319adf"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"b8da2163eb668f6945981579e150b551a475f1e9","unresolved":false,"context_lines":[{"line_number":599,"context_line":"                    address\u003daddress,"},{"line_number":600,"context_line":"                    reason\u003d_(\u0027Address not present on interface\u0027))"},{"line_number":601,"context_line":"            # Since both \u0027dadfailed\u0027 and \u0027tentative\u0027 will be set if DAD fails,"},{"line_number":602,"context_line":"            # check \u0027dadfailed\u0027 first so we don\u0027t loop unneccessarily"},{"line_number":603,"context_line":"            if addr_info[\u0027dadfailed\u0027]:"},{"line_number":604,"context_line":"                raise AddressNotReady("},{"line_number":605,"context_line":"                    address\u003daddress, reason\u003d_(\u0027Duplicate address detected\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"f7629152_ffb7ee8c","line":602,"range":{"start_line":602,"start_character":38,"end_line":602,"end_character":69},"in_reply_to":"54d020fd_c82e226d","updated":"2023-01-24 14:54:27.000000000","message":"Done","commit_id":"c3e5718f7fb9db372f88eeafe2c6ab797e319adf"},{"author":{"_account_id":23567,"name":"Luis Tomas Bolivar","email":"ltomasbo@redhat.com","username":"ltomasbo"},"change_message_id":"2fa50bf108c8ccd806c979353da4e3792ca81ca8","unresolved":true,"context_lines":[{"line_number":599,"context_line":"                    address\u003daddress,"},{"line_number":600,"context_line":"                    reason\u003d_(\u0027Address not present on interface\u0027))"},{"line_number":601,"context_line":"            # Since both \u0027dadfailed\u0027 and \u0027tentative\u0027 will be set if DAD fails,"},{"line_number":602,"context_line":"            # check \u0027dadfailed\u0027 first so we don\u0027t loop unneccessarily"},{"line_number":603,"context_line":"            if addr_info[\u0027dadfailed\u0027]:"},{"line_number":604,"context_line":"                raise AddressNotReady("},{"line_number":605,"context_line":"                    address\u003daddress, reason\u003d_(\u0027Duplicate address detected\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"25628ff9_b761f9da","line":602,"range":{"start_line":602,"start_character":38,"end_line":602,"end_character":69},"in_reply_to":"b93243cf_775ada7e","updated":"2023-01-24 06:41:03.000000000","message":"perhaps now it will be better with:\n\n    if addr_info[\u0027tentative\u0027]:\n        return False\n    return True","commit_id":"c3e5718f7fb9db372f88eeafe2c6ab797e319adf"}]}