)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31664,"name":"Omer Schwartz","email":"oschwart@redhat.com","username":"oschwart"},"change_message_id":"da9b9488ae70256e91a1c023e8500b35b1c753e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0ac8bbd6_af396917","updated":"2022-10-24 16:15:40.000000000","message":"Looks good to me","commit_id":"c450dfca52fa99aab4f94dab4a93b3ff229731f5"},{"author":{"_account_id":11628,"name":"Michael Johnson","email":"johnsomor@gmail.com","username":"johnsom"},"change_message_id":"62bdb60620f8f0144d1befe2e1b2e64ac70b0b7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1b1f79d0_4e5c8b15","updated":"2023-06-15 17:25:16.000000000","message":"LGTM\nZuul -1 is due to a cirros issue being fixed in devstack","commit_id":"134d4728cad95e567fd886be3cf1e89150696c40"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"bd7da25947571803f1662c08ce69d1f5c4c5c48f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"32da5d53_59af323f","updated":"2023-06-21 11:25:13.000000000","message":"recheck fix for cirros 0.6.2 merged","commit_id":"134d4728cad95e567fd886be3cf1e89150696c40"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"008ec669ac959705fc815246446d03f5c5e4c5bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2e324c18_8b332b62","updated":"2023-06-22 05:59:26.000000000","message":"recheck octavia-grenade nova issue","commit_id":"134d4728cad95e567fd886be3cf1e89150696c40"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"ba0d7bf1ae27851b852b4ebc6981e861d3cf6251","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2cbc620a_bd050d0b","updated":"2023-06-21 19:59:13.000000000","message":"recheck octavia-v2-dsvm-scenario-traffic-ops TIMEOUT\ndeployment looked really slow, took almost 40min to build the amphora image","commit_id":"134d4728cad95e567fd886be3cf1e89150696c40"}],"octavia/db/repositories.py":[{"author":{"_account_id":34429,"name":"Tom Weininger","email":"dienste@weinimo.de","username":"tweining"},"change_message_id":"a592ad98b89774caaac4121fbb56547a8e20cdd8","unresolved":true,"context_lines":[{"line_number":1359,"context_line":"        lb \u003d {}"},{"line_number":1360,"context_line":"        listeners \u003d {}"},{"line_number":1361,"context_line":"        pools \u003d {}"},{"line_number":1362,"context_line":"        for row in rows:"},{"line_number":1363,"context_line":"            row \u003d row._mapping"},{"line_number":1364,"context_line":"            if not lb:"},{"line_number":1365,"context_line":"                lb[\u0027id\u0027] \u003d row[\u0027id\u0027]"},{"line_number":1366,"context_line":"                lb[\u0027enabled\u0027] \u003d row[\u0027enabled\u0027] \u003d\u003d 1"}],"source_content_type":"text/x-python","patch_set":4,"id":"1f98c3fd_ad1ee105","line":1363,"range":{"start_line":1362,"start_character":8,"end_line":1363,"end_character":30},"updated":"2023-04-28 14:20:07.000000000","message":"Why not `for row in rows.mappings():` as suggested by the migration guide? `row._mapping` is obviously not intended to be used outside sqlalchemy.","commit_id":"5776a3c39b7eacff6b44d547d66e27c254e254d3"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"4e253ea7c33aa06e8e767a202a9cd1909d16f670","unresolved":true,"context_lines":[{"line_number":1359,"context_line":"        lb \u003d {}"},{"line_number":1360,"context_line":"        listeners \u003d {}"},{"line_number":1361,"context_line":"        pools \u003d {}"},{"line_number":1362,"context_line":"        for row in rows:"},{"line_number":1363,"context_line":"            row \u003d row._mapping"},{"line_number":1364,"context_line":"            if not lb:"},{"line_number":1365,"context_line":"                lb[\u0027id\u0027] \u003d row[\u0027id\u0027]"},{"line_number":1366,"context_line":"                lb[\u0027enabled\u0027] \u003d row[\u0027enabled\u0027] \u003d\u003d 1"}],"source_content_type":"text/x-python","patch_set":4,"id":"629c49e6_79d517aa","line":1363,"range":{"start_line":1362,"start_character":8,"end_line":1363,"end_character":30},"in_reply_to":"1f98c3fd_ad1ee105","updated":"2023-04-28 14:59:19.000000000","message":"hmm you\u0027re right, I will update it\n(it was not in the guide when I started this commit)","commit_id":"5776a3c39b7eacff6b44d547d66e27c254e254d3"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"32d97fa3f09d3e2883d2beb3cf8cd904ba7f9f88","unresolved":false,"context_lines":[{"line_number":1359,"context_line":"        lb \u003d {}"},{"line_number":1360,"context_line":"        listeners \u003d {}"},{"line_number":1361,"context_line":"        pools \u003d {}"},{"line_number":1362,"context_line":"        for row in rows:"},{"line_number":1363,"context_line":"            row \u003d row._mapping"},{"line_number":1364,"context_line":"            if not lb:"},{"line_number":1365,"context_line":"                lb[\u0027id\u0027] \u003d row[\u0027id\u0027]"},{"line_number":1366,"context_line":"                lb[\u0027enabled\u0027] \u003d row[\u0027enabled\u0027] \u003d\u003d 1"}],"source_content_type":"text/x-python","patch_set":4,"id":"1edc2263_79a85d53","line":1363,"range":{"start_line":1362,"start_character":8,"end_line":1363,"end_character":30},"in_reply_to":"629c49e6_79d517aa","updated":"2023-05-03 07:07:41.000000000","message":"Done","commit_id":"5776a3c39b7eacff6b44d547d66e27c254e254d3"}]}
