)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"e3085f4458f82038e6a1537806f75b7662abf69b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"40c0fc4a_c43acd36","updated":"2023-09-14 13:39:58.000000000","message":"The patch itself is fine, but the commit message needs work. It should focus on describing the technical reasons behind it, which in this case is it fixes a bug [1] in releases prior to zed. The background info (discussions with people, references to downstream releases like RHOSP) are not necessary.\n\n[1] https://bugs.launchpad.net/cinder/+bug/2033398\n\nTo reference the LP bug, include this in the commit message just above the Change-Id:\n\nCloses-Bug: #2033398","commit_id":"7b4e279e4885997e1d06fe01d479a0649e2e5e62"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"bf4772d84912c60ff2d76c6b053f1cddd680fab6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"42e9248f_36562eac","updated":"2023-09-14 09:51:47.000000000","message":"recheck","commit_id":"7b4e279e4885997e1d06fe01d479a0649e2e5e62"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"6e2f77548aaf1c69549a03a886d7f6e689711893","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"44501d75_9ade6d2f","in_reply_to":"40c0fc4a_c43acd36","updated":"2023-09-14 13:54:01.000000000","message":"Done, thank you Alan.","commit_id":"7b4e279e4885997e1d06fe01d479a0649e2e5e62"},{"author":{"_account_id":31016,"name":"Ivan Pchelintsev","email":"i.pchelintsev@yadro.com","username":"pcheli"},"change_message_id":"3ff3e858fa880bbfd803bcfc0ed3fc3857115251","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7acc3906_be5c1d84","updated":"2023-09-18 07:04:35.000000000","message":"Add release notes please.","commit_id":"edab48bc5c0dd6d6267974f10790102ab916fa29"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0b07d34b2f7bad75baefd9a72f957b0a6ac0edec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"681af8d6_c6893eb4","updated":"2023-10-03 15:59:12.000000000","message":"This change is in zed; it was part of I0c0f3d2246b84032657 namely this part:\nhttps://review.opendev.org/c/openstack/cinder/+/796286/24/cinder/volume/drivers/dell_emc/powermax/rest.py#1575","commit_id":"edab48bc5c0dd6d6267974f10790102ab916fa29"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"170720fc9b93bb56d502edd7e77a5ec64042b07e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e8b27372_7c3dee07","updated":"2023-10-03 16:03:12.000000000","message":"This is a minor change, isolated to a single driver, that addresses an operator pain point.  The change is already in zed+, so I think this is OK to backport.  A release note would be nice.","commit_id":"edab48bc5c0dd6d6267974f10790102ab916fa29"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"b359f6eb3ae46cb21c4c893821e3504c75a4346f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7eeb6119_6642bdeb","in_reply_to":"e8b27372_7c3dee07","updated":"2023-10-05 08:35:05.000000000","message":"Thks Brian. \nRelease note added.","commit_id":"edab48bc5c0dd6d6267974f10790102ab916fa29"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3465106b5000fc4e7773876510cb24194cf41d00","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"701916a5_6b3a7743","updated":"2023-10-10 12:43:13.000000000","message":"As I mentioned in an earlier comment, this change is in zed+ and meets backport criteria.","commit_id":"d02c88346dbb586f82de91804e4d1bda34308608"},{"author":{"_account_id":32074,"name":"Harsh Ailani","email":"haailani@in.ibm.com","username":"haailani"},"change_message_id":"a056e1cfcc928402c08a886eed0dcd447a0f4865","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"193a990c_608b8b27","updated":"2023-10-10 09:09:34.000000000","message":"LGTM","commit_id":"d02c88346dbb586f82de91804e4d1bda34308608"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"7cd707a1dd3e2a09baa75220296a4aa350283eff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"47ebc35a_ef7b0e1c","updated":"2023-10-10 09:24:55.000000000","message":"Thanks Eric, can we have another +2?","commit_id":"d02c88346dbb586f82de91804e4d1bda34308608"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"c7120b2500ec19dca66e6474d5d7e94c0b2926c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"07458e2b_1dbbec46","in_reply_to":"701916a5_6b3a7743","updated":"2023-10-10 13:04:02.000000000","message":"Thks Brian!","commit_id":"d02c88346dbb586f82de91804e4d1bda34308608"}]}
