)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"b81698918be7e102f7e8f46de630df538174ad42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"81721b02_7dd6f9d4","updated":"2026-04-08 13:35:46.000000000","message":"It looks like the openstack-tox-pep8 failure is due to pre-commit formatting.\n\nAfter fixing that, you could recheck to confirm the pipeline passes.","commit_id":"87b4c2583d282912a38831924eac5e3c26f74886"},{"author":{"_account_id":38982,"name":"Meer Hashaam Khan","display_name":"hashaaam","email":"hashaamkhan975@gmail.com","username":"hashaaam"},"change_message_id":"a16bbcd2935af6a3d92c61f63842e4a0a23fbe5e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ca18d808_cc05f049","in_reply_to":"81721b02_7dd6f9d4","updated":"2026-04-11 17:09:34.000000000","message":"I have made some changes to fix. please check now. and thanks for reviewing","commit_id":"87b4c2583d282912a38831924eac5e3c26f74886"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"72a08490fd2b6cfed0922d88a67ed3c1e13b5079","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ebd39f20_3e895801","updated":"2026-04-10 21:23:10.000000000","message":"The tests look great Hashaam. I noticed test_get_metadata and test_set_metadata only test with a plain string unlike test_delete_metadata and test_metadata_update_all which also cover object variants. Would it be worth adding those so that a share passed as an object is also accounted for? Otherwise it could silently break and no test would catch it.","commit_id":"cd765d287297bb2871b1edb5d7c72446a44b5406"},{"author":{"_account_id":38982,"name":"Meer Hashaam Khan","display_name":"hashaaam","email":"hashaamkhan975@gmail.com","username":"hashaaam"},"change_message_id":"a16bbcd2935af6a3d92c61f63842e4a0a23fbe5e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e0282226_e1aedbc0","in_reply_to":"ebd39f20_3e895801","updated":"2026-04-11 17:09:34.000000000","message":"Thank you Silvia! I have updated test_get_metadata and test_set_metadata \nin patchset 3 to also cover object variants (ShareUUID and ShareID), \nconsistent with the other tests.","commit_id":"cd765d287297bb2871b1edb5d7c72446a44b5406"},{"author":{"_account_id":38982,"name":"Meer Hashaam Khan","display_name":"hashaaam","email":"hashaamkhan975@gmail.com","username":"hashaaam"},"change_message_id":"a16bbcd2935af6a3d92c61f63842e4a0a23fbe5e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"63717aee_49d34c8b","updated":"2026-04-11 17:09:34.000000000","message":"recheck","commit_id":"7456ab7ed2308a191862099c4cc4cd3f7968a0c8"}]}