)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"77ba45fb12944811cc057c342c8e4d6379c4be4b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ecadcc7a_25cd0ee5","updated":"2026-08-07 19:04:10.000000000","message":"LGTM, thank you Denver","commit_id":"9d2aff8aeb1d31e63e0e6637e3dfba0d40fb34b5"}],"manila/tests/share/test_share_types.py":[{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"4f7e4927f08a0297e8ec3b9322972b61ee8e9db1","unresolved":true,"context_lines":[{"line_number":678,"context_line":"            constants.ExtraSpecs.SNAPSHOT_SUPPORT: \u0027true\u0027,"},{"line_number":679,"context_line":"            constants.ExtraSpecs.CREATE_SHARE_FROM_SNAPSHOT_SUPPORT: \u0027false\u0027,"},{"line_number":680,"context_line":"        }"},{"line_number":681,"context_line":"        share_type \u003d share_types.create(self.context, \u0027type1\u0027, extra_specs)"},{"line_number":682,"context_line":"        share_type_id \u003d share_type.get(\u0027id\u0027)"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"        share_types.destroy(self.context, share_type_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"b7a18f7f_e29e4e24","line":681,"range":{"start_line":681,"start_character":8,"end_line":681,"end_character":75},"updated":"2026-07-15 14:38:53.000000000","message":"same issue as I commented in another change: this test should be unit testing something, we should not be calling other functions rather than the one we are attempting to test.  Please mock this","commit_id":"ae9d3e04e0f078dffb19d7c509ef1f4b9415df0e"},{"author":{"_account_id":38471,"name":"Denver Baraka","display_name":"Denver Baraka","email":"denverbaraka@gmail.com","username":"denver-baraka"},"change_message_id":"ef144296313bc335ae47f5ae2d315a3e5a8944ab","unresolved":false,"context_lines":[{"line_number":678,"context_line":"            constants.ExtraSpecs.SNAPSHOT_SUPPORT: \u0027true\u0027,"},{"line_number":679,"context_line":"            constants.ExtraSpecs.CREATE_SHARE_FROM_SNAPSHOT_SUPPORT: \u0027false\u0027,"},{"line_number":680,"context_line":"        }"},{"line_number":681,"context_line":"        share_type \u003d share_types.create(self.context, \u0027type1\u0027, extra_specs)"},{"line_number":682,"context_line":"        share_type_id \u003d share_type.get(\u0027id\u0027)"},{"line_number":683,"context_line":""},{"line_number":684,"context_line":"        share_types.destroy(self.context, share_type_id)"}],"source_content_type":"text/x-python","patch_set":1,"id":"2c7caa6d_50e94097","line":681,"range":{"start_line":681,"start_character":8,"end_line":681,"end_character":75},"in_reply_to":"b7a18f7f_e29e4e24","updated":"2026-07-27 17:02:52.000000000","message":"fixed, thanks","commit_id":"ae9d3e04e0f078dffb19d7c509ef1f4b9415df0e"}]}