)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"8afe8a0ff522eec4b77e2182d329386604058116","unresolved":true,"context_lines":[{"line_number":20,"context_line":"tests will be added in glance once we release glance_store with this change."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Related-Bug: #2086675"},{"line_number":23,"context_line":"Change-Id: If6861f3271f50680af271090b516f2add6e392b9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"8584341b_ced43efe","line":23,"range":{"start_line":23,"start_character":11,"end_line":23,"end_character":52},"updated":"2024-11-12 15:24:55.000000000","message":"If you need another patchset, can you reindent the commit message?","commit_id":"93df723e846c247fa0e46f497ec40a1de3ee6d90"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"621f6b8ee879a47600a2d61b998247f86989f715","unresolved":false,"context_lines":[{"line_number":20,"context_line":"tests will be added in glance once we release glance_store with this change."},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"Related-Bug: #2086675"},{"line_number":23,"context_line":"Change-Id: If6861f3271f50680af271090b516f2add6e392b9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5dc5b9a5_71dc2750","line":23,"range":{"start_line":23,"start_character":11,"end_line":23,"end_character":52},"in_reply_to":"8584341b_ced43efe","updated":"2024-11-12 17:18:32.000000000","message":"Acknowledged","commit_id":"93df723e846c247fa0e46f497ec40a1de3ee6d90"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":19138,"name":"Pranali Deore","email":"pdeore@redhat.com","username":"PranaliD"},"change_message_id":"b12e9b659dc62fdb2111637d7e2979e2d88e4f18","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1c0484b2_7f3dd8c5","updated":"2024-11-11 07:05:43.000000000","message":"Looks Good! Needs a release note as well.","commit_id":"93df723e846c247fa0e46f497ec40a1de3ee6d90"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"8afe8a0ff522eec4b77e2182d329386604058116","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"35aa3482_a4fa304b","updated":"2024-11-12 15:24:55.000000000","message":"OK with the idea, some notes about the implementation and the commit message.","commit_id":"93df723e846c247fa0e46f497ec40a1de3ee6d90"},{"author":{"_account_id":33765,"name":"Mridula Joshi","email":"mrjoshi@redhat.com","username":"mrjoshi"},"change_message_id":"378fa55f5a28bffd5e81415fbeabce65f5bde616","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4037a2c5_542b5a58","updated":"2024-11-11 06:48:26.000000000","message":"Thank you!","commit_id":"93df723e846c247fa0e46f497ec40a1de3ee6d90"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"2bd8e9bbe603f6aa58ff644bfd3f8b95b80778f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b350a588_f7b29076","updated":"2024-11-08 07:41:17.000000000","message":"recheck error while resizing volume","commit_id":"93df723e846c247fa0e46f497ec40a1de3ee6d90"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"47e67f9811293399a66526a6e1bc9f7a0f9d8b29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"85cc123b_51d5d39f","updated":"2024-11-08 05:04:31.000000000","message":"recheck unrelated timeout","commit_id":"93df723e846c247fa0e46f497ec40a1de3ee6d90"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"488fad958cbd1cfda5e703737d504003b558b1e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ab45cea5_ac249d12","updated":"2024-11-14 14:06:26.000000000","message":"Thanks!","commit_id":"bb4587392a4791d349230b186d50e3045fe2c664"}],"glance_store/multi_backend.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"8afe8a0ff522eec4b77e2182d329386604058116","unresolved":true,"context_lines":[{"line_number":349,"context_line":"    if scheme not in location.SCHEME_TO_CLS_BACKEND_MAP:"},{"line_number":350,"context_line":"        raise exceptions.UnknownScheme(scheme\u003dscheme)"},{"line_number":351,"context_line":""},{"line_number":352,"context_line":"    scheme_info \u003d location.SCHEME_TO_CLS_BACKEND_MAP[scheme][store_identifier]"},{"line_number":353,"context_line":"    store \u003d scheme_info[\u0027store\u0027]"},{"line_number":354,"context_line":"    if store:"},{"line_number":355,"context_line":"        return store.weight"}],"source_content_type":"text/x-python","patch_set":1,"id":"dc4ae0b1_26e96a11","line":352,"range":{"start_line":352,"start_character":4,"end_line":352,"end_character":15},"updated":"2024-11-12 15:24:55.000000000","message":"It is kind of weird to check for the key at line 349 when above we used a try/except for something similar. Why not try/except here as well?","commit_id":"93df723e846c247fa0e46f497ec40a1de3ee6d90"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"621f6b8ee879a47600a2d61b998247f86989f715","unresolved":false,"context_lines":[{"line_number":349,"context_line":"    if scheme not in location.SCHEME_TO_CLS_BACKEND_MAP:"},{"line_number":350,"context_line":"        raise exceptions.UnknownScheme(scheme\u003dscheme)"},{"line_number":351,"context_line":""},{"line_number":352,"context_line":"    scheme_info \u003d location.SCHEME_TO_CLS_BACKEND_MAP[scheme][store_identifier]"},{"line_number":353,"context_line":"    store \u003d scheme_info[\u0027store\u0027]"},{"line_number":354,"context_line":"    if store:"},{"line_number":355,"context_line":"        return store.weight"}],"source_content_type":"text/x-python","patch_set":1,"id":"9f372f2b_b14db16b","line":352,"range":{"start_line":352,"start_character":4,"end_line":352,"end_character":15},"in_reply_to":"dc4ae0b1_26e96a11","updated":"2024-11-12 17:18:32.000000000","message":"Acknowledged","commit_id":"93df723e846c247fa0e46f497ec40a1de3ee6d90"},{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"919e359684bedbbef622194327edb678843d0652","unresolved":true,"context_lines":[{"line_number":347,"context_line":"        raise exceptions.UnknownScheme(msg)"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"    if scheme not in location.SCHEME_TO_CLS_BACKEND_MAP:"},{"line_number":350,"context_line":"        raise exceptions.UnknownScheme(scheme\u003dscheme)"},{"line_number":351,"context_line":""},{"line_number":352,"context_line":"    try:"},{"line_number":353,"context_line":"        backend_map \u003d location.SCHEME_TO_CLS_BACKEND_MAP[scheme]"}],"source_content_type":"text/x-python","patch_set":2,"id":"af86efe4_ea5c944d","line":350,"range":{"start_line":350,"start_character":0,"end_line":350,"end_character":8},"updated":"2024-11-12 21:40:26.000000000","message":"I think you can remove the above 2 lines now :)","commit_id":"65aff8cb32c4b337e54d0d4c8541816d886f04ad"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"818c363701215b6fabe12bf958abb78173b6aaba","unresolved":false,"context_lines":[{"line_number":347,"context_line":"        raise exceptions.UnknownScheme(msg)"},{"line_number":348,"context_line":""},{"line_number":349,"context_line":"    if scheme not in location.SCHEME_TO_CLS_BACKEND_MAP:"},{"line_number":350,"context_line":"        raise exceptions.UnknownScheme(scheme\u003dscheme)"},{"line_number":351,"context_line":""},{"line_number":352,"context_line":"    try:"},{"line_number":353,"context_line":"        backend_map \u003d location.SCHEME_TO_CLS_BACKEND_MAP[scheme]"}],"source_content_type":"text/x-python","patch_set":2,"id":"8d9b0729_883a4e1c","line":350,"range":{"start_line":350,"start_character":0,"end_line":350,"end_character":8},"in_reply_to":"af86efe4_ea5c944d","updated":"2024-11-13 05:39:24.000000000","message":"Acknowledged","commit_id":"65aff8cb32c4b337e54d0d4c8541816d886f04ad"}]}