)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"fa5fb63c94baa5636148ed5693a46bc95e977c47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a7fc7b27_7ada4c69","updated":"2025-07-25 06:29:31.000000000","message":"run-DellEMC PowerScale CI","commit_id":"0455bc5de8ca034b920e4700ed0279498b8e13a4"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"da82588f5b3576c238de7f7881845a2947af01ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1ce491e6_8d50d1ea","updated":"2025-07-31 22:06:12.000000000","message":"Hi thank you for your patience with the reviews here. Please see my comment inline","commit_id":"afd2c3b67339ee6a0ef081b1e305a44135082a0b"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"5e111d4bcbc06dfde762fa8f0412575a3296a29a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"55b748ce_291a274f","updated":"2025-07-28 12:44:13.000000000","message":"recheck manila-tempest-plugin-dummy-dhss Unable to locate package ksmtuned","commit_id":"afd2c3b67339ee6a0ef081b1e305a44135082a0b"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"eb16081aab38641b5595b671d0d74b8d86e41935","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"2c1fe611_48ee0a2e","updated":"2025-07-28 12:44:34.000000000","message":"run-DellEMC PowerScale CI","commit_id":"afd2c3b67339ee6a0ef081b1e305a44135082a0b"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"24a0412cd842243da893659f02124d2aa9213d83","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cd106f50_438052a4","updated":"2025-08-12 14:21:36.000000000","message":"LGTM, thanks for the fix. Please address Goutham\u0027s suggestion in a follow-up change","commit_id":"28c30b5065db0d6fa95f8063f4ffe6d9051d428a"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"a72297802ee9c040b568e97a910c9141a9a478e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d44bb962_9f8bb375","updated":"2025-08-02 07:56:02.000000000","message":"recheck","commit_id":"28c30b5065db0d6fa95f8063f4ffe6d9051d428a"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"20cfdab8d5ea146940d7f95739659c01e2a729e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ee765e2a_04e62948","updated":"2025-08-03 05:09:41.000000000","message":"recheck manila-tempest-plugin-lvm","commit_id":"28c30b5065db0d6fa95f8063f4ffe6d9051d428a"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"182dc5d82fe8ae01f0c82452d8aba9f4cab80035","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a51b62e6_87a83dd0","updated":"2025-08-02 10:12:17.000000000","message":"recheck manila-tempest-plugin-lvm Unable to locate package ksmtuned","commit_id":"28c30b5065db0d6fa95f8063f4ffe6d9051d428a"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"ee8da852afaae2dc5e595018101c7f2d6c03b7db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"ce59c1ee_8469a707","updated":"2025-08-02 08:36:06.000000000","message":"recheck manila-tempest-plugin-lvm Unable to locate package ksmtuned","commit_id":"28c30b5065db0d6fa95f8063f4ffe6d9051d428a"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"38018660e1b53c3d393d7a9802b02a3af3f3ad53","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"258e5a10_59afafbc","updated":"2025-08-03 07:46:07.000000000","message":"run-DellEMC PowerScale CI","commit_id":"28c30b5065db0d6fa95f8063f4ffe6d9051d428a"}],"manila/share/drivers/dell_emc/plugins/isilon/isilon.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"da82588f5b3576c238de7f7881845a2947af01ec","unresolved":true,"context_lines":[{"line_number":349,"context_line":"            self.reserved_share_extend_percentage)"},{"line_number":350,"context_line":"        stats_dict[\u0027max_over_subscription_ratio\u0027] \u003d ("},{"line_number":351,"context_line":"            self.max_over_subscription_ratio)"},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"        spaces \u003d self._isilon_api.get_space_stats()"},{"line_number":354,"context_line":"        if spaces:"},{"line_number":355,"context_line":"            stats_dict[\u0027total_capacity_gb\u0027] \u003d spaces[\u0027total\u0027] // units.Gi"}],"source_content_type":"text/x-python","patch_set":6,"id":"86939bdd_967d634f","line":352,"updated":"2025-07-31 22:06:12.000000000","message":"manila\u0027s scheduler is \"pool-aware\" - this means that a backend storage system can represent multiple pools of storage. Does the Powerscale back end support multiple storage pools? \n\nFor example, the unity driver supports pools: https://opendev.org/openstack/manila/src/commit/dc1320c963651fae770d1d057a7923219e8df579/manila/share/drivers/dell_emc/plugins/unity/connection.py#L583-L607\n\nif the driver doesn\u0027t support pools, you can avoid the scheduler having to wrangle things by sending a pool structure setting exactly one pool, like the way it is done in the powerflex driver:\n\nhttps://opendev.org/openstack/manila/src/commit/dc1320c963651fae770d1d057a7923219e8df579/manila/share/drivers/dell_emc/plugins/powerflex/connection.py#L353-L381\n\n\nIt\u0027s best practice to stick to the newer data structure that the scheduler expects: https://docs.openstack.org/manila/latest/contributor/pool-aware-manila-scheduler.html#developer-impact","commit_id":"afd2c3b67339ee6a0ef081b1e305a44135082a0b"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"82f300972f1cbdbf977af39165c8abda7f2afd07","unresolved":false,"context_lines":[{"line_number":349,"context_line":"            self.reserved_share_extend_percentage)"},{"line_number":350,"context_line":"        stats_dict[\u0027max_over_subscription_ratio\u0027] \u003d ("},{"line_number":351,"context_line":"            self.max_over_subscription_ratio)"},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"        spaces \u003d self._isilon_api.get_space_stats()"},{"line_number":354,"context_line":"        if spaces:"},{"line_number":355,"context_line":"            stats_dict[\u0027total_capacity_gb\u0027] \u003d spaces[\u0027total\u0027] // units.Gi"}],"source_content_type":"text/x-python","patch_set":6,"id":"880ad54f_b75546b1","line":352,"in_reply_to":"86939bdd_967d634f","updated":"2025-08-02 07:09:35.000000000","message":"PowerScale does not support pools.\nReported stats with one pool structure and adapted to the newer data structure.","commit_id":"afd2c3b67339ee6a0ef081b1e305a44135082a0b"}],"manila/tests/share/drivers/dell_emc/plugins/isilon/test_isilon.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"a571eb9345e2c0c8ff0e122d46ce0a1e20ab9272","unresolved":true,"context_lines":[{"line_number":528,"context_line":"            \u0027total_capacity_gb\u0027: 1000,"},{"line_number":529,"context_line":"            \u0027free_capacity_gb\u0027: 100,"},{"line_number":530,"context_line":"            \u0027allocated_capacity_gb\u0027: 1,"},{"line_number":531,"context_line":"            \u0027qos\u0027: False"},{"line_number":532,"context_line":"        }"},{"line_number":533,"context_line":"        expected_stats \u003d {"},{"line_number":534,"context_line":"            \u0027share_backend_name\u0027: \u0027PowerScale_backend\u0027,"}],"source_content_type":"text/x-python","patch_set":7,"id":"9230ddeb_b4810bd4","line":531,"range":{"start_line":531,"start_character":19,"end_line":531,"end_character":24},"updated":"2025-08-07 14:31:51.000000000","message":"nit: add a trailing comma","commit_id":"28c30b5065db0d6fa95f8063f4ffe6d9051d428a"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"a571eb9345e2c0c8ff0e122d46ce0a1e20ab9272","unresolved":true,"context_lines":[{"line_number":534,"context_line":"            \u0027share_backend_name\u0027: \u0027PowerScale_backend\u0027,"},{"line_number":535,"context_line":"            \u0027driver_version\u0027: isilon.VERSION,"},{"line_number":536,"context_line":"            \u0027storage_protocol\u0027: \u0027NFS_CIFS\u0027,"},{"line_number":537,"context_line":"            \u0027pools\u0027: [expected_pool_stats]"},{"line_number":538,"context_line":"        }"},{"line_number":539,"context_line":"        self.assertEqual(expected_stats, stats_dict)"},{"line_number":540,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"873f4d92_88901570","line":537,"range":{"start_line":537,"start_character":41,"end_line":537,"end_character":42},"updated":"2025-08-07 14:31:51.000000000","message":"nit: add a trailing comma","commit_id":"28c30b5065db0d6fa95f8063f4ffe6d9051d428a"}]}