)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"6ace952df89791a25a433f074c74b8b347586e6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ac2f251c_96a88f34","updated":"2025-02-12 14:30:58.000000000","message":"In addition to the typo I noted, I don\u0027t understand the term \"Laurel\" in the subject line. As far as I can tell, the Epoxy support is identical to Caracal\u0027s.\n\nAlso, what about Dalmation? If the intention is to add a section for each release then you definitely need to include Dalmation. You could include it in this patch, but a separate patch would be required if you intend to propose a backport to stable/2024.2.","commit_id":"d71037d063cf322fd15dd3301b741abdc5963dbb"},{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"71d7d1fc4d9f028def935ce18b4e20c9ae1d26f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"774ad20a_8097d06b","updated":"2025-02-12 09:08:34.000000000","message":"LGTM","commit_id":"d71037d063cf322fd15dd3301b741abdc5963dbb"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"14e57e613033096a77934752219f15d928ac0b51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"233c327f_1f7d52b0","updated":"2025-02-14 16:25:50.000000000","message":"See the comment I added to [1].\n\n[1] https://review.opendev.org/c/openstack/cinder/+/941628","commit_id":"d9d9930078b887bbf9e35b72f27f27b464a0dc81"},{"author":{"_account_id":36132,"name":"Cuiye Liu","display_name":"Cherry","email":"cuiye.liu@dell.com","username":"cuiyeliu"},"change_message_id":"acf3757f9bb16687c00766adeedd6bf23a1f55cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"7ec9da6e_572ae0be","updated":"2025-02-26 08:39:49.000000000","message":"@rosmaita.fossdev@gmail.com @rajatdhasmana@gmail.com \ncan you review this patch? thanks in advance.","commit_id":"9c5351906c81746106b471a6ce02a98ad5918b96"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"aca34f77385df134c40ba6dabeb436a40903c47e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"16584a5d_99a9a110","updated":"2025-02-17 17:06:53.000000000","message":"I\u0027m still not sure what you mean by \"Laurel Release\" and it also might make sense to mention Dalmation, but I also don\u0027t want to be too picky. Assuming this patch merges on master, you can propose a backport to stable/2024.2.\n\nI assume you\u0027ll also want to propose another patch for Epoxy. At that point you will probably need to mention \"Epoxy\" (rather than \"Laurel\") somewhere in the commit message.","commit_id":"9c5351906c81746106b471a6ce02a98ad5918b96"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b1312175e851664bbbe412a868caee1524fee614","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"44c506d0_2faf419c","updated":"2025-03-06 13:24:19.000000000","message":"LGTM.","commit_id":"9c5351906c81746106b471a6ce02a98ad5918b96"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"b0911d2c5be2a2cbc8293bced0e4faba883cbc55","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3d39fafb_aecc0c3c","updated":"2025-02-18 19:52:26.000000000","message":"This seems okay to me.","commit_id":"9c5351906c81746106b471a6ce02a98ad5918b96"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"48c5b70607f1a15f18a7533f71c668d05bf5ffac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"5c94de41_0b98a37d","in_reply_to":"16584a5d_99a9a110","updated":"2025-02-18 19:51:02.000000000","message":"Laurel is the codename for PowerMax 10.1 release.","commit_id":"9c5351906c81746106b471a6ce02a98ad5918b96"},{"author":{"_account_id":36132,"name":"Cuiye Liu","display_name":"Cherry","email":"cuiye.liu@dell.com","username":"cuiyeliu"},"change_message_id":"d53d3fd59d599911c9fa666ae52c8128f4b69933","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f968b89e_ca18562d","in_reply_to":"16584a5d_99a9a110","updated":"2025-02-25 01:23:37.000000000","message":"Yes, I will create a new patch that includes epoxy after","commit_id":"9c5351906c81746106b471a6ce02a98ad5918b96"}],"doc/source/configuration/block-storage/drivers/dell-emc-powermax-driver.rst":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"6ace952df89791a25a433f074c74b8b347586e6d","unresolved":true,"context_lines":[{"line_number":59,"context_line":"   | OpenStack | Unisphere    | PowerMax OS | Supported Arrays               |"},{"line_number":60,"context_line":"   | release   | for PowerMax |             |                                |"},{"line_number":61,"context_line":"   +\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+"},{"line_number":62,"context_line":"   | EPoxy     | 10.1.0       | 10.1.0      | PowerMax 2500,8500             |"},{"line_number":63,"context_line":"   |           |              | (6079.225)  |                                |"},{"line_number":64,"context_line":"   |           |              +-------------+--------------------------------+"},{"line_number":65,"context_line":"   |           |              | 5978.711    | PowerMax 2000,8000             |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"8ffa5484_533f2276","line":62,"range":{"start_line":62,"start_character":5,"end_line":62,"end_character":10},"updated":"2025-02-12 14:30:58.000000000","message":"\"Epoxy\" (lower case \u0027p\u0027)","commit_id":"d71037d063cf322fd15dd3301b741abdc5963dbb"},{"author":{"_account_id":36132,"name":"Cuiye Liu","display_name":"Cherry","email":"cuiye.liu@dell.com","username":"cuiyeliu"},"change_message_id":"24b77ae695b5b69ced3fdea6c82ee4f5f4258076","unresolved":false,"context_lines":[{"line_number":59,"context_line":"   | OpenStack | Unisphere    | PowerMax OS | Supported Arrays               |"},{"line_number":60,"context_line":"   | release   | for PowerMax |             |                                |"},{"line_number":61,"context_line":"   +\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+"},{"line_number":62,"context_line":"   | EPoxy     | 10.1.0       | 10.1.0      | PowerMax 2500,8500             |"},{"line_number":63,"context_line":"   |           |              | (6079.225)  |                                |"},{"line_number":64,"context_line":"   |           |              +-------------+--------------------------------+"},{"line_number":65,"context_line":"   |           |              | 5978.711    | PowerMax 2000,8000             |"}],"source_content_type":"text/x-rst","patch_set":2,"id":"51a418ba_20154bba","line":62,"range":{"start_line":62,"start_character":5,"end_line":62,"end_character":10},"in_reply_to":"8ffa5484_533f2276","updated":"2025-02-13 06:58:21.000000000","message":"Correct. Thanks.","commit_id":"d71037d063cf322fd15dd3301b741abdc5963dbb"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"7c92433b22c96c21cf5aff703ed0309ef7411823","unresolved":true,"context_lines":[{"line_number":59,"context_line":"   | OpenStack | Unisphere    | PowerMax OS | Supported Arrays               |"},{"line_number":60,"context_line":"   | release   | for PowerMax |             |                                |"},{"line_number":61,"context_line":"   +\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+"},{"line_number":62,"context_line":"   | Epoxy     | 10.1.0       | 10.1.0      | PowerMax 2500,8500             |"},{"line_number":63,"context_line":"   |           |              | (6079.225)  |                                |"},{"line_number":64,"context_line":"   |           |              +-------------+--------------------------------+"},{"line_number":65,"context_line":"   |           |              | 5978.711    | PowerMax 2000,8000             |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"950392bb_55769b46","line":62,"updated":"2025-02-13 21:52:17.000000000","message":"Thank you for fixing the typo, but you do need to address the issue of Dalmation not appearing in the table.\n\nI think the best approach is to create a separate patch for Dalmation, and base this (Epoxy) patch on it. Then, after the Dalmation patch merges on master, you can propose it to stable/2024.2.","commit_id":"55d12d9588797f979265abd87a8a0a444c8feca1"},{"author":{"_account_id":36132,"name":"Cuiye Liu","display_name":"Cherry","email":"cuiye.liu@dell.com","username":"cuiyeliu"},"change_message_id":"ded5a079a435e6ed0c27b145c8e09a9fdcfc7d7b","unresolved":false,"context_lines":[{"line_number":59,"context_line":"   | OpenStack | Unisphere    | PowerMax OS | Supported Arrays               |"},{"line_number":60,"context_line":"   | release   | for PowerMax |             |                                |"},{"line_number":61,"context_line":"   +\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d+"},{"line_number":62,"context_line":"   | Epoxy     | 10.1.0       | 10.1.0      | PowerMax 2500,8500             |"},{"line_number":63,"context_line":"   |           |              | (6079.225)  |                                |"},{"line_number":64,"context_line":"   |           |              +-------------+--------------------------------+"},{"line_number":65,"context_line":"   |           |              | 5978.711    | PowerMax 2000,8000             |"}],"source_content_type":"text/x-rst","patch_set":3,"id":"459b3d41_4b8f88b2","line":62,"in_reply_to":"950392bb_55769b46","updated":"2025-02-14 09:43:09.000000000","message":"Thanks for your comment. I have created a new patch for Dalmation.https://review.opendev.org/c/openstack/cinder/+/941628","commit_id":"55d12d9588797f979265abd87a8a0a444c8feca1"}]}
