)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"7bfceba9456ad6a423f06d086221f06fb1767006","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"45180b78_6f209d5b","updated":"2022-11-09 09:46:47.000000000","message":"Thanks for the commit message ","commit_id":"9df653571d4da06c25222189be27e87a6da75628"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f20229f35eaef4d39d0ccb7608c0d5ca72aa3a54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f0abd58c_d7a6f037","updated":"2022-11-09 14:58:39.000000000","message":"Thanks!","commit_id":"9df653571d4da06c25222189be27e87a6da75628"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"a4dc0d83664ab55d1b120fc8a29fb3405b9bedfc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0ef9829b_3cd63661","updated":"2022-11-09 14:11:29.000000000","message":"This seems to fix my bug\nhttps://bugs.launchpad.net/python-cinderclient/+bug/1995883\n\nShould add closes-bug tag for this review","commit_id":"9df653571d4da06c25222189be27e87a6da75628"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"2dff392c849c0f87e79d56f466647a697de6c2a4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9de9a0f7_0674abfc","updated":"2022-11-09 15:00:31.000000000","message":"This should probably be backported to stable/xena, where change I7408d0e3a5ed7f4cbcaf65 introduced the problem.","commit_id":"9df653571d4da06c25222189be27e87a6da75628"}],"cinderclient/v3/shell.py":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"7bfceba9456ad6a423f06d086221f06fb1767006","unresolved":false,"context_lines":[{"line_number":2216,"context_line":"@utils.arg(\u0027--force\u0027,"},{"line_number":2217,"context_line":"           metavar\u003d\u0027\u003cTrue\u003e\u0027,"},{"line_number":2218,"context_line":"           nargs\u003d\u0027?\u0027,"},{"line_number":2219,"context_line":"           default\u003dNone,"},{"line_number":2220,"context_line":"           start_version\u003d\u00273.66\u0027,"},{"line_number":2221,"context_line":"           help\u003dargparse.SUPPRESS)"},{"line_number":2222,"context_line":"@utils.arg(\u0027--name\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"e261b59e_67798224","side":"PARENT","line":2219,"updated":"2022-11-09 09:46:47.000000000","message":"OK, so this is None because the \u0027force\u0027 parameter, which is technically not allowed\nin mv 3.66.","commit_id":"6f67187b8255ae231f82a9deaaf9156c868153a0"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f20229f35eaef4d39d0ccb7608c0d5ca72aa3a54","unresolved":false,"context_lines":[{"line_number":2216,"context_line":"@utils.arg(\u0027--force\u0027,"},{"line_number":2217,"context_line":"           metavar\u003d\u0027\u003cTrue\u003e\u0027,"},{"line_number":2218,"context_line":"           nargs\u003d\u0027?\u0027,"},{"line_number":2219,"context_line":"           default\u003dNone,"},{"line_number":2220,"context_line":"           start_version\u003d\u00273.66\u0027,"},{"line_number":2221,"context_line":"           help\u003dargparse.SUPPRESS)"},{"line_number":2222,"context_line":"@utils.arg(\u0027--name\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"c2f0bff1_8f3be4c3","side":"PARENT","line":2219,"in_reply_to":"ab5fb6ff_2caba98d","updated":"2022-11-09 14:58:39.000000000","message":"Done","commit_id":"6f67187b8255ae231f82a9deaaf9156c868153a0"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"a4b207179f2e1d560e8dc330dfbad8d276a02960","unresolved":true,"context_lines":[{"line_number":2216,"context_line":"@utils.arg(\u0027--force\u0027,"},{"line_number":2217,"context_line":"           metavar\u003d\u0027\u003cTrue\u003e\u0027,"},{"line_number":2218,"context_line":"           nargs\u003d\u0027?\u0027,"},{"line_number":2219,"context_line":"           default\u003dNone,"},{"line_number":2220,"context_line":"           start_version\u003d\u00273.66\u0027,"},{"line_number":2221,"context_line":"           help\u003dargparse.SUPPRESS)"},{"line_number":2222,"context_line":"@utils.arg(\u0027--name\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ed1e23d8_8845707a","side":"PARENT","line":2219,"in_reply_to":"e261b59e_67798224","updated":"2022-11-09 13:20:26.000000000","message":"I\u0027m really not sure if this is necessary or not.  I removed lines 2217-2222 and all tests passed, but since they also pass with this code present, I figured that removing it should be part of a different change.\n\nI don\u0027t remember why I added it in change I7408d0e3a5ed7f4cb ... it may have been left over from an attempt to handle the new microversion in the original function instead of adding a new one, and when I realized that I had to write a new do_shapshot_create() for 3.66, this may have been left over (and I didn\u0027t notice because it doesn\u0027t affect any tests).  I\u0027m fairly certain that this hypothesis is correct (though not certain enough to remove these lines as part of this change).\n\nIt doesn\u0027t affect the display of help text (which it shouldn\u0027t, just want to leave a note that I tried it just in case).  Help text for --os-volume-api-version 3.65 and --os-volume-api-version 3.66 both display correctly with lines 2217-2222 removed.","commit_id":"6f67187b8255ae231f82a9deaaf9156c868153a0"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3e8f8a08c874a653c6c8e580bd6e6f59c29083e5","unresolved":true,"context_lines":[{"line_number":2216,"context_line":"@utils.arg(\u0027--force\u0027,"},{"line_number":2217,"context_line":"           metavar\u003d\u0027\u003cTrue\u003e\u0027,"},{"line_number":2218,"context_line":"           nargs\u003d\u0027?\u0027,"},{"line_number":2219,"context_line":"           default\u003dNone,"},{"line_number":2220,"context_line":"           start_version\u003d\u00273.66\u0027,"},{"line_number":2221,"context_line":"           help\u003dargparse.SUPPRESS)"},{"line_number":2222,"context_line":"@utils.arg(\u0027--name\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"ab5fb6ff_2caba98d","side":"PARENT","line":2219,"in_reply_to":"ed1e23d8_8845707a","updated":"2022-11-09 13:21:44.000000000","message":"... and when I say lines 2217-2222, I mean on PS 3.","commit_id":"6f67187b8255ae231f82a9deaaf9156c868153a0"}]}