)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5873f96f27f00021d909b21fdbbc1388efccfdf0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f9d6fbe7_b67d4a11","updated":"2022-05-18 12:33:01.000000000","message":"My only hesitation is that changing db/sqlalchemy/api.py my throw a stack of patches in Stephen\u0027s remove-legacyfacade effort into merge conflict.  Otherwise, this LGTM and handles the question of what the proper type for the \u0027inactive\u0027 parameter should be by removing it.","commit_id":"bb898f39ef789984c041eacf3154cdaa9a496122"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"498ce7ff88af41ad6d6797fcad778c0a7fc20f34","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1d9054cf_29f8d2d8","updated":"2022-05-17 16:03:41.000000000","message":"recheck","commit_id":"bb898f39ef789984c041eacf3154cdaa9a496122"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"543cafc42caebb5669de7b3b1216a147dfd39cd7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ef15e2e3_b3b82a86","in_reply_to":"f9d6fbe7_b67d4a11","updated":"2022-05-19 18:36:02.000000000","message":"Yes, it might, I mostly wanted to get this up here so I didn\u0027t forget about it, I can rebase it on top of his series, or we can wait.","commit_id":"bb898f39ef789984c041eacf3154cdaa9a496122"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e89e2662ee7d4d538c670fd561f630445ad36006","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ba8fdcb9_f7eb7806","updated":"2023-09-02 13:37:34.000000000","message":"Rebasing just because it\u0027s been sitting so long and to get fresh CI logs.  LGTM otherwise.","commit_id":"1c7c31f58dbd0bd888419029605ec078d7fcd36b"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"ffaef44434cafc0e5f7145e5768d100032bc5b09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f9e41a22_f5d24da3","updated":"2022-07-26 15:14:26.000000000","message":"recheck","commit_id":"1c7c31f58dbd0bd888419029605ec078d7fcd36b"}],"cinder/db/sqlalchemy/api.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"7a527f9450ded797ef51ed76e31e17e9b3745d3c","unresolved":true,"context_lines":[{"line_number":4098,"context_line":"        return result"},{"line_number":4099,"context_line":""},{"line_number":4100,"context_line":""},{"line_number":4101,"context_line":"# TODO: remove \"inactive\" arg here"},{"line_number":4102,"context_line":"@require_context"},{"line_number":4103,"context_line":"def group_type_get_all(context, inactive\u003dFalse, filters\u003dNone, marker\u003dNone,"},{"line_number":4104,"context_line":"                       limit\u003dNone, sort_keys\u003dNone, sort_dirs\u003dNone,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9cc911e4_5e14300d","line":4101,"range":{"start_line":4101,"start_character":2,"end_line":4101,"end_character":34},"updated":"2022-05-10 15:56:24.000000000","message":"Need to go do this.","commit_id":"7c070e5be3cc39a317028bf510290bcbd10685f1"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"2f40221f956967e0903c68dac0165024f3f964f5","unresolved":false,"context_lines":[{"line_number":4098,"context_line":"        return result"},{"line_number":4099,"context_line":""},{"line_number":4100,"context_line":""},{"line_number":4101,"context_line":"# TODO: remove \"inactive\" arg here"},{"line_number":4102,"context_line":"@require_context"},{"line_number":4103,"context_line":"def group_type_get_all(context, inactive\u003dFalse, filters\u003dNone, marker\u003dNone,"},{"line_number":4104,"context_line":"                       limit\u003dNone, sort_keys\u003dNone, sort_dirs\u003dNone,"}],"source_content_type":"text/x-python","patch_set":1,"id":"65fece5e_e2e9b9d3","line":4101,"range":{"start_line":4101,"start_character":2,"end_line":4101,"end_character":34},"in_reply_to":"9cc911e4_5e14300d","updated":"2022-05-17 16:04:08.000000000","message":"Done","commit_id":"7c070e5be3cc39a317028bf510290bcbd10685f1"}]}