)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cc86c3001b952866c58d4909dc0f6022842ec150","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4977e2ec_edaeb035","updated":"2022-05-03 21:33:25.000000000","message":"I think it\u0027s better to be explicit, and it looks like that\u0027s the direction the type-checking community is going.  Plus, we currently have a mix of explicit and implicit optionals, so might as well be consistent.","commit_id":"d3afa4db009b9c35176c3640d63ce6a0bca6f766"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"b622a37f0804b4c97351ccd94c8699684396708d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"16c6677a_8086ad08","updated":"2022-05-10 19:18:17.000000000","message":"LGTM","commit_id":"d3afa4db009b9c35176c3640d63ce6a0bca6f766"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"da17e56f7c5f9337246bb5cc32eb8406cefb7c35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"baaae912_ed1f1e06","updated":"2022-05-12 16:43:17.000000000","message":"recheck\n\ncinder-plugin-ceph-tempest","commit_id":"d3afa4db009b9c35176c3640d63ce6a0bca6f766"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"27ea615b2ac3ce44b4047ccfc069dd3e92068ee7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"71c47d96_a244ff3d","updated":"2022-05-12 22:31:08.000000000","message":"recheck\n\ncinder-plugin-ceph-tempest OOM again","commit_id":"d3afa4db009b9c35176c3640d63ce6a0bca6f766"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"119d9cab38e60e15fd7be2171d2ae349359ab256","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"6fad7c10_897a8b86","updated":"2022-05-11 13:35:57.000000000","message":"recheck cinder-plugin-ceph-tempest - 43 failures, lots of 500s from identity service, which Eric has found are related to OOM errors","commit_id":"d3afa4db009b9c35176c3640d63ce6a0bca6f766"}],"cinder/backup/api.py":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"5f0d065815b230ec05db8c03ed7bdbdfc39f56cb","unresolved":true,"context_lines":[{"line_number":123,"context_line":"                marker: Optional[str] \u003d None,"},{"line_number":124,"context_line":"                limit: Optional[int] \u003d None,"},{"line_number":125,"context_line":"                offset: Optional[int] \u003d None,"},{"line_number":126,"context_line":"                sort_keys: Optional[List[str]] \u003d None,"},{"line_number":127,"context_line":"                sort_dirs: Optional[List[str]] \u003d None) -\u003e \u0027objects.BackupList\u0027:"},{"line_number":128,"context_line":"        context.authorize(policy.GET_ALL_POLICY)"},{"line_number":129,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"b00fc06a_69222cc8","line":126,"range":{"start_line":126,"start_character":36,"end_line":126,"end_character":40},"updated":"2022-05-11 20:43:20.000000000","message":"In light of [1], should we import __future__ and switch this to list (not List)?\n\n[1] https://review.opendev.org/c/openstack/cinder/+/839987\n\nThere are other instances in this patch, but I\u0027m just commenting on this one.","commit_id":"d3afa4db009b9c35176c3640d63ce6a0bca6f766"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"de634e961363578a7a1e598748f18d91a54775f0","unresolved":true,"context_lines":[{"line_number":123,"context_line":"                marker: Optional[str] \u003d None,"},{"line_number":124,"context_line":"                limit: Optional[int] \u003d None,"},{"line_number":125,"context_line":"                offset: Optional[int] \u003d None,"},{"line_number":126,"context_line":"                sort_keys: Optional[List[str]] \u003d None,"},{"line_number":127,"context_line":"                sort_dirs: Optional[List[str]] \u003d None) -\u003e \u0027objects.BackupList\u0027:"},{"line_number":128,"context_line":"        context.authorize(policy.GET_ALL_POLICY)"},{"line_number":129,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"fb441892_42b3ef61","line":126,"range":{"start_line":126,"start_character":36,"end_line":126,"end_character":40},"in_reply_to":"b00fc06a_69222cc8","updated":"2022-05-12 16:47:01.000000000","message":"I\u0027m cleaning them all up in\n    https://review.opendev.org/c/openstack/cinder/+/839987\n\nnow.","commit_id":"d3afa4db009b9c35176c3640d63ce6a0bca6f766"}]}