)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":597,"name":"Pete Zaitcev","email":"zaitcev@kotori.zaitcev.us","username":"zaitcev"},"change_message_id":"e51901fecb4beb14951dca2eaeb4392f1cd4e34c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7b2ba726_c2a48358","updated":"2024-02-07 23:19:29.000000000","message":"Looks good. Mechanical. The duplicate was a little funny.","commit_id":"7557594923212a5701dd4897bef4bbdae95f08c6"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"3c8c521eff0d5fd3402b11f24bc642d652a51247","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"31cf694e_3aa2732f","updated":"2024-02-08 02:29:22.000000000","message":"Will +A it as soon as it comes back ok from the recheck!","commit_id":"ce9e56a6d17ccab33d7d7c9f0cdff6a96b37fa31"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"347818e92421128d3e20d1f6f9dd21eba22c6be2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ee227040_01aca0ad","updated":"2024-02-08 02:30:10.000000000","message":"lol, just as I sent my review up apparently 😋","commit_id":"ce9e56a6d17ccab33d7d7c9f0cdff6a96b37fa31"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"d2e0d9f7d51d5f4fc5c1cb7056b8f4c2f0323653","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b4b9c3f7_19d8c234","updated":"2024-02-08 01:11:18.000000000","message":"recheck\n\nLooks like some flakey probe tests?\n```\nrun-last-failure: rerun previous 2 failures\n\ntest/probe/test_sharder.py::TestContainerSharding::test_listing_under_populated_replica PASSED [ 50%]\ntest/probe/test_sharder.py::TestManagedContainerSharding::test_manage_shard_ranges_repair_shard FAILED [100%]\n```","commit_id":"ce9e56a6d17ccab33d7d7c9f0cdff6a96b37fa31"}],"tox.ini":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"3c8c521eff0d5fd3402b11f24bc642d652a51247","unresolved":true,"context_lines":[{"line_number":146,"context_line":"# it\u0027s not a bug that we aren\u0027t using all of hacking, ignore:"},{"line_number":147,"context_line":"# H101: Use TODO(NAME)"},{"line_number":148,"context_line":"# H202: assertRaises Exception too broad"},{"line_number":149,"context_line":"# H211/H212: Use assert{Is,IsNot}instance"},{"line_number":150,"context_line":"# H214: Use assertIn/NotIn ..."},{"line_number":151,"context_line":"# H216: The unittest.mock module should be used rather than ..."},{"line_number":152,"context_line":"# H301: one import per line"}],"source_content_type":"text/x-properties","patch_set":2,"id":"99f2b662_f8095842","side":"PARENT","line":149,"updated":"2024-02-08 02:29:22.000000000","message":"Yay we can remove some ignores!","commit_id":"76ca11773eb6508e1140da2b04ddb2ebc59cd753"}]}