)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"f75147a3d4a30ba3798950598f22a5625f5cae75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"842f9f61_429643a0","updated":"2022-09-02 15:56:08.000000000","message":"this would be fine\n\nhttps://review.opendev.org/c/openstack/swift/+/855681\n\n... thanks for debugging the test","commit_id":"69cb35072b46b28f4e9c142f15d3ef80a4de0f2c"}],"test/probe/test_sharder.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"f75147a3d4a30ba3798950598f22a5625f5cae75","unresolved":true,"context_lines":[{"line_number":437,"context_line":"        # part-specific index"},{"line_number":438,"context_line":"        node_copy \u003d dict(node)"},{"line_number":439,"context_line":"        node_copy.pop(\u0027index\u0027, None)"},{"line_number":440,"context_line":"        return node_copy"},{"line_number":441,"context_line":""},{"line_number":442,"context_line":""},{"line_number":443,"context_line":"class BaseAutoContainerSharding(BaseTestContainerSharding):"}],"source_content_type":"text/x-python","patch_set":1,"id":"8da140d6_1913a6a6","line":440,"updated":"2022-09-02 15:56:08.000000000","message":"this function does not use self and it may not be reasonable for it to be a method since it doesn\u0027t seem to modify any state.","commit_id":"69cb35072b46b28f4e9c142f15d3ef80a4de0f2c"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"f75147a3d4a30ba3798950598f22a5625f5cae75","unresolved":true,"context_lines":[{"line_number":3485,"context_line":"        self.container_replicators.once("},{"line_number":3486,"context_line":"            additional_args\u003d\u0027--partitions\u003d%s\u0027 % shard_part)"},{"line_number":3487,"context_line":"        node0 \u003d self.pop_index(self.brain.nodes[0])"},{"line_number":3488,"context_line":"        for node in [n for n in shard_nodes if self.pop_index(n) !\u003d node0]:"},{"line_number":3489,"context_line":"            self.assert_container_state("},{"line_number":3490,"context_line":"                node, \u0027unsharded\u0027, 2, account\u003dshard_ranges[0].account,"},{"line_number":3491,"context_line":"                container\u003dshard_ranges[0].container, part\u003dshard_part)"}],"source_content_type":"text/x-python","patch_set":1,"id":"8d8f36f3_f970940c","line":3488,"updated":"2022-09-02 15:56:08.000000000","message":"oic, so it\u0027s an index-less comparision - but you don\u0027t want to modify the dicts","commit_id":"69cb35072b46b28f4e9c142f15d3ef80a4de0f2c"}]}