)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"2bd9fbc2e95c53db66e5cd10c842928fea0ab70a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"17c65316_3828bcb2","updated":"2025-07-07 17:49:19.000000000","message":"Looking for CI results from PowerFlex but I\u0027m not seeing it, can you point me in the right direction?","commit_id":"82ca6aa95e60bdcab9af6db1edf71f2b3fd70333"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"d98403a6a8a039cc841ff5ce4cde73d8ee6db32b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"af9b6d61_3b0b9abe","updated":"2025-06-19 12:07:59.000000000","message":"recheck grenade-skip-level-always TIMED_OUT","commit_id":"82ca6aa95e60bdcab9af6db1edf71f2b3fd70333"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"fc0c0fb73b0d8307828a72a3b17ea9a387f92e35","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2dfcd3e7_7c4fad19","updated":"2025-06-19 07:22:26.000000000","message":"recheck openstacksdk-functional-devstack OSError: [Errno 101] Network is unreachable","commit_id":"82ca6aa95e60bdcab9af6db1edf71f2b3fd70333"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"94717be18a3a5acf31a621dcff960a8d8acb8c8a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a495dd79_d78fc4cf","updated":"2025-07-08 04:33:01.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"82ca6aa95e60bdcab9af6db1edf71f2b3fd70333"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"68cf06cf259aa04fadb5296f2c59cdc1a3f0d33e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"fb826723_87148a31","in_reply_to":"17c65316_3828bcb2","updated":"2025-07-11 02:40:25.000000000","message":"Hi @jobernar@redhat.com,\n\nCI jobs `powerflex-v4-cinder-tempest` and `powerflex-v3-cinder-tempest` passed.\nThe job `powerflex-nvme-tcp-cinder-tempest` fails as nvme-tcp support is not merged.\n\nPlease kindly review. Thanks!","commit_id":"82ca6aa95e60bdcab9af6db1edf71f2b3fd70333"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"a48437f136e14d1b147ad3a18d2138edc43ae22d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"32304101_0b3491db","updated":"2025-07-08 05:06:23.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"58105374a4f2c49b5a57b7a5c5605749041df9f8"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"74733f2a8fa767a51724384822b39ae6e8bd195b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8b5f51f6_9243d10b","updated":"2025-07-08 05:25:52.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"58105374a4f2c49b5a57b7a5c5605749041df9f8"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"6b6e18261410980669b8238b6d28846cf480b24e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2021d578_0d88cb84","updated":"2025-07-09 12:45:51.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"31a62514f624ec2025e15f8f46cb990366e0ddab"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"06bb3fdaa8f821299845f9b63a2909046f3b08d1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"6f945aff_c8c99073","updated":"2025-07-10 05:11:40.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"31a62514f624ec2025e15f8f46cb990366e0ddab"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"e9fa34f4f0a39abf76dc6769cb003e5d3d20d6a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a81242ca_03081b3c","updated":"2025-07-09 14:38:16.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"31a62514f624ec2025e15f8f46cb990366e0ddab"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"5bcad3a1043853e191d74d19091ffb9c9949205f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ad6c62a8_5a7cdf3c","updated":"2025-07-10 07:57:15.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"31a62514f624ec2025e15f8f46cb990366e0ddab"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"22eabcf3c38ab9c57d7437bdd6d3e85dd4632007","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"a1798ece_cec61e68","updated":"2025-07-16 18:15:58.000000000","message":"Failing PowerFlex 3rd party CI","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f59a029956543bd7b262c98414c2b175fbece88a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"16b88c75_5e199232","updated":"2025-08-04 10:44:44.000000000","message":"Thanks for proposing this change. This makes the powerflex driver behave similar to other drivers for attach/detach operations. Few comments inline.","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"cb276ee71789e9864fb58cda0ab033fc070148c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"b1094860_c3dadfca","updated":"2025-07-10 09:19:46.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"29814e98b5570c4ec0bd9201406239be9c435f5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"20b7bc59_db5a5312","in_reply_to":"a1798ece_cec61e68","updated":"2025-07-17 02:18:53.000000000","message":"PowerFlex v3 and v4 SDC connection CI jobs passed.\n`powerflex-nvme-tcp-cinder` failed as expected: the nvme-tcp patch is not merged.","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"4a4c89f6d05a01cd38bdb1406479f463016659ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"63e634ec_99270994","updated":"2025-08-11 20:07:01.000000000","message":"Apart from the lack of backward compatibility, overall it looks good to me and Powerflex CI is passing (though inaccessible).","commit_id":"817868fc9511986a7719d46c523d66e8decef338"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1656245c55b0e0d4f8140d3da807ada17aea4458","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"4642b016_a0109d37","updated":"2025-08-10 07:44:54.000000000","message":"Thanks for reviewing! Comments are resolved.","commit_id":"817868fc9511986a7719d46c523d66e8decef338"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"740da8b766b26afecbb0ef586973b653d86101ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c14f089d_a87323d5","updated":"2025-08-08 15:47:52.000000000","message":"recheck","commit_id":"817868fc9511986a7719d46c523d66e8decef338"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"82317684740c735d00e780779c762cf714d258d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"14c8275f_08368897","updated":"2025-08-09 03:36:10.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"817868fc9511986a7719d46c523d66e8decef338"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"42759e4048b7bfa6fc4810e2b4b2cb899cfdd1d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"d97afaf4_0e13d520","updated":"2025-08-18 15:46:22.000000000","message":"Mostly looks good.  See comments inline about the release note and some of the unit tests.","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"da4f7a7034b0516de0d721dca0c3cc1a30360d90","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5e8bcf4d_9af45df4","updated":"2025-08-12 16:44:43.000000000","message":"Thanks for reviewing! Comments are resolved.\nI have escalated the access issue of the log server. Hope it will be back online soon.","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"a1a862f6d1655efcdc56ac8ff312c2c059879d81","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"de32417c_e2d98d38","updated":"2025-08-13 03:47:55.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3c810f9ae7243126cdc4c4ab1b7cfe364da83bbc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"9997fb4e_82e4f097","updated":"2025-08-25 13:15:01.000000000","message":"Downgrading to +1, I want to look again at the upgrade issue that Rajat raised on an earlier patch set.","commit_id":"bfb51064cab225352b00df38c196cce84f15c704"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"43735b075fadab86be269a85b817329cebd2bf4c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"14b015c2_22c82dad","updated":"2025-08-21 12:12:47.000000000","message":"Just a note that os-brick 6.13.0 will contain the change in the Depends-On for this patch, and it should be released soon: https://review.opendev.org/c/openstack/releases/+/957706","commit_id":"bfb51064cab225352b00df38c196cce84f15c704"},{"author":{"_account_id":38067,"name":"Sandip Rajbanshi","display_name":"Sandip27","email":"sandip.rajbanshi@ibm.com","username":"sandip-rajbanshi-27"},"change_message_id":"253d737cac59422a2c9bdff5ebc0520ed9c5a4cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"64710e13_3a0a3ad4","updated":"2025-08-25 10:19:02.000000000","message":"LGTM","commit_id":"bfb51064cab225352b00df38c196cce84f15c704"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"47d1914e03ca11c63910e886e50760da5091d622","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"b4a8a9c6_17fdd25d","updated":"2025-08-21 12:07:14.000000000","message":"PowerFlex CI has succeeded, so upgrading to +2.","commit_id":"bfb51064cab225352b00df38c196cce84f15c704"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"d5790ee6b5a38d8294757c6b3723d2cef99adaf1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"008cb980_4f630f98","updated":"2025-08-19 13:26:44.000000000","message":"PowerFlex SDC v3 and v4 CI jobs passed. NVMe-TCP job failed as expected.","commit_id":"bfb51064cab225352b00df38c196cce84f15c704"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"eeff6fb080feeb589829f89dab6125aae50907b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8ba78b77_37bd7100","updated":"2025-08-19 12:34:06.000000000","message":"Revisions LGTM. Only a +1 while waiting for PowerFlex CI to approve this patch, otherwise I\u0027m +2 on this.","commit_id":"bfb51064cab225352b00df38c196cce84f15c704"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"35cf51ee687279e9d44294de48197ed4b87e8502","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"67bcbc34_4c74688d","updated":"2025-08-26 17:14:55.000000000","message":"We need to be really clear on what the upgrade path is.  This is my understanding, but I\u0027m not sure it\u0027s accurate:\n\n(1)  Upgrade os-brick first to 6.13.0, but do NOT remove the configuration file\n(os-brick 6.13.0 can do the mapping itself if the driver hasn\u0027t done it yet, but only if the file is still there\n(2)  Then upgrade the PowerFlex driver to \u003e\u003d 3.6.0.  The 3.6.0 driver requires os-brick \u003e\u003d 6.13.0 to function correctly; it will not work with an earlier brick\n\nIs that correct?  Please add something like that to the release note.","commit_id":"bfb51064cab225352b00df38c196cce84f15c704"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"5a9713b5c2bbdbc9001959f6fc8872167eb28115","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"e2fc11f4_877c013d","updated":"2025-08-19 11:44:07.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"bfb51064cab225352b00df38c196cce84f15c704"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"5bb4c69c51d8f642a41b8fc99a249e0b714ef72a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"289706e1_5d9d6c99","in_reply_to":"67bcbc34_4c74688d","updated":"2025-09-01 01:21:12.000000000","message":"Added upgrade steps.","commit_id":"bfb51064cab225352b00df38c196cce84f15c704"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"9a340b18d5691c62cd66a10c94e5a8a47bdd009a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"a6b1efb7_799c3157","updated":"2025-09-01 01:23:18.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"1416e9e27f0707708bb4b3dc6fad5e6a60c0c9fc"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"73ba5bbcb264dba0485a47cc78a51608c0a65368","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"6dbe9398_5286f24c","updated":"2025-09-01 08:11:24.000000000","message":"Code and tests LGTM. The upgrade note is added which specifies that the cinder code is not compatible with older os-brick and mentions a clear upgrade path.\nPowerflex v3 and v4 CI are passing. The NVMe-TCP driver isn\u0027t merged yet to the nvme-tcp job failure is a non-issue. LGTM.","commit_id":"83d8a87ac8a89e6a8d46c1632051f8d5d4f841ae"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"fdc1b7df5bcf1cba9acb5bd5eab3df7262eb5a5f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"492a68ba_c790a94e","updated":"2025-09-01 04:39:48.000000000","message":"PowerFlex SDC v3 and v4 CI jobs passed. NVMe-TCP job failed as expected.","commit_id":"83d8a87ac8a89e6a8d46c1632051f8d5d4f841ae"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"a742bfd595c98a639f9342f1702b4c1ee23074df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"bcb68803_11073263","updated":"2025-09-01 02:06:08.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"83d8a87ac8a89e6a8d46c1632051f8d5d4f841ae"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0d6ab93c4b12ee7062b380d5d4024902b6fd534c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"2cd75c39_6ea6bb2b","updated":"2025-09-02 13:03:13.000000000","message":"Minor issue with the release note; otherwise looks ok.","commit_id":"14fba988949f08e5ebc4cd03cbd8a2843625ea0b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"82f0f730248e26b10cacb5ae42d2a27f7eea5aae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"cad7bc9c_14638e01","updated":"2025-09-01 10:52:22.000000000","message":"Thanks!","commit_id":"14fba988949f08e5ebc4cd03cbd8a2843625ea0b"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"30d4825d6ceff2addf83cdc144d79a4358ff23d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"d054952f_6c494cb1","updated":"2025-09-01 08:40:58.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"14fba988949f08e5ebc4cd03cbd8a2843625ea0b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"9cde1bfc5c87ae0e6bedf6f75f7cab4de018d904","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"a0114f49_6e103514","updated":"2025-09-02 15:23:01.000000000","message":"Revision LGTM, hopefully Zuul and Dell PowerFlex CI will agree.","commit_id":"e3d26011095a8ba5b2e8fa5062d60782e70aa213"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"f18d782ac93842af762bd4b1aad066c036b30344","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"10f8ac95_be860da0","updated":"2025-09-03 07:25:13.000000000","message":"Thank you all for making it happen!","commit_id":"e3d26011095a8ba5b2e8fa5062d60782e70aa213"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"df29dea488bd9ee0a411b9a8271dd0233a8a2ea2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"12420103_3fbf1ad8","updated":"2025-09-02 17:19:58.000000000","message":"Zuul and third-party CI are green.  Carrying over Rajat\u0027s +2 from PS 12 given that the only change between PS 12 and 13 was a release note change.","commit_id":"e3d26011095a8ba5b2e8fa5062d60782e70aa213"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"78560efb0cb04128c51c11bc0189efc4c1fb6acc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"32cfd8f9_6778756c","updated":"2025-09-03 03:26:13.000000000","message":"recheck \ncinder-tempest-plugin-lvm-lio-barbican \"Server failed to delete and is in ERROR status\"\ndevstack-plugin-nfs-tempest-full  \"POST_FAILURE\"","commit_id":"e3d26011095a8ba5b2e8fa5062d60782e70aa213"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"7ab8120a3cb06b6b6781de9533b3e1000ebd0d05","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"f2803281_fcf860b0","updated":"2025-09-02 20:22:33.000000000","message":"recheck - openstack-tox-pep8 failed in post-failure.","commit_id":"e3d26011095a8ba5b2e8fa5062d60782e70aa213"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"b1359fac0e39162607fe96cc59937e9c92368b69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"f35d5acc_542efb47","updated":"2025-09-03 00:46:55.000000000","message":"recheck grenade-skip-level-always","commit_id":"e3d26011095a8ba5b2e8fa5062d60782e70aa213"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"dbd1f866c4dd2f2264b6801305ca957597e52604","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"46051350_0ab4bda5","updated":"2025-09-02 13:55:06.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"e3d26011095a8ba5b2e8fa5062d60782e70aa213"}],"cinder/tests/unit/volume/drivers/dell_emc/powerflex/mockup/query_sdc_by_id_response.json":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f59a029956543bd7b262c98414c2b175fbece88a","unresolved":true,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"hostOsFullType\": null,"},{"line_number":3,"context_line":"    \"systemId\": \"2c4a220db6e0520f\","},{"line_number":4,"context_line":"    \"name\": null,"}],"source_content_type":"application/json","patch_set":6,"id":"423e317a_875d23c2","line":1,"updated":"2025-08-04 10:44:44.000000000","message":"why not use a python dictionary rather than serializing/de-serializing the json files for UTs?","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1656245c55b0e0d4f8140d3da807ada17aea4458","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"hostOsFullType\": null,"},{"line_number":3,"context_line":"    \"systemId\": \"2c4a220db6e0520f\","},{"line_number":4,"context_line":"    \"name\": null,"}],"source_content_type":"application/json","patch_set":6,"id":"ef8e921d_a716e6b1","line":1,"in_reply_to":"423e317a_875d23c2","updated":"2025-08-10 07:44:54.000000000","message":"The json files are precise as collected by rest calls to PowerFlex, and can be consumed directly as rest call responses.","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"}],"cinder/tests/unit/volume/drivers/dell_emc/powerflex/test_attach_detach_volume.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"887a3b45bf20689f3ad5581ed2f61a29a9fe471b","unresolved":true,"context_lines":[{"line_number":29,"context_line":"            ctx, **{\u0027provider_id\u0027: fake.PROVIDER_ID})"},{"line_number":30,"context_line":"        self.driver.connector \u003d FakeConnector()"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    def test_attach_volume(self):"},{"line_number":33,"context_line":"        pass"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def test_detach_volume(self):"},{"line_number":36,"context_line":"        pass"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"class FakeConnector(object):"},{"line_number":40,"context_line":"    def connect_volume(self, connection_properties):"}],"source_content_type":"text/x-python","patch_set":7,"id":"12d1779e_8ecfe8f7","line":37,"range":{"start_line":32,"start_character":0,"end_line":37,"end_character":0},"updated":"2025-08-11 20:09:33.000000000","message":"nit: we should remove these tests as they are no-op now","commit_id":"817868fc9511986a7719d46c523d66e8decef338"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"da4f7a7034b0516de0d721dca0c3cc1a30360d90","unresolved":false,"context_lines":[{"line_number":29,"context_line":"            ctx, **{\u0027provider_id\u0027: fake.PROVIDER_ID})"},{"line_number":30,"context_line":"        self.driver.connector \u003d FakeConnector()"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    def test_attach_volume(self):"},{"line_number":33,"context_line":"        pass"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"    def test_detach_volume(self):"},{"line_number":36,"context_line":"        pass"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"class FakeConnector(object):"},{"line_number":40,"context_line":"    def connect_volume(self, connection_properties):"}],"source_content_type":"text/x-python","patch_set":7,"id":"55245533_49b6ed77","line":37,"range":{"start_line":32,"start_character":0,"end_line":37,"end_character":0},"in_reply_to":"12d1779e_8ecfe8f7","updated":"2025-08-12 16:44:43.000000000","message":"Done","commit_id":"817868fc9511986a7719d46c523d66e8decef338"}],"cinder/tests/unit/volume/drivers/dell_emc/powerflex/test_initialize_connection.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"887a3b45bf20689f3ad5581ed2f61a29a9fe471b","unresolved":true,"context_lines":[{"line_number":104,"context_line":"        self.driver._get_volumetype_qos.return_value \u003d qos"},{"line_number":105,"context_line":"        self.driver._get_volumetype_extraspecs \u003d mock.MagicMock()"},{"line_number":106,"context_line":"        self.driver._get_volumetype_extraspecs.return_value \u003d extraspecs"},{"line_number":107,"context_line":"        self.driver._attach_volume_to_host \u003d mock.MagicMock("},{"line_number":108,"context_line":"            return_value\u003dNone"},{"line_number":109,"context_line":"        )"},{"line_number":110,"context_line":"        self.driver._check_volume_mapped \u003d mock.MagicMock("},{"line_number":111,"context_line":"            return_value\u003dNone"},{"line_number":112,"context_line":"        )"},{"line_number":113,"context_line":"        self.driver.primary_client.set_sdc_limits \u003d mock.MagicMock()"},{"line_number":114,"context_line":"        return self.driver.initialize_connection(self.volume, self.connector)"}],"source_content_type":"text/x-python","patch_set":7,"id":"dc71d0e7_65f12a03","line":113,"range":{"start_line":107,"start_character":8,"end_line":113,"end_character":68},"updated":"2025-08-11 20:09:33.000000000","message":"we are not asserting anything here that these specified methods were called and with which parameters","commit_id":"817868fc9511986a7719d46c523d66e8decef338"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"da4f7a7034b0516de0d721dca0c3cc1a30360d90","unresolved":false,"context_lines":[{"line_number":104,"context_line":"        self.driver._get_volumetype_qos.return_value \u003d qos"},{"line_number":105,"context_line":"        self.driver._get_volumetype_extraspecs \u003d mock.MagicMock()"},{"line_number":106,"context_line":"        self.driver._get_volumetype_extraspecs.return_value \u003d extraspecs"},{"line_number":107,"context_line":"        self.driver._attach_volume_to_host \u003d mock.MagicMock("},{"line_number":108,"context_line":"            return_value\u003dNone"},{"line_number":109,"context_line":"        )"},{"line_number":110,"context_line":"        self.driver._check_volume_mapped \u003d mock.MagicMock("},{"line_number":111,"context_line":"            return_value\u003dNone"},{"line_number":112,"context_line":"        )"},{"line_number":113,"context_line":"        self.driver.primary_client.set_sdc_limits \u003d mock.MagicMock()"},{"line_number":114,"context_line":"        return self.driver.initialize_connection(self.volume, self.connector)"}],"source_content_type":"text/x-python","patch_set":7,"id":"4f55f850_c2f54470","line":113,"range":{"start_line":107,"start_character":8,"end_line":113,"end_character":68},"in_reply_to":"dc71d0e7_65f12a03","updated":"2025-08-12 16:44:43.000000000","message":"Done.\nThe method `_initialize_connection` is invoked by other `test_*` methods, and \u0027self.driver.primary_client.set_sdc_limits\u0027 is asserted in the `test_*` methods.","commit_id":"817868fc9511986a7719d46c523d66e8decef338"}],"cinder/tests/unit/volume/drivers/dell_emc/powerflex/test_initialize_connection_snapshot.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"42759e4048b7bfa6fc4810e2b4b2cb899cfdd1d5","unresolved":true,"context_lines":[{"line_number":56,"context_line":"        \"\"\"Test initializing when we do not know the snapshot size."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        ScaleIO can determine QOS specs based upon volume/snapshot size"},{"line_number":59,"context_line":"        The QOS keys should always be returned"},{"line_number":60,"context_line":"        \"\"\""},{"line_number":61,"context_line":"        snapshot \u003d fake_snapshot.fake_snapshot_obj("},{"line_number":62,"context_line":"            self.ctx, **{\u0027volume\u0027: self.volume,"}],"source_content_type":"text/x-python","patch_set":8,"id":"b951b3f6_fb5abbc2","line":59,"range":{"start_line":59,"start_character":8,"end_line":59,"end_character":46},"updated":"2025-08-18 15:46:22.000000000","message":"The test no longer checks for this, which seems kind of important.  Is there another test that checks this?  Or do you no longer care about whether the QOS keys are in the props?\n\nSame issue in the next test.","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1c1ab0b3fd769bcd063fc4220d44b5fbbfb9f748","unresolved":false,"context_lines":[{"line_number":56,"context_line":"        \"\"\"Test initializing when we do not know the snapshot size."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"        ScaleIO can determine QOS specs based upon volume/snapshot size"},{"line_number":59,"context_line":"        The QOS keys should always be returned"},{"line_number":60,"context_line":"        \"\"\""},{"line_number":61,"context_line":"        snapshot \u003d fake_snapshot.fake_snapshot_obj("},{"line_number":62,"context_line":"            self.ctx, **{\u0027volume\u0027: self.volume,"}],"source_content_type":"text/x-python","patch_set":8,"id":"16bb03fb_70dfcd26","line":59,"range":{"start_line":59,"start_character":8,"end_line":59,"end_character":46},"in_reply_to":"b951b3f6_fb5abbc2","updated":"2025-08-19 04:34:56.000000000","message":"The QOS keys are not returned anymore.","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"}],"cinder/volume/drivers/dell_emc/powerflex/driver.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f59a029956543bd7b262c98414c2b175fbece88a","unresolved":true,"context_lines":[{"line_number":1236,"context_line":"                    qos[key] \u003d value"},{"line_number":1237,"context_line":"        return qos"},{"line_number":1238,"context_line":""},{"line_number":1239,"context_line":"    def _sio_attach_volume(self, volume):"},{"line_number":1240,"context_line":"        \"\"\"Call connector.connect_volume() and return the path.\"\"\""},{"line_number":1241,"context_line":""},{"line_number":1242,"context_line":"        LOG.info(\"Call os-brick to attach PowerFlex volume.\")"},{"line_number":1243,"context_line":"        connection_properties \u003d self._get_client().connection_properties"},{"line_number":1244,"context_line":"        connection_properties[\"scaleIO_volname\"] \u003d flex_utils.id_to_base64("},{"line_number":1245,"context_line":"            volume.id"},{"line_number":1246,"context_line":"        )"},{"line_number":1247,"context_line":"        connection_properties[\"scaleIO_volume_id\"] \u003d volume.provider_id"},{"line_number":1248,"context_line":"        connection_properties[\"config_group\"] \u003d self.configuration.config_group"},{"line_number":1249,"context_line":"        connection_properties[\"failed_over\"] \u003d self._is_failed_over"},{"line_number":1250,"context_line":"        connection_properties[\"verify_certificate\"] \u003d ("},{"line_number":1251,"context_line":"            self._get_client().verify_certificate"},{"line_number":1252,"context_line":"        )"},{"line_number":1253,"context_line":"        connection_properties[\"certificate_path\"] \u003d ("},{"line_number":1254,"context_line":"            self._get_client().certificate_path"},{"line_number":1255,"context_line":"        )"},{"line_number":1256,"context_line":"        device_info \u003d self.connector.connect_volume(connection_properties)"},{"line_number":1257,"context_line":"        return device_info[\"path\"]"},{"line_number":1258,"context_line":""},{"line_number":1259,"context_line":"    def _sio_detach_volume(self, volume):"},{"line_number":1260,"context_line":"        \"\"\"Call the connector.disconnect().\"\"\""},{"line_number":1261,"context_line":""},{"line_number":1262,"context_line":"        LOG.info(\"Call os-brick to detach PowerFlex volume.\")"},{"line_number":1263,"context_line":"        connection_properties \u003d self._get_client().connection_properties"},{"line_number":1264,"context_line":"        connection_properties[\"scaleIO_volname\"] \u003d flex_utils.id_to_base64("},{"line_number":1265,"context_line":"            volume.id"},{"line_number":1266,"context_line":"        )"},{"line_number":1267,"context_line":"        connection_properties[\"scaleIO_volume_id\"] \u003d volume.provider_id"},{"line_number":1268,"context_line":"        connection_properties[\"config_group\"] \u003d self.configuration.config_group"},{"line_number":1269,"context_line":"        connection_properties[\"failed_over\"] \u003d self._is_failed_over"},{"line_number":1270,"context_line":"        connection_properties[\"verify_certificate\"] \u003d ("},{"line_number":1271,"context_line":"            self._get_client().verify_certificate"},{"line_number":1272,"context_line":"        )"},{"line_number":1273,"context_line":"        connection_properties[\"certificate_path\"] \u003d ("},{"line_number":1274,"context_line":"            self._get_client().certificate_path"},{"line_number":1275,"context_line":"        )"},{"line_number":1276,"context_line":""},{"line_number":1277,"context_line":"        self.connector.disconnect_volume(connection_properties, volume)"},{"line_number":1278,"context_line":""},{"line_number":1279,"context_line":"    def copy_image_to_volume(self, context, volume, image_service, image_id,"},{"line_number":1280,"context_line":"                             disable_sparse\u003dFalse):"},{"line_number":1281,"context_line":"        \"\"\"Fetch image from image service and write it to volume.\"\"\""},{"line_number":1282,"context_line":""},{"line_number":1283,"context_line":"        LOG.info(\"Copy image %(image_id)s from image service %(service)s \""},{"line_number":1284,"context_line":"                 \"to volume %(vol_id)s.\","},{"line_number":1285,"context_line":"                 {"},{"line_number":1286,"context_line":"                     \"image_id\": image_id,"},{"line_number":1287,"context_line":"                     \"service\": image_service,"},{"line_number":1288,"context_line":"                     \"vol_id\": volume.id,"},{"line_number":1289,"context_line":"                 })"},{"line_number":1290,"context_line":"        try:"},{"line_number":1291,"context_line":"            image_utils.fetch_to_raw(context,"},{"line_number":1292,"context_line":"                                     image_service,"},{"line_number":1293,"context_line":"                                     image_id,"},{"line_number":1294,"context_line":"                                     self._sio_attach_volume(volume),"},{"line_number":1295,"context_line":"                                     BLOCK_SIZE,"},{"line_number":1296,"context_line":"                                     size\u003dvolume.size,"},{"line_number":1297,"context_line":"                                     disable_sparse\u003ddisable_sparse)"},{"line_number":1298,"context_line":"        finally:"},{"line_number":1299,"context_line":"            self._sio_detach_volume(volume)"},{"line_number":1300,"context_line":""},{"line_number":1301,"context_line":"    def copy_volume_to_image(self, context, volume, image_service, image_meta):"},{"line_number":1302,"context_line":"        \"\"\"Copy volume to image on image service.\"\"\""},{"line_number":1303,"context_line":""},{"line_number":1304,"context_line":"        LOG.info(\"Copy volume %(vol_id)s to image on \""},{"line_number":1305,"context_line":"                 \"image service %(service)s. Image meta: %(meta)s.\","},{"line_number":1306,"context_line":"                 {"},{"line_number":1307,"context_line":"                     \"vol_id\": volume.id,"},{"line_number":1308,"context_line":"                     \"service\": image_service,"},{"line_number":1309,"context_line":"                     \"meta\": image_meta,"},{"line_number":1310,"context_line":"                 })"},{"line_number":1311,"context_line":"        try:"},{"line_number":1312,"context_line":"            volume_utils.upload_volume(context,"},{"line_number":1313,"context_line":"                                       image_service,"},{"line_number":1314,"context_line":"                                       image_meta,"},{"line_number":1315,"context_line":"                                       self._sio_attach_volume(volume),"},{"line_number":1316,"context_line":"                                       volume)"},{"line_number":1317,"context_line":"        finally:"},{"line_number":1318,"context_line":"            self._sio_detach_volume(volume)"},{"line_number":1319,"context_line":""},{"line_number":1320,"context_line":"    def migrate_volume(self, ctxt, volume, host):"},{"line_number":1321,"context_line":"        \"\"\"Migrate PowerFlex volume within the same backend.\"\"\""},{"line_number":1322,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"8634a4cc_d2aca858","side":"PARENT","line":1319,"range":{"start_line":1239,"start_character":0,"end_line":1319,"end_character":0},"updated":"2025-08-04 10:44:44.000000000","message":"this is a great improvement/simplification in the workflow by using the generic mechanism for these operations","commit_id":"c948b22eace9f6aa299a0af3ac374864ff8ff163"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1656245c55b0e0d4f8140d3da807ada17aea4458","unresolved":false,"context_lines":[{"line_number":1236,"context_line":"                    qos[key] \u003d value"},{"line_number":1237,"context_line":"        return qos"},{"line_number":1238,"context_line":""},{"line_number":1239,"context_line":"    def _sio_attach_volume(self, volume):"},{"line_number":1240,"context_line":"        \"\"\"Call connector.connect_volume() and return the path.\"\"\""},{"line_number":1241,"context_line":""},{"line_number":1242,"context_line":"        LOG.info(\"Call os-brick to attach PowerFlex volume.\")"},{"line_number":1243,"context_line":"        connection_properties \u003d self._get_client().connection_properties"},{"line_number":1244,"context_line":"        connection_properties[\"scaleIO_volname\"] \u003d flex_utils.id_to_base64("},{"line_number":1245,"context_line":"            volume.id"},{"line_number":1246,"context_line":"        )"},{"line_number":1247,"context_line":"        connection_properties[\"scaleIO_volume_id\"] \u003d volume.provider_id"},{"line_number":1248,"context_line":"        connection_properties[\"config_group\"] \u003d self.configuration.config_group"},{"line_number":1249,"context_line":"        connection_properties[\"failed_over\"] \u003d self._is_failed_over"},{"line_number":1250,"context_line":"        connection_properties[\"verify_certificate\"] \u003d ("},{"line_number":1251,"context_line":"            self._get_client().verify_certificate"},{"line_number":1252,"context_line":"        )"},{"line_number":1253,"context_line":"        connection_properties[\"certificate_path\"] \u003d ("},{"line_number":1254,"context_line":"            self._get_client().certificate_path"},{"line_number":1255,"context_line":"        )"},{"line_number":1256,"context_line":"        device_info \u003d self.connector.connect_volume(connection_properties)"},{"line_number":1257,"context_line":"        return device_info[\"path\"]"},{"line_number":1258,"context_line":""},{"line_number":1259,"context_line":"    def _sio_detach_volume(self, volume):"},{"line_number":1260,"context_line":"        \"\"\"Call the connector.disconnect().\"\"\""},{"line_number":1261,"context_line":""},{"line_number":1262,"context_line":"        LOG.info(\"Call os-brick to detach PowerFlex volume.\")"},{"line_number":1263,"context_line":"        connection_properties \u003d self._get_client().connection_properties"},{"line_number":1264,"context_line":"        connection_properties[\"scaleIO_volname\"] \u003d flex_utils.id_to_base64("},{"line_number":1265,"context_line":"            volume.id"},{"line_number":1266,"context_line":"        )"},{"line_number":1267,"context_line":"        connection_properties[\"scaleIO_volume_id\"] \u003d volume.provider_id"},{"line_number":1268,"context_line":"        connection_properties[\"config_group\"] \u003d self.configuration.config_group"},{"line_number":1269,"context_line":"        connection_properties[\"failed_over\"] \u003d self._is_failed_over"},{"line_number":1270,"context_line":"        connection_properties[\"verify_certificate\"] \u003d ("},{"line_number":1271,"context_line":"            self._get_client().verify_certificate"},{"line_number":1272,"context_line":"        )"},{"line_number":1273,"context_line":"        connection_properties[\"certificate_path\"] \u003d ("},{"line_number":1274,"context_line":"            self._get_client().certificate_path"},{"line_number":1275,"context_line":"        )"},{"line_number":1276,"context_line":""},{"line_number":1277,"context_line":"        self.connector.disconnect_volume(connection_properties, volume)"},{"line_number":1278,"context_line":""},{"line_number":1279,"context_line":"    def copy_image_to_volume(self, context, volume, image_service, image_id,"},{"line_number":1280,"context_line":"                             disable_sparse\u003dFalse):"},{"line_number":1281,"context_line":"        \"\"\"Fetch image from image service and write it to volume.\"\"\""},{"line_number":1282,"context_line":""},{"line_number":1283,"context_line":"        LOG.info(\"Copy image %(image_id)s from image service %(service)s \""},{"line_number":1284,"context_line":"                 \"to volume %(vol_id)s.\","},{"line_number":1285,"context_line":"                 {"},{"line_number":1286,"context_line":"                     \"image_id\": image_id,"},{"line_number":1287,"context_line":"                     \"service\": image_service,"},{"line_number":1288,"context_line":"                     \"vol_id\": volume.id,"},{"line_number":1289,"context_line":"                 })"},{"line_number":1290,"context_line":"        try:"},{"line_number":1291,"context_line":"            image_utils.fetch_to_raw(context,"},{"line_number":1292,"context_line":"                                     image_service,"},{"line_number":1293,"context_line":"                                     image_id,"},{"line_number":1294,"context_line":"                                     self._sio_attach_volume(volume),"},{"line_number":1295,"context_line":"                                     BLOCK_SIZE,"},{"line_number":1296,"context_line":"                                     size\u003dvolume.size,"},{"line_number":1297,"context_line":"                                     disable_sparse\u003ddisable_sparse)"},{"line_number":1298,"context_line":"        finally:"},{"line_number":1299,"context_line":"            self._sio_detach_volume(volume)"},{"line_number":1300,"context_line":""},{"line_number":1301,"context_line":"    def copy_volume_to_image(self, context, volume, image_service, image_meta):"},{"line_number":1302,"context_line":"        \"\"\"Copy volume to image on image service.\"\"\""},{"line_number":1303,"context_line":""},{"line_number":1304,"context_line":"        LOG.info(\"Copy volume %(vol_id)s to image on \""},{"line_number":1305,"context_line":"                 \"image service %(service)s. Image meta: %(meta)s.\","},{"line_number":1306,"context_line":"                 {"},{"line_number":1307,"context_line":"                     \"vol_id\": volume.id,"},{"line_number":1308,"context_line":"                     \"service\": image_service,"},{"line_number":1309,"context_line":"                     \"meta\": image_meta,"},{"line_number":1310,"context_line":"                 })"},{"line_number":1311,"context_line":"        try:"},{"line_number":1312,"context_line":"            volume_utils.upload_volume(context,"},{"line_number":1313,"context_line":"                                       image_service,"},{"line_number":1314,"context_line":"                                       image_meta,"},{"line_number":1315,"context_line":"                                       self._sio_attach_volume(volume),"},{"line_number":1316,"context_line":"                                       volume)"},{"line_number":1317,"context_line":"        finally:"},{"line_number":1318,"context_line":"            self._sio_detach_volume(volume)"},{"line_number":1319,"context_line":""},{"line_number":1320,"context_line":"    def migrate_volume(self, ctxt, volume, host):"},{"line_number":1321,"context_line":"        \"\"\"Migrate PowerFlex volume within the same backend.\"\"\""},{"line_number":1322,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"7ba19050_a89dc61e","side":"PARENT","line":1319,"range":{"start_line":1239,"start_character":0,"end_line":1319,"end_character":0},"in_reply_to":"8634a4cc_d2aca858","updated":"2025-08-10 07:44:54.000000000","message":"Agree.","commit_id":"c948b22eace9f6aa299a0af3ac374864ff8ff163"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f59a029956543bd7b262c98414c2b175fbece88a","unresolved":true,"context_lines":[{"line_number":880,"context_line":"        PowerFlex driver returns a driver_volume_type of \u0027scaleio\u0027."},{"line_number":881,"context_line":"        \"\"\""},{"line_number":882,"context_line":""},{"line_number":883,"context_line":"        try:"},{"line_number":884,"context_line":"            sdc_guid \u003d connector[\"sdc_guid\"]"},{"line_number":885,"context_line":"        except Exception:"},{"line_number":886,"context_line":"            msg \u003d \"SDC guid is not configured.\""},{"line_number":887,"context_line":"            raise exception.InvalidHost(reason\u003dmsg)"},{"line_number":888,"context_line":""},{"line_number":889,"context_line":"        LOG.info(\"Initialize connection for %(vol_id)s to SDC %(sdc)s.\","},{"line_number":890,"context_line":"                 {\"vol_id\": vol_or_snap.id, \"sdc\": sdc_guid})"}],"source_content_type":"text/x-python","patch_set":6,"id":"36e00749_42e64606","line":887,"range":{"start_line":883,"start_character":8,"end_line":887,"end_character":51},"updated":"2025-08-04 10:44:44.000000000","message":"what happens if the os-brick is from an older release and we would like to try a legacy path where os-brick doesn\u0027t return the GUID and maps the volume itself?\nwe already support all the legacy paths in os-brick so the vice versa could be true as well","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1656245c55b0e0d4f8140d3da807ada17aea4458","unresolved":false,"context_lines":[{"line_number":880,"context_line":"        PowerFlex driver returns a driver_volume_type of \u0027scaleio\u0027."},{"line_number":881,"context_line":"        \"\"\""},{"line_number":882,"context_line":""},{"line_number":883,"context_line":"        try:"},{"line_number":884,"context_line":"            sdc_guid \u003d connector[\"sdc_guid\"]"},{"line_number":885,"context_line":"        except Exception:"},{"line_number":886,"context_line":"            msg \u003d \"SDC guid is not configured.\""},{"line_number":887,"context_line":"            raise exception.InvalidHost(reason\u003dmsg)"},{"line_number":888,"context_line":""},{"line_number":889,"context_line":"        LOG.info(\"Initialize connection for %(vol_id)s to SDC %(sdc)s.\","},{"line_number":890,"context_line":"                 {\"vol_id\": vol_or_snap.id, \"sdc\": sdc_guid})"}],"source_content_type":"text/x-python","patch_set":6,"id":"e225ef9d_5732e174","line":887,"range":{"start_line":883,"start_character":8,"end_line":887,"end_character":51},"in_reply_to":"36e00749_42e64606","updated":"2025-08-10 07:44:54.000000000","message":"The new driver is not compatible with the old \u0027os-brick\u0027.\nConsidering the upcoming changes, e.g., NVMe-TCP support and PowerFlex 5.x support, maintaining compatibility with the old \u0027os-brick\u0027 is overwhelming.","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"4a4c89f6d05a01cd38bdb1406479f463016659ad","unresolved":false,"context_lines":[{"line_number":880,"context_line":"        PowerFlex driver returns a driver_volume_type of \u0027scaleio\u0027."},{"line_number":881,"context_line":"        \"\"\""},{"line_number":882,"context_line":""},{"line_number":883,"context_line":"        try:"},{"line_number":884,"context_line":"            sdc_guid \u003d connector[\"sdc_guid\"]"},{"line_number":885,"context_line":"        except Exception:"},{"line_number":886,"context_line":"            msg \u003d \"SDC guid is not configured.\""},{"line_number":887,"context_line":"            raise exception.InvalidHost(reason\u003dmsg)"},{"line_number":888,"context_line":""},{"line_number":889,"context_line":"        LOG.info(\"Initialize connection for %(vol_id)s to SDC %(sdc)s.\","},{"line_number":890,"context_line":"                 {\"vol_id\": vol_or_snap.id, \"sdc\": sdc_guid})"}],"source_content_type":"text/x-python","patch_set":6,"id":"9a96eb70_59d73444","line":887,"range":{"start_line":883,"start_character":8,"end_line":887,"end_character":51},"in_reply_to":"e225ef9d_5732e174","updated":"2025-08-11 20:07:01.000000000","message":"I understand that but in an OpenStack deployment, there are possible scenarios where the compute node is older than the controller node so technically we will be running a \"new\" cinder on controller with \"old\" os-brick code on compute and not supporting backward compatibility seems like a recipe for disaster.","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f59a029956543bd7b262c98414c2b175fbece88a","unresolved":true,"context_lines":[{"line_number":937,"context_line":"            ids \u003d [sdc[\"sdcId\"] for sdc in vol[\"mappedSdcInfo\"]]"},{"line_number":938,"context_line":"            if sdc_id in ids:"},{"line_number":939,"context_line":"                LOG.debug(\"PowerFlex volume %(volume_name)s \""},{"line_number":940,"context_line":"                          \"with id %(volume_id)s is alreday attached to \""},{"line_number":941,"context_line":"                          \"host %(host_name)s. \""},{"line_number":942,"context_line":"                          \"PowerFlex volume id: %(volume_provider_id)s, \""},{"line_number":943,"context_line":"                          \"host id: %(host_provider_id)s. \","}],"source_content_type":"text/x-python","patch_set":6,"id":"20eb0696_f784988b","line":940,"range":{"start_line":940,"start_character":52,"end_line":940,"end_character":59},"updated":"2025-08-04 10:44:44.000000000","message":"already","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1656245c55b0e0d4f8140d3da807ada17aea4458","unresolved":false,"context_lines":[{"line_number":937,"context_line":"            ids \u003d [sdc[\"sdcId\"] for sdc in vol[\"mappedSdcInfo\"]]"},{"line_number":938,"context_line":"            if sdc_id in ids:"},{"line_number":939,"context_line":"                LOG.debug(\"PowerFlex volume %(volume_name)s \""},{"line_number":940,"context_line":"                          \"with id %(volume_id)s is alreday attached to \""},{"line_number":941,"context_line":"                          \"host %(host_name)s. \""},{"line_number":942,"context_line":"                          \"PowerFlex volume id: %(volume_provider_id)s, \""},{"line_number":943,"context_line":"                          \"host id: %(host_provider_id)s. \","}],"source_content_type":"text/x-python","patch_set":6,"id":"934e71cb_d8fb47ce","line":940,"range":{"start_line":940,"start_character":52,"end_line":940,"end_character":59},"in_reply_to":"20eb0696_f784988b","updated":"2025-08-10 07:44:54.000000000","message":"Done","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f59a029956543bd7b262c98414c2b175fbece88a","unresolved":true,"context_lines":[{"line_number":977,"context_line":"    def _check_volume_mapped(self, sdc_id, volume_id):"},{"line_number":978,"context_line":"        mappedVols \u003d self._get_client().query_sdc_volumes(sdc_id)"},{"line_number":979,"context_line":"        if volume_id not in mappedVols:"},{"line_number":980,"context_line":"            msg \u003d f\u0027Volume {volume_id} is not mapped to SDC {sdc_id}.\u0027"},{"line_number":981,"context_line":"            raise exception.VolumeBackendAPIException(msg)"},{"line_number":982,"context_line":"        LOG.info(\"Volume %s is mapped to SDC %s.\", volume_id, sdc_id)"},{"line_number":983,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"f204036d_92bb76ad","line":980,"range":{"start_line":980,"start_character":68,"end_line":980,"end_character":70},"updated":"2025-08-04 10:44:44.000000000","message":"nit: maybe add a \"retrying {i}th time\" to indicate we re-tried N times","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1656245c55b0e0d4f8140d3da807ada17aea4458","unresolved":false,"context_lines":[{"line_number":977,"context_line":"    def _check_volume_mapped(self, sdc_id, volume_id):"},{"line_number":978,"context_line":"        mappedVols \u003d self._get_client().query_sdc_volumes(sdc_id)"},{"line_number":979,"context_line":"        if volume_id not in mappedVols:"},{"line_number":980,"context_line":"            msg \u003d f\u0027Volume {volume_id} is not mapped to SDC {sdc_id}.\u0027"},{"line_number":981,"context_line":"            raise exception.VolumeBackendAPIException(msg)"},{"line_number":982,"context_line":"        LOG.info(\"Volume %s is mapped to SDC %s.\", volume_id, sdc_id)"},{"line_number":983,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"2ee8d527_e140e742","line":980,"range":{"start_line":980,"start_character":68,"end_line":980,"end_character":70},"in_reply_to":"f204036d_92bb76ad","updated":"2025-08-10 07:44:54.000000000","message":"It seems the \u0027@utils.retry\u0027 decorator didn\u0027t provide an approach to get the retry count \u0027i\u0027.","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f59a029956543bd7b262c98414c2b175fbece88a","unresolved":true,"context_lines":[{"line_number":1061,"context_line":"            if is_multiattached:"},{"line_number":1062,"context_line":"                # Do not detach volume if it is attached to more than one"},{"line_number":1063,"context_line":"                # instance on the same host."},{"line_number":1064,"context_line":"                LOG.info(\"Will not terminate connection for \""},{"line_number":1065,"context_line":"                         \"%(vol_id)s to initiator at %(sdc)s.\","},{"line_number":1066,"context_line":"                         {\"vol_id\": volume_or_snap.id, \"sdc\": sdc_guid})"},{"line_number":1067,"context_line":"                return"},{"line_number":1068,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"f33078d1_aaa3a652","line":1065,"range":{"start_line":1064,"start_character":26,"end_line":1065,"end_character":63},"updated":"2025-08-04 10:44:44.000000000","message":"probably add the reason why we won\u0027t terminate it like\n\n    to initiator at %(sdc)s because it\u0027s multiattach.","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1656245c55b0e0d4f8140d3da807ada17aea4458","unresolved":false,"context_lines":[{"line_number":1061,"context_line":"            if is_multiattached:"},{"line_number":1062,"context_line":"                # Do not detach volume if it is attached to more than one"},{"line_number":1063,"context_line":"                # instance on the same host."},{"line_number":1064,"context_line":"                LOG.info(\"Will not terminate connection for \""},{"line_number":1065,"context_line":"                         \"%(vol_id)s to initiator at %(sdc)s.\","},{"line_number":1066,"context_line":"                         {\"vol_id\": volume_or_snap.id, \"sdc\": sdc_guid})"},{"line_number":1067,"context_line":"                return"},{"line_number":1068,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"cc924686_89692403","line":1065,"range":{"start_line":1064,"start_character":26,"end_line":1065,"end_character":63},"in_reply_to":"f33078d1_aaa3a652","updated":"2025-08-10 07:44:54.000000000","message":"Done","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f59a029956543bd7b262c98414c2b175fbece88a","unresolved":true,"context_lines":[{"line_number":1111,"context_line":"        # check if volume is already detached"},{"line_number":1112,"context_line":"        if not vol[\"mappedSdcInfo\"]:"},{"line_number":1113,"context_line":"            LOG.debug(\"PowerFlex volume %(volume_name)s \""},{"line_number":1114,"context_line":"                      \"with id %(volume_id)s is alreday detached from \""},{"line_number":1115,"context_line":"                      \"all hosts. \""},{"line_number":1116,"context_line":"                      \"PowerFlex volume id: %(volume_provider_id)s. \","},{"line_number":1117,"context_line":"                      {"}],"source_content_type":"text/x-python","patch_set":6,"id":"9df4801b_c8a64e0a","line":1114,"range":{"start_line":1114,"start_character":48,"end_line":1114,"end_character":55},"updated":"2025-08-04 10:44:44.000000000","message":"already","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1656245c55b0e0d4f8140d3da807ada17aea4458","unresolved":false,"context_lines":[{"line_number":1111,"context_line":"        # check if volume is already detached"},{"line_number":1112,"context_line":"        if not vol[\"mappedSdcInfo\"]:"},{"line_number":1113,"context_line":"            LOG.debug(\"PowerFlex volume %(volume_name)s \""},{"line_number":1114,"context_line":"                      \"with id %(volume_id)s is alreday detached from \""},{"line_number":1115,"context_line":"                      \"all hosts. \""},{"line_number":1116,"context_line":"                      \"PowerFlex volume id: %(volume_provider_id)s. \","},{"line_number":1117,"context_line":"                      {"}],"source_content_type":"text/x-python","patch_set":6,"id":"35e85e45_dacba214","line":1114,"range":{"start_line":1114,"start_character":48,"end_line":1114,"end_character":55},"in_reply_to":"9df4801b_c8a64e0a","updated":"2025-08-10 07:44:54.000000000","message":"Done","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f59a029956543bd7b262c98414c2b175fbece88a","unresolved":true,"context_lines":[{"line_number":1127,"context_line":"            ids \u003d [sdc[\"sdcId\"] for sdc in vol[\"mappedSdcInfo\"]]"},{"line_number":1128,"context_line":"            if sdc_id not in ids:"},{"line_number":1129,"context_line":"                LOG.debug(\"PowerFlex volume %(volume_name)s \""},{"line_number":1130,"context_line":"                          \"with id %(volume_id)s is alreday detached from \""},{"line_number":1131,"context_line":"                          \"host %(host_name)s. \""},{"line_number":1132,"context_line":"                          \"PowerFlex volume id: %(volume_provider_id)s, \""},{"line_number":1133,"context_line":"                          \"host id: %(host_provider_id)s. \","}],"source_content_type":"text/x-python","patch_set":6,"id":"f7edf0a9_a0e2719a","line":1130,"range":{"start_line":1130,"start_character":52,"end_line":1130,"end_character":59},"updated":"2025-08-04 10:44:44.000000000","message":"already","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1656245c55b0e0d4f8140d3da807ada17aea4458","unresolved":false,"context_lines":[{"line_number":1127,"context_line":"            ids \u003d [sdc[\"sdcId\"] for sdc in vol[\"mappedSdcInfo\"]]"},{"line_number":1128,"context_line":"            if sdc_id not in ids:"},{"line_number":1129,"context_line":"                LOG.debug(\"PowerFlex volume %(volume_name)s \""},{"line_number":1130,"context_line":"                          \"with id %(volume_id)s is alreday detached from \""},{"line_number":1131,"context_line":"                          \"host %(host_name)s. \""},{"line_number":1132,"context_line":"                          \"PowerFlex volume id: %(volume_provider_id)s, \""},{"line_number":1133,"context_line":"                          \"host id: %(host_provider_id)s. \","}],"source_content_type":"text/x-python","patch_set":6,"id":"0699271e_1cccd683","line":1130,"range":{"start_line":1130,"start_character":52,"end_line":1130,"end_character":59},"in_reply_to":"f7edf0a9_a0e2719a","updated":"2025-08-10 07:44:54.000000000","message":"Done","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"}],"releasenotes/notes/bug-2114879-dell-powerflex-improve-secret-handling-b1217791a9dceb1a.yaml":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"f59a029956543bd7b262c98414c2b175fbece88a","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell PowerFlex driver `Bug #2114879"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_: Improved"},{"line_number":6,"context_line":"    PowerFlex secret handling. The connector configuration is"},{"line_number":7,"context_line":"    deprecated."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"b285ecf9_2a787694","line":7,"range":{"start_line":4,"start_character":0,"end_line":7,"end_character":15},"updated":"2025-08-04 10:44:44.000000000","message":"we are not just improving the secret handling, we are moving the whole logic to map/unmap volume from os-brick to cinder.\nThis releasenote should have the security, upgrades, deprecations etc sections same as the os-brick patch.","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1656245c55b0e0d4f8140d3da807ada17aea4458","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell PowerFlex driver `Bug #2114879"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_: Improved"},{"line_number":6,"context_line":"    PowerFlex secret handling. The connector configuration is"},{"line_number":7,"context_line":"    deprecated."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"ab809e69_b09daee6","line":7,"range":{"start_line":4,"start_character":0,"end_line":7,"end_character":15},"in_reply_to":"5c8dae57_027fa6a4","updated":"2025-08-10 07:44:54.000000000","message":"Done","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"92f834647c662d2a2244f491ca9a921cd5239259","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell PowerFlex driver `Bug #2114879"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_: Improved"},{"line_number":6,"context_line":"    PowerFlex secret handling. The connector configuration is"},{"line_number":7,"context_line":"    deprecated."}],"source_content_type":"text/x-yaml","patch_set":6,"id":"5c8dae57_027fa6a4","line":7,"range":{"start_line":4,"start_character":0,"end_line":7,"end_character":15},"in_reply_to":"b285ecf9_2a787694","updated":"2025-08-07 12:13:15.000000000","message":"You can use the release note from the original fix as a model:\nreleasenotes/notes/bug-1823200-victoria-ecd2d99c9223d84b.yaml","commit_id":"f0eb9803530a20099678e9d6970d5551ec8e6c9e"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"42759e4048b7bfa6fc4810e2b4b2cb899cfdd1d5","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell PowerFlex driver: This release contains a fix for"},{"line_number":5,"context_line":"    `Bug #2114879 \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_."},{"line_number":6,"context_line":"    It removed the limitation of use with bare metal hosts mentioned in"},{"line_number":7,"context_line":"    `OSSN-0086 \u003chttps://wiki.openstack.org/wiki/OSSN/OSSN-0086\u003e`_."},{"line_number":8,"context_line":"upgrade:"},{"line_number":9,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"8f09f406_73dc7d55","line":6,"range":{"start_line":6,"start_character":13,"end_line":6,"end_character":14},"updated":"2025-08-18 15:46:22.000000000","message":"nit: removes","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1c1ab0b3fd769bcd063fc4220d44b5fbbfb9f748","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell PowerFlex driver: This release contains a fix for"},{"line_number":5,"context_line":"    `Bug #2114879 \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_."},{"line_number":6,"context_line":"    It removed the limitation of use with bare metal hosts mentioned in"},{"line_number":7,"context_line":"    `OSSN-0086 \u003chttps://wiki.openstack.org/wiki/OSSN/OSSN-0086\u003e`_."},{"line_number":8,"context_line":"upgrade:"},{"line_number":9,"context_line":"  - |"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"60ac66be_19397569","line":6,"range":{"start_line":6,"start_character":13,"end_line":6,"end_character":14},"in_reply_to":"8f09f406_73dc7d55","updated":"2025-08-19 04:34:56.000000000","message":"Done","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"42759e4048b7bfa6fc4810e2b4b2cb899cfdd1d5","unresolved":true,"context_lines":[{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    The fix for `Bug #2114879"},{"line_number":11,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_ requires"},{"line_number":12,"context_line":"    ``os-brick`` version 6.13.0 or greater. users do not need to"},{"line_number":13,"context_line":"    create the `/opt/emc/scaleio/openstack/connector.conf` file"},{"line_number":14,"context_line":"    on the hosts using ``os-brick``."},{"line_number":15,"context_line":"fixes:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"b3a5cfe4_b6f824fd","line":12,"range":{"start_line":12,"start_character":44,"end_line":12,"end_character":45},"updated":"2025-08-18 15:46:22.000000000","message":"nit: should be uppercase \u0027U\u0027.","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1c1ab0b3fd769bcd063fc4220d44b5fbbfb9f748","unresolved":false,"context_lines":[{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    The fix for `Bug #2114879"},{"line_number":11,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_ requires"},{"line_number":12,"context_line":"    ``os-brick`` version 6.13.0 or greater. users do not need to"},{"line_number":13,"context_line":"    create the `/opt/emc/scaleio/openstack/connector.conf` file"},{"line_number":14,"context_line":"    on the hosts using ``os-brick``."},{"line_number":15,"context_line":"fixes:"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"109f8e11_7b002945","line":12,"range":{"start_line":12,"start_character":44,"end_line":12,"end_character":45},"in_reply_to":"b3a5cfe4_b6f824fd","updated":"2025-08-19 04:34:56.000000000","message":"Done","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"42759e4048b7bfa6fc4810e2b4b2cb899cfdd1d5","unresolved":true,"context_lines":[{"line_number":17,"context_line":"    Dell PowerFlex driver `Bug #2114879"},{"line_number":18,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_:"},{"line_number":19,"context_line":"    This release contains an updated Dell PowerFlex driver. It must"},{"line_number":20,"context_line":"    be used with ``os-brick`` version 6.13.0 or greater. The connector"},{"line_number":21,"context_line":"    passed PowerFlex SDC GUID in the connector properties. Dell PowerFlex"},{"line_number":22,"context_line":"    cinder driver retrieved SDC GUID from the connector properties and"},{"line_number":23,"context_line":"    handled the mapping logic."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"25bea836_7fd6cdd0","line":23,"range":{"start_line":20,"start_character":57,"end_line":23,"end_character":30},"updated":"2025-08-18 15:46:22.000000000","message":"same comment as in the os-brick patch about the mix of tenses, it makes it difficult to follow.","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1c1ab0b3fd769bcd063fc4220d44b5fbbfb9f748","unresolved":false,"context_lines":[{"line_number":17,"context_line":"    Dell PowerFlex driver `Bug #2114879"},{"line_number":18,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_:"},{"line_number":19,"context_line":"    This release contains an updated Dell PowerFlex driver. It must"},{"line_number":20,"context_line":"    be used with ``os-brick`` version 6.13.0 or greater. The connector"},{"line_number":21,"context_line":"    passed PowerFlex SDC GUID in the connector properties. Dell PowerFlex"},{"line_number":22,"context_line":"    cinder driver retrieved SDC GUID from the connector properties and"},{"line_number":23,"context_line":"    handled the mapping logic."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"e44161d2_3ff677a9","line":23,"range":{"start_line":20,"start_character":57,"end_line":23,"end_character":30},"in_reply_to":"25bea836_7fd6cdd0","updated":"2025-08-19 04:34:56.000000000","message":"Done","commit_id":"39149aabbabd6a8e6a5b55c2cc89292df14c7166"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"73ba5bbcb264dba0485a47cc78a51608c0a65368","unresolved":true,"context_lines":[{"line_number":19,"context_line":"       the configuration file. This version can perform mapping"},{"line_number":20,"context_line":"       if the driver has not yet done so, provided the configuration"},{"line_number":21,"context_line":"       file remains intact."},{"line_number":22,"context_line":"    2. Then upgrade the PowerFlex driver to version 3.6.0 or later."},{"line_number":23,"context_line":"       Note that driver version 3.6.0 requires ``os-brick`` version 6.13.0"},{"line_number":24,"context_line":"       or higher to function correctly and will not operate with"},{"line_number":25,"context_line":"       earlier versions of ``os-brick``."},{"line_number":26,"context_line":"fixes:"},{"line_number":27,"context_line":"  - |"},{"line_number":28,"context_line":"    Dell PowerFlex driver `Bug #2114879"},{"line_number":29,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"3b8c6f42_800bba5a","line":26,"range":{"start_line":22,"start_character":7,"end_line":26,"end_character":0},"updated":"2025-09-01 08:11:24.000000000","message":"Since we are restricting users from removing the config file in step 1, we should mention that we can remove the configuration file after this step","commit_id":"83d8a87ac8a89e6a8d46c1632051f8d5d4f841ae"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"f753a13118709456fc4432ed8b30e093f0c4e81a","unresolved":false,"context_lines":[{"line_number":19,"context_line":"       the configuration file. This version can perform mapping"},{"line_number":20,"context_line":"       if the driver has not yet done so, provided the configuration"},{"line_number":21,"context_line":"       file remains intact."},{"line_number":22,"context_line":"    2. Then upgrade the PowerFlex driver to version 3.6.0 or later."},{"line_number":23,"context_line":"       Note that driver version 3.6.0 requires ``os-brick`` version 6.13.0"},{"line_number":24,"context_line":"       or higher to function correctly and will not operate with"},{"line_number":25,"context_line":"       earlier versions of ``os-brick``."},{"line_number":26,"context_line":"fixes:"},{"line_number":27,"context_line":"  - |"},{"line_number":28,"context_line":"    Dell PowerFlex driver `Bug #2114879"},{"line_number":29,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"ac056b04_1c3edbb9","line":26,"range":{"start_line":22,"start_character":7,"end_line":26,"end_character":0},"in_reply_to":"3b8c6f42_800bba5a","updated":"2025-09-01 08:40:31.000000000","message":"Done","commit_id":"83d8a87ac8a89e6a8d46c1632051f8d5d4f841ae"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"0d6ab93c4b12ee7062b380d5d4024902b6fd534c","unresolved":true,"context_lines":[{"line_number":7,"context_line":"    `OSSN-0086 \u003chttps://wiki.openstack.org/wiki/OSSN/OSSN-0086\u003e`_."},{"line_number":8,"context_line":"upgrade:"},{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    The fix for `Bug #2114879"},{"line_number":11,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_ requires"},{"line_number":12,"context_line":"    ``os-brick`` version 6.13.0 or greater. Users do not need to"},{"line_number":13,"context_line":"    create the `/opt/emc/scaleio/openstack/connector.conf` file"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"b3e22bc7_b5729f28","line":10,"updated":"2025-09-02 13:03:13.000000000","message":"You\u0027re going to hate me for this, but take a look at https://90d93d4fa3e6e69abef7-05cf8d45f98e4972de234be1a80e25da.ssl.cf5.rackcdn.com/openstack/50c1206f13a24ca0a4906fc61ca275e9/docs/unreleased.html\n\nYou really need to prefix line 10 with \u0027Dell PowerFlex driver:\u0027 so that it\u0027s obvious what this upgrade note applies to.","commit_id":"14fba988949f08e5ebc4cd03cbd8a2843625ea0b"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"b240c847f3419cb6476c7411dbfe437708957056","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    `OSSN-0086 \u003chttps://wiki.openstack.org/wiki/OSSN/OSSN-0086\u003e`_."},{"line_number":8,"context_line":"upgrade:"},{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    The fix for `Bug #2114879"},{"line_number":11,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/2114879\u003e`_ requires"},{"line_number":12,"context_line":"    ``os-brick`` version 6.13.0 or greater. Users do not need to"},{"line_number":13,"context_line":"    create the `/opt/emc/scaleio/openstack/connector.conf` file"}],"source_content_type":"text/x-yaml","patch_set":12,"id":"67de5638_f6898f17","line":10,"in_reply_to":"b3e22bc7_b5729f28","updated":"2025-09-02 13:54:35.000000000","message":"Done","commit_id":"14fba988949f08e5ebc4cd03cbd8a2843625ea0b"}]}
