)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"fe661e90ce7ff3141b8dc85d055985f8b4b192c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"90e2960b_c38ec356","updated":"2024-06-05 10:41:37.000000000","message":"Few pointers to begin with:\n1] The \"CeaStor_CI\" should be listed in below page:\nhttps://wiki.openstack.org/wiki/ThirdPartySystems\n\nAlso the CI should pass on this patch.\n\n2] A documentation file (.rst) should be added:\ndoc/source/configuration/block-storage/drivers/Ceastor.rst\n\n3] A release note (.yaml) should be added:\nreleasenotes/notes/Ceastor-driver.yaml\n\n4] Below files would have to be updated:\ncinder/opts.py\ndoc/source/reference/support-matrix.ini\n\nBelow code review link can be used as reference:\nAdd Cinder driver for TOYOU ACS5000\nhttps://review.opendev.org/c/openstack/cinder/+/767290","commit_id":"8c8f92f3234abfcd9f91729a5a33051011b0c18d"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"1cd580a8f81db134713c181d3c13c7786d694930","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ec1c6f4c_3a505bb5","in_reply_to":"42f7be7e_ff426522","updated":"2024-06-19 03:29:02.000000000","message":"Done","commit_id":"8c8f92f3234abfcd9f91729a5a33051011b0c18d"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"c6c229bc324269411d0d7d9ef407913d6da6d79b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"bfd3df4b_94f1ed4e","in_reply_to":"90e2960b_c38ec356","updated":"2024-06-14 06:49:05.000000000","message":"Please, we have an internal CI system that ensures the project\u0027s operation. Do I still need to integrate the company\u0027s CI system with the OpenStack Gerrit system? Is this necessary?","commit_id":"8c8f92f3234abfcd9f91729a5a33051011b0c18d"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"9b2d8f573d703ebe6486e3eef40ee5e78351b292","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":6,"id":"42f7be7e_ff426522","in_reply_to":"bfd3df4b_94f1ed4e","updated":"2024-06-14 06:54:37.000000000","message":"I have already added \"CeaStor CI\" to the ThirdPartySystems (https://wiki.openstack.org/wiki/ThirdPartySystems) list.","commit_id":"8c8f92f3234abfcd9f91729a5a33051011b0c18d"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"9eba36bdf3f8e42baa0d728d87258f47b90ef33c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"b766a531_5611c20d","updated":"2024-06-07 03:08:55.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"03443330b8f00b21e908f4c246b9dffd70abb80a"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d301d42823cf67ce1e1e447e6bdc91942f4cc8ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"a644c211_6c58c74c","updated":"2024-06-13 12:10:34.000000000","message":"recheck docs failed due to missing Crypto module","commit_id":"3c95a26e0a6e11bd13bd13602af38decfc9f930c"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"28cef15c193f3062f4eb64076073aeae5a4faa3d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"40097502_5458585b","updated":"2024-06-19 18:32:11.000000000","message":"recheck","commit_id":"3cf9381864dc639a55ac0f9a331505642146474f"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"4e2cb4193e2b019bb22e85ad5ed48ed061e99af0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"1d7f8334_cd4500a9","updated":"2024-06-19 18:26:41.000000000","message":"recheck tempest","commit_id":"3cf9381864dc639a55ac0f9a331505642146474f"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"4187f88bc7a867fbdcae1054d25e543d56500648","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"d589ebff_1aab11b1","updated":"2024-07-01 06:50:23.000000000","message":"1] Unit test (UT) code can be added at: cinder/tests/unit/volume/drivers\n2] Minor comments inline.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"77f025124189a9f9d32df93e55c88a770a489ebe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"2b62f82e_bdb6f22b","updated":"2024-06-24 09:26:14.000000000","message":"Please review this MR.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"2c163424ec11ddc718aea95c575a5ca7e0febc36","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"a2747474_badb02d0","updated":"2024-06-26 14:37:43.000000000","message":"Thank you for submitting this.  Our checklist is https://docs.openstack.org/cinder/latest/contributor/new_driver_checklist.html\n\nWe need 3rd party CI coverage to verify correct execution, is this something you can provide during this cycle?","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"17385ed5652727c90ddc1d532232c377d9c5e13c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"52633b8d_f6c0f97e","updated":"2024-06-20 12:30:36.000000000","message":"recheck","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"18fd121506c443e75c22b4caec69f3cea3b4b8e8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"d4241043_34f890ec","updated":"2024-07-16 02:45:45.000000000","message":"run ceastor-ci","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"0fca202eb187035da563de7c87b3a1fb655c42f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"e6828708_b46fceeb","updated":"2024-07-15 06:50:47.000000000","message":"run ceastor-ci","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"5110d992f3967d1b3abee3e17342f31622fabb24","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"e5da1c06_f19b4656","in_reply_to":"a2747474_badb02d0","updated":"2024-06-26 14:40:07.000000000","message":"And the wiki is https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver  We need CI by milestone 2, which is coming up soon.  See https://releases.openstack.org/dalmatian/schedule.html for the release schedule.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"348c16797e7150d45fe657f66b82d430a71cb5cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"93684930_a2001c2c","in_reply_to":"c65e17fe_db8e3a81","updated":"2024-07-31 06:11:11.000000000","message":"Thank you for pointing out these issues. I have made the necessary changes. Please continue with the review.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"3ab937043066df4b7e90aefc35edef268ea7c933","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"c65e17fe_db8e3a81","in_reply_to":"e5da1c06_f19b4656","updated":"2024-07-25 07:57:48.000000000","message":"Thank you, it can be implemented within a week.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"023887a4955e36920193ef207aaa931eeae44b6c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"f1d35f40_dc521dac","updated":"2024-07-16 03:26:53.000000000","message":"recheck","commit_id":"43476cdb2c266e1f6d8f8d9b6806d5be7b049976"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"ec23eab000dc54edd9d46b625bee8a0565bed859","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"4f9abb11_027f7754","updated":"2024-07-16 03:24:17.000000000","message":"rujn ceastor_cinder_ci","commit_id":"43476cdb2c266e1f6d8f8d9b6806d5be7b049976"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"c3b47ed15a20bf0d707de8c691e286bdae5ab4ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"1b897b61_807a4fcd","updated":"2024-07-16 03:34:20.000000000","message":"run ceastor-ci","commit_id":"43476cdb2c266e1f6d8f8d9b6806d5be7b049976"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"90c99dda50cf8d3807e00a6d1f932c2881c1e2b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"4b4c1dc6_2323b97d","updated":"2024-07-16 03:34:03.000000000","message":"run ceastor-ci","commit_id":"43476cdb2c266e1f6d8f8d9b6806d5be7b049976"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"4d59861d8ec5ce7f5d79c5724596946cad454cb5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"54cc0ca4_02432db6","updated":"2024-07-16 03:30:46.000000000","message":"run ceastor-ci","commit_id":"43476cdb2c266e1f6d8f8d9b6806d5be7b049976"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"210614ee3c6ef140402de6424d48335453578140","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"76b0fe60_a287eb00","updated":"2024-07-16 03:25:42.000000000","message":"run ceastor-ci","commit_id":"43476cdb2c266e1f6d8f8d9b6806d5be7b049976"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"4493a9906096765830d696be36aca5974f47a91f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"d192eadf_df449a06","updated":"2024-07-16 09:05:19.000000000","message":"run ceastor-ci","commit_id":"43476cdb2c266e1f6d8f8d9b6806d5be7b049976"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"0af1a85159d4c466b778f5b1fead827402f16914","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"daca8ddc_0738879d","updated":"2024-07-16 03:31:28.000000000","message":"run ceastor-ci","commit_id":"43476cdb2c266e1f6d8f8d9b6806d5be7b049976"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"fe578bdf71aa81be12668adfd7da7f40e89eb417","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"dee9f7e3_0545c3e6","updated":"2024-07-16 03:41:09.000000000","message":"run ceastor-ci","commit_id":"43476cdb2c266e1f6d8f8d9b6806d5be7b049976"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"b2af433cfa352330ad121c31c2bda8c0f5111af3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":29,"id":"d5ef8ed2_8374583a","updated":"2024-07-16 03:24:34.000000000","message":"run ceastor_cinder_ci","commit_id":"43476cdb2c266e1f6d8f8d9b6806d5be7b049976"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"1c6c5483cfa5f54acb1818600cd135c38e10fd2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":30,"id":"561a4be6_3b9fa269","updated":"2024-07-25 07:52:33.000000000","message":"please check the mr","commit_id":"7a35388166b768c32804f6a8775231454320e291"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"3ab937043066df4b7e90aefc35edef268ea7c933","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":30,"id":"2ede1abe_dcec55f3","updated":"2024-07-25 07:57:48.000000000","message":"run ceastor-ci","commit_id":"7a35388166b768c32804f6a8775231454320e291"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d09740c34bad1c255b4d742be47ca3c4744972e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":30,"id":"f752628b_faf33546","updated":"2024-07-25 12:53:12.000000000","message":"run ceastor-ci","commit_id":"7a35388166b768c32804f6a8775231454320e291"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"a7fba13aa173f7e9baedcabef0640f03f40c252d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"50f6c418_03d60939","updated":"2024-07-26 04:02:16.000000000","message":"run ceastor-ci","commit_id":"011ea8117159d3d7f16d218bf8ee538106a795af"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"f15dffc8782379893cdb99a714e72cb505b16246","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"908fab13_1178dbff","updated":"2024-07-26 02:53:37.000000000","message":"run ceastor-ci","commit_id":"011ea8117159d3d7f16d218bf8ee538106a795af"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"e97098a11142bf9c04ee1b337dec62cb0f7a14ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"ac07aa56_b52bbfb5","updated":"2024-07-26 02:57:59.000000000","message":"run ceastor-ci","commit_id":"011ea8117159d3d7f16d218bf8ee538106a795af"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"ae5eb0ed93e97a09c9d2d909fe9788c7fcad7929","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"d1b16e69_d17847af","updated":"2024-07-26 02:22:25.000000000","message":"run ceastor-ci","commit_id":"011ea8117159d3d7f16d218bf8ee538106a795af"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"f92cc67474913c9cecbc3d6369405af5a41e6d4f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"e0f47793_e9f7776a","updated":"2024-07-26 03:59:05.000000000","message":"run ceastor-ci","commit_id":"011ea8117159d3d7f16d218bf8ee538106a795af"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"f4a6972fd70fa0e6e0f04c45af0eb59ee8441f40","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"e4fe8c3c_be079d4e","updated":"2024-07-26 03:38:37.000000000","message":"run ceastor-ci","commit_id":"011ea8117159d3d7f16d218bf8ee538106a795af"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"74155f873e46f75f2d72f32ba60cc50186c2bcbf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"f665d5dd_dbff94c0","updated":"2024-07-26 03:54:59.000000000","message":"run ceastor-ci","commit_id":"011ea8117159d3d7f16d218bf8ee538106a795af"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"a1c698ba54eb60644b5e1150c29fac1e4263ae93","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"fe8fc6a6_128e0145","updated":"2024-07-26 04:00:46.000000000","message":"run ceastor-ci","commit_id":"011ea8117159d3d7f16d218bf8ee538106a795af"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"05a550377c268326d00819e30d021ca75d46a7fb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"11233784_57469f1c","updated":"2024-07-26 06:00:59.000000000","message":"run ceastor-ci","commit_id":"dd85281eb1789d9757fdf165d7cfc0742d87ce58"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"aa5f7a97019a92e34ea412f758967f5d000d6a7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"299d806b_95ca4594","updated":"2024-07-26 05:57:04.000000000","message":"run ceastor-ci","commit_id":"dd85281eb1789d9757fdf165d7cfc0742d87ce58"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"0a2a01c16e2fcb2c0feba4e88de06e6c40cd497b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"8bf9e54f_88031e80","updated":"2024-07-26 06:04:45.000000000","message":"run ceastor-ci","commit_id":"dd85281eb1789d9757fdf165d7cfc0742d87ce58"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"e575085e2d958c1d0400b46cbf4a5222da081dfe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"9ceffc5c_33c1ccb6","updated":"2024-07-26 06:02:29.000000000","message":"run ceastor-ci","commit_id":"dd85281eb1789d9757fdf165d7cfc0742d87ce58"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"7036e11322e67355d84611e4468882a36aeec940","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":33,"id":"e0b762a0_2eedb6b1","updated":"2024-07-26 05:58:34.000000000","message":"run ceastor-ci","commit_id":"dd85281eb1789d9757fdf165d7cfc0742d87ce58"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"3f25832d8db859cd73cdaf15d497280bc6c11ff4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"69576646_1e6ae641","updated":"2024-07-26 14:52:45.000000000","message":"Third party CI needs to fix the format of how the result is posted. See details in https://docs.opendev.org/opendev/system-config/latest/third_party.html.\n\nAlso linked results are not accessible and currently pointing to a localhost URL.","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"f9e9c86a72c1231a87e99fe03ebe1f51a14c96f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"32f5dad3_4e83e4b8","updated":"2024-07-29 02:09:07.000000000","message":"recheck","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d4f5bf9f0c46c7c711127024bce9bb67a2dc8425","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"e984fda6_cb1bb51b","updated":"2024-07-29 06:37:48.000000000","message":"recheck","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"b2a30afd02fdcc09039eda2d4466d1f9856190af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"35110b39_3bf80fd8","updated":"2024-07-29 07:20:27.000000000","message":"run ceastor-ci","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"591d06e78641a2532a41aeaf8f2ec70945e78091","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"428dae10_6a29caa3","updated":"2024-07-29 06:59:44.000000000","message":"run ceastor-ci","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"9103b92b724402da17d5039ef78817c81eb6a3b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"4ef7f1f5_35912c7c","updated":"2024-07-29 07:13:38.000000000","message":"run ceastor-ci","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"8f4ced6a6871aaf46559321f3d162c6f3d1c7f96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"9f6174c0_205ed2c6","updated":"2024-07-29 06:54:05.000000000","message":"run ceastor-ci","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"23b9d3df3cd4386f784786f834f8803152b164a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"a4842a08_2f1518a6","updated":"2024-07-29 06:49:07.000000000","message":"run ceastor-ci","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"b408030b1873d0d85c967cef1129cd7222b7fc52","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"e524fc70_93704224","updated":"2024-07-29 07:07:16.000000000","message":"run ceastor-ci","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"afa60656c7d2a1affa01debeb16d50dd9d227f90","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"ee2de561_357d30ef","updated":"2024-07-29 06:56:57.000000000","message":"run ceastor-ci","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"fa40d7436055a7388683bdcad530dc8266aec9e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":34,"id":"a386fba3_29ff32cc","in_reply_to":"69576646_1e6ae641","updated":"2024-07-31 06:09:19.000000000","message":"Thank you, I have been debugging, and it is now complete.","commit_id":"fcb5a56640b485a0088fe6b11337044143bd4e25"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"2d9979ca5bdf1d2aa9ff710f9a672e1f2c03af39","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":35,"id":"418888d3_a0f3e17c","updated":"2024-07-29 08:19:34.000000000","message":"run ceastor-ci","commit_id":"509d27e24f1ea8d86fd77c64474b4b2e4eb32767"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"e10b5cddb84c2c622d591882fb3e4598cb8bc9c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":35,"id":"4f3b072b_04aaee05","updated":"2024-07-29 08:46:21.000000000","message":"run ceastor-ci","commit_id":"509d27e24f1ea8d86fd77c64474b4b2e4eb32767"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"3b785d573a1ac90cd1a3a66b0771da7c45a9a4dc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":35,"id":"5c532f3e_0150b3f8","updated":"2024-07-29 07:26:18.000000000","message":"run ceastor-ci","commit_id":"509d27e24f1ea8d86fd77c64474b4b2e4eb32767"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"783b51d1c5a3beedd6a95cb329be456e27f84ec3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":35,"id":"713c0d28_c71643b4","updated":"2024-07-29 08:16:17.000000000","message":"run ceastor-ci","commit_id":"509d27e24f1ea8d86fd77c64474b4b2e4eb32767"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"52671cfe8e8fe4627f54ae36d1440d83581b05c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":35,"id":"8427557c_237e165f","updated":"2024-07-29 08:29:03.000000000","message":"run ceastor-ci","commit_id":"509d27e24f1ea8d86fd77c64474b4b2e4eb32767"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"5d36f8b8f2b0f620ad892a3d955c08a28db67204","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":35,"id":"b6b6d67a_1d735281","updated":"2024-07-29 08:30:16.000000000","message":"run ceastor-ci","commit_id":"509d27e24f1ea8d86fd77c64474b4b2e4eb32767"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"cff6fc821b112a8323647d0c2cb02df32db88230","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":35,"id":"cdee1ad3_428781d7","updated":"2024-07-29 07:47:36.000000000","message":"run ceastor-ci","commit_id":"509d27e24f1ea8d86fd77c64474b4b2e4eb32767"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"980025fb0f71a98b28393c1ca9d339ac5eb2b144","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"05027b4f_261ffc9c","updated":"2024-07-31 01:54:59.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"78d4447e53bccf3ef686e27c568dd71a90f823b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"098d7b21_1d892f6d","updated":"2024-07-31 03:47:56.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d4b9517134ba3bad352a5fdd6bfa5377fca2ba5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"186de604_868d4384","updated":"2024-07-29 12:02:56.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"c36f83fb905009502ca332698ac39a17930c1c73","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"1adc8bee_3b896f63","updated":"2024-07-29 13:04:47.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"09e0ffc30c4d5146d2c42fe02782edf14ad95b5c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"1c35bd41_63accda0","updated":"2024-07-31 04:11:58.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"30f5471387664187c25e124b501b2975b64e2175","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"1c4732e0_42b70f30","updated":"2024-07-29 11:35:57.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d3af5c895bafac5fcc67a9aaf0c922da1bf206fe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"26dcfd94_afc0c26c","updated":"2024-07-31 03:34:59.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"429508f665463697e884a86331198aba3476c473","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"2e7395fa_1db8f9dc","updated":"2024-07-31 03:13:16.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d59b59e2e880695ef92e2aa7db8270bc0bcbb422","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"2f6c4a72_b1304fb8","updated":"2024-07-31 04:12:46.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d357ec5ed9296f1e8331f2ae90220d97aa0fcddd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"32c219fb_df0c47e6","updated":"2024-07-29 11:48:30.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"3eae76fbd6a83cd1101bda405b62674952f5178a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"3ce97583_28123b41","updated":"2024-07-31 02:44:39.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"1d54b4276e275e43d32aa1ad47496f6cbb238e21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"521411f8_a1e50cee","updated":"2024-07-29 12:08:58.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"ccf352f819a1b3719c2da0fa050b635c94d4a26e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"54e53f12_81ec3c48","updated":"2024-07-29 11:59:28.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"f4a8c9f55e595465ffbd7873b0faf60381a1e6c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"576179ff_b61e7f15","updated":"2024-07-31 03:17:51.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"0373cbbcb6d64dd27d3c0cf86d4a685c8eccdc78","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"5ca34f4f_67314257","updated":"2024-07-29 12:12:55.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"3b8006ddb461f4d732ed5fca722fd5fe347a616d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"654dee4b_f86d248a","updated":"2024-07-31 03:37:21.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"11d10c412cf0de5e9b5c11eedc9c82c4e3096788","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"66b2e540_0f055358","updated":"2024-07-29 13:00:29.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"fabb10c4ce8d7cb86ceeec7747c1815c3999a285","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"6aaaeabb_d6959e03","updated":"2024-07-31 05:38:05.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"f5892dba08aef511c6c44cab673b8f1cb483a5ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"70d17951_1c1ff741","updated":"2024-07-31 01:35:48.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"e1de7a4683ea8b091d2ae3ee49b2f1b028c111d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"74d51b10_05e98773","updated":"2024-07-31 04:13:33.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"8b83b3cc31360c2b15f71b0b0a6029d2c699d9a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"77d7392e_ab1e47fb","updated":"2024-07-29 12:16:40.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"afccb77feec851fa3718c51f29e078cf517a02dd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"7a653a61_5a2aa0b1","updated":"2024-07-31 01:39:18.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"97f74297951c25d156e4c56ddb11d8d50bf9a6e3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"7a9b5b0b_8938ea68","updated":"2024-07-29 11:52:44.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"c18d978748d87c0109d0a98e720e9b8912f8ac7a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"7df4a919_df1d8764","updated":"2024-07-29 11:39:47.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"4d257860979e1f9ed0826b31b9752c87c1a77e34","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"80d6ff69_de4615d9","updated":"2024-07-31 02:19:53.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"64fc1493c0c0a5ab58ea82bde3839bebc73fa1b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"81bb2157_4f29856b","updated":"2024-07-31 03:31:59.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"b4a20d64d57b2aa3fe3d8da62e7975ee14a27ff0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"8678e1b5_3e3e55b3","updated":"2024-07-31 03:47:01.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"34591190bac2f21e38e81148ad5e26d2268197c5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"99dd4804_c4c5fbf4","updated":"2024-07-31 06:02:53.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"0b396798d88078a2e8649ce2455f1034e52e0b74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"9ac99a65_6e7bb3f0","updated":"2024-07-31 03:42:37.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"436ad5a2d9e400feb5500a0c0625862c679d2f00","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"9ed886b3_89bfd36f","updated":"2024-07-29 12:42:25.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"8d9b424bc2131e26c501777ee0a74ba7ad8fc9e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"a4f9d904_2dd496d5","updated":"2024-07-29 12:55:50.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"4dca430c1a51fca6011e9adbebc07c3283e9c4a5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"a5247032_3a3fda78","updated":"2024-07-31 03:33:36.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"e0fbf7b97f118ede3096fbe9ca93475d76ce3aac","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"a67de204_b75bb3b8","updated":"2024-07-29 11:52:07.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"f61406caa59fe50c08a76e360d3ce4ef359260a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"b96a5b67_e15e172d","updated":"2024-07-31 03:27:39.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"4679387e82907966addaabe88ab079fb2c6148f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"bcb789e3_f60406e5","updated":"2024-07-31 01:51:51.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"f7fb840d9078093d2e963bd8a521cc990f1085ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"c2596dba_029b6764","updated":"2024-07-29 12:38:49.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"b2383bcd1c934a493b2645ddb6ac4d71835af407","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"c39a6209_38e9efca","updated":"2024-07-29 12:50:44.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"2ff48babdf426556df85927c85f1156bf614f250","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"c3b28849_45e46aa6","updated":"2024-07-29 13:07:22.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"4882c2820d00a5ad33bdb857978aa7015f37b41e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"c40cbf66_83dfae20","updated":"2024-07-29 12:54:20.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"e704b8d920eed2dc17b7bf6ace027614294b2e0c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"cf9d7c0e_78100a3c","updated":"2024-07-31 03:13:46.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"36f12e8eec7c93c4e42b2df11d4fe8f0624412ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"d0f7b785_43f07336","updated":"2024-07-31 02:37:10.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"65c1e289c2d40dd25a00a12b35709512484ae35e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"daa11f5a_ed6827c6","updated":"2024-07-31 03:40:12.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"e1cc45b735a7761106d783e70f9db111840bda3c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"e0db98dc_0cea2e2a","updated":"2024-07-31 03:49:30.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"fdd05b3b8d53b2313cba03d91b3b4d01a4f45927","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"ecfece66_b2fabb22","updated":"2024-07-31 01:36:14.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"946e416dd76ab536eba5b349f040c90beb61df8b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"f55fd19a_085a7f05","updated":"2024-07-29 11:51:34.000000000","message":"run ceastor-ci","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"a422aebc6916a6f541c44474f73c56552c8809ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"27076dc7_05ab126c","updated":"2024-07-31 09:22:48.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"06b3bbc82c33ffeeb2b4de5f6437fce9b8d01e19","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"28782471_fc0364fd","updated":"2024-08-02 02:46:32.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"5d7f493128bee69c63db2f1cb4fd65e652988c7f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"2ab1931d_b3e658ab","updated":"2024-07-31 12:59:57.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"daf5a4a447139417e2972d2eed20245c4ccf3ed3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"318e0ad3_5b2da692","updated":"2024-08-01 01:50:54.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"db02402d827cc31ffc575d43cfe9406200a78d5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"3e2bdb28_c760c3ee","updated":"2024-08-01 01:54:55.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"194040b6edd0512a12d8f7a5db927abcdf7f3888","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"63d26b9c_4d1f2881","updated":"2024-08-02 02:09:47.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"272f9e0d205dc5f8fe4e3323e2fce4c3e48dc6c2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"72b598d7_e689527b","updated":"2024-07-31 14:40:22.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"b9a68ba43226f29c6738b4d9d808c83129e32bd4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"792d53fa_d0dfcc94","updated":"2024-07-31 08:29:21.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"9710b709e87efa57b1117d5ccb318a86dee39e09","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"7f3ed687_21114e33","updated":"2024-08-02 05:54:18.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"6fe36724a6be757c422be87f04b724498a07f36c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"8d559352_b2871af2","updated":"2024-08-01 01:57:22.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"795587cae86eece2d7269fb9ccca8ff26e9e7aba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"8d9cbd9f_4de86235","updated":"2024-08-02 03:00:58.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"8e5712d1c5914e1c08aed6b6c3e522d1c534af92","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"adc4f8f2_2bc29ec5","updated":"2024-08-02 02:22:33.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"f85405c270ef9e780a1f230444efb5275c520281","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"adc74767_e7e09946","updated":"2024-07-31 09:25:37.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"2587a3431cd7542cf893f43e2c80279cdc30d693","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"b20ca98e_23a7bc56","updated":"2024-08-01 03:44:31.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"0fa07a31f21806bab210a361f7020e46a44cf10a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"be068908_9de9b3d1","updated":"2024-08-01 03:36:27.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"8ed6d12b414d36d8fb08870e8d239d0838ef0212","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"ca7c512c_5aea2066","updated":"2024-08-01 03:21:49.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"426cbc451e7f1e991cae91f8c8d7b32c9699c01c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"e929e798_6f7060cb","updated":"2024-07-31 13:06:06.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"12be3800431a9f3c80475142288d270e655c59cc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"f5fd971c_91f03235","updated":"2024-08-02 06:05:47.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"8d5efe0887e41442029308e0970bceb4f12d90db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":36,"id":"fffde724_7217fcc2","updated":"2024-07-31 06:06:21.000000000","message":"run-CeaStor CI","commit_id":"6a87714c8c8da23a1536c065a23c24233c5b5d75"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"5267b3a87d4f536593903146e7f621df51e0a977","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":37,"id":"f68fc9e9_d6b6d0dc","updated":"2024-08-27 09:44:19.000000000","message":"Also CI reporting is not in correct format. And while not strictly required, I would highly recommend not using GitHub for your static file serving.","commit_id":"217adcb456df4e55888ec5ae3ee361388153dd7a"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"6413071587b423e2c1c3d67e89ef75bf5fef822b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":37,"id":"7dd2f8ca_0c1c36c3","updated":"2024-08-16 12:17:13.000000000","message":"I don\u0027t see a passing CI run for this driver.  The last one posted is on August 2nd and it failed.","commit_id":"217adcb456df4e55888ec5ae3ee361388153dd7a"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"b17d3ad3b102a7171053558663c46434ec0086a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":37,"id":"9d662d01_8f321b87","updated":"2024-10-28 14:06:11.000000000","message":"This is for test.","commit_id":"217adcb456df4e55888ec5ae3ee361388153dd7a"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"6783e6c9d3a46c32387260e8a5352e42998a4fc5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":37,"id":"b2e880c2_af7457a3","updated":"2024-08-13 06:13:14.000000000","message":"please review the MR.","commit_id":"217adcb456df4e55888ec5ae3ee361388153dd7a"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"3e246a87fae7ebcc03de452097fb2dd8918fcaaa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":37,"id":"3ff800d6_881cb7a3","in_reply_to":"7dd2f8ca_0c1c36c3","updated":"2024-09-24 07:05:08.000000000","message":"Could you please review the code first? There are still some issues being rectified on the CI side.","commit_id":"217adcb456df4e55888ec5ae3ee361388153dd7a"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"3e246a87fae7ebcc03de452097fb2dd8918fcaaa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":37,"id":"daae3feb_76220ea5","in_reply_to":"f68fc9e9_d6b6d0dc","updated":"2024-09-24 07:05:08.000000000","message":"Could you please specify where the format is incorrect? The company can transition to a static server later; for now, we are using GitHub for file hosting.","commit_id":"217adcb456df4e55888ec5ae3ee361388153dd7a"}],"cinder/volume/drivers/ceastor/ceastor_client.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# Copyright (c) 2022 CESTC Technologies Co., Ltd."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":27,"id":"e4fbd5b4_510ff8d8","line":1,"updated":"2024-07-06 12:50:39.000000000","message":"2024?","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright (c) 2022 CESTC Technologies Co., Ltd."},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-python","patch_set":27,"id":"aa9e2cb0_d2f1e865","line":1,"in_reply_to":"e4fbd5b4_510ff8d8","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":27,"context_line":"from cryptography.hazmat.primitives import serialization"},{"line_number":28,"context_line":"from oslo_log import log as logging"},{"line_number":29,"context_line":"import requests"},{"line_number":30,"context_line":"import six"},{"line_number":31,"context_line":"import urllib3"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"from cinder import exception"}],"source_content_type":"text/x-python","patch_set":27,"id":"0b2ea196_68444a1d","line":30,"updated":"2024-07-06 12:50:39.000000000","message":"Six should not be used. That is only needed for py2/py3 support, but OpenStack has only supported py3 for years now and py2 is long end of life.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":27,"context_line":"from cryptography.hazmat.primitives import serialization"},{"line_number":28,"context_line":"from oslo_log import log as logging"},{"line_number":29,"context_line":"import requests"},{"line_number":30,"context_line":"import six"},{"line_number":31,"context_line":"import urllib3"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"from cinder import exception"}],"source_content_type":"text/x-python","patch_set":27,"id":"46d1452d_37be6813","line":30,"in_reply_to":"0b2ea196_68444a1d","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":69,"context_line":"            try:"},{"line_number":70,"context_line":"                self.login()"},{"line_number":71,"context_line":"            except Exception as e:"},{"line_number":72,"context_line":"                LOG.error(\"Login failed, attempt %d: %s\", x + 1, str(e))"},{"line_number":73,"context_line":"                time.sleep(1)"},{"line_number":74,"context_line":"                continue"},{"line_number":75,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":27,"id":"f93744af_a2b0719c","line":72,"range":{"start_line":72,"start_character":65,"end_line":72,"end_character":67},"updated":"2024-07-06 12:50:39.000000000","message":"No need to explicitly covert to string since %s is already doing that.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":69,"context_line":"            try:"},{"line_number":70,"context_line":"                self.login()"},{"line_number":71,"context_line":"            except Exception as e:"},{"line_number":72,"context_line":"                LOG.error(\"Login failed, attempt %d: %s\", x + 1, str(e))"},{"line_number":73,"context_line":"                time.sleep(1)"},{"line_number":74,"context_line":"                continue"},{"line_number":75,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":27,"id":"25b98d6c_fb854d74","line":72,"range":{"start_line":72,"start_character":65,"end_line":72,"end_character":67},"in_reply_to":"f93744af_a2b0719c","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":70,"context_line":"                self.login()"},{"line_number":71,"context_line":"            except Exception as e:"},{"line_number":72,"context_line":"                LOG.error(\"Login failed, attempt %d: %s\", x + 1, str(e))"},{"line_number":73,"context_line":"                time.sleep(1)"},{"line_number":74,"context_line":"                continue"},{"line_number":75,"context_line":"            else:"},{"line_number":76,"context_line":"                flag \u003d False"}],"source_content_type":"text/x-python","patch_set":27,"id":"21ebf394_75f3a895","line":73,"updated":"2024-07-06 12:50:39.000000000","message":"Where is there a sleep statement in an init call?","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":70,"context_line":"                self.login()"},{"line_number":71,"context_line":"            except Exception as e:"},{"line_number":72,"context_line":"                LOG.error(\"Login failed, attempt %d: %s\", x + 1, str(e))"},{"line_number":73,"context_line":"                time.sleep(1)"},{"line_number":74,"context_line":"                continue"},{"line_number":75,"context_line":"            else:"},{"line_number":76,"context_line":"                flag \u003d False"}],"source_content_type":"text/x-python","patch_set":27,"id":"cabaa98d_e061f028","line":73,"in_reply_to":"21ebf394_75f3a895","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":96,"context_line":"            \"Referer\": constants.REFERER_URL.format(vip\u003dself.address),"},{"line_number":97,"context_line":"            \"X-CSRFToken\": self.token"},{"line_number":98,"context_line":"        })"},{"line_number":99,"context_line":"        self.session.verify \u003d False"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    def call(self, url: str,"},{"line_number":102,"context_line":"             method: str,"}],"source_content_type":"text/x-python","patch_set":27,"id":"950b5bde_56a89dd9","line":99,"updated":"2024-07-06 12:50:39.000000000","message":"This should be controlled by the existing config option.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":96,"context_line":"            \"Referer\": constants.REFERER_URL.format(vip\u003dself.address),"},{"line_number":97,"context_line":"            \"X-CSRFToken\": self.token"},{"line_number":98,"context_line":"        })"},{"line_number":99,"context_line":"        self.session.verify \u003d False"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    def call(self, url: str,"},{"line_number":102,"context_line":"             method: str,"}],"source_content_type":"text/x-python","patch_set":27,"id":"eaeb8e98_0cc08b09","line":99,"in_reply_to":"950b5bde_56a89dd9","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":121,"context_line":"            call_url \u003d constants.BASE_URL.format(vip\u003dself.address, url\u003durl)"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"        if not filter_flag:"},{"line_number":124,"context_line":"            LOG.info(\"\u003d\u003d\u003d\u003d\u003d\u003d %(method)s %(url)s %(param)s\", {"},{"line_number":125,"context_line":"                \"method\": method, \"url\": url, \"param\": data"},{"line_number":126,"context_line":"            })"},{"line_number":127,"context_line":"        func \u003d getattr(self.session, method.lower())"}],"source_content_type":"text/x-python","patch_set":27,"id":"ae2d85fa_0a7d9091","line":124,"updated":"2024-07-06 12:50:39.000000000","message":"This looks more like debug level code. Probably better to use the trace decorator.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":121,"context_line":"            call_url \u003d constants.BASE_URL.format(vip\u003dself.address, url\u003durl)"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"        if not filter_flag:"},{"line_number":124,"context_line":"            LOG.info(\"\u003d\u003d\u003d\u003d\u003d\u003d %(method)s %(url)s %(param)s\", {"},{"line_number":125,"context_line":"                \"method\": method, \"url\": url, \"param\": data"},{"line_number":126,"context_line":"            })"},{"line_number":127,"context_line":"        func \u003d getattr(self.session, method.lower())"}],"source_content_type":"text/x-python","patch_set":27,"id":"4cc896e7_6ad3a83d","line":124,"in_reply_to":"ae2d85fa_0a7d9091","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"}],"cinder/volume/drivers/ceastor/driver.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":36,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"volume_opts \u003d ["},{"line_number":39,"context_line":"    cfg.StrOpt(\u0027ceastor_ip\u0027,"},{"line_number":40,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":41,"context_line":"               help\u003d\u0027IP address of CeaStor\u0027),"},{"line_number":42,"context_line":"    cfg.StrOpt(\u0027ceastor_username\u0027,"},{"line_number":43,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":44,"context_line":"               help\u003d\u0027Username of CeaStor\u0027),"},{"line_number":45,"context_line":"    cfg.StrOpt(\u0027ceastor_password\u0027,"},{"line_number":46,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":47,"context_line":"               help\u003d\u0027Password of CeaStor\u0027,"},{"line_number":48,"context_line":"               secret\u003dTrue),"},{"line_number":49,"context_line":"    cfg.ListOpt(\u0027ceastor_pools\u0027,"},{"line_number":50,"context_line":"                default\u003d[],"},{"line_number":51,"context_line":"                help\u003d\u0027Comma separated list of storage system storage.\u0027"}],"source_content_type":"text/x-python","patch_set":27,"id":"6c5edad5_615dc885","line":48,"range":{"start_line":39,"start_character":4,"end_line":48,"end_character":28},"updated":"2024-07-06 12:50:39.000000000","message":"Please use the existing san_ip, san_user, and san_password config options rather than adding new ones for the same thing.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":36,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"volume_opts \u003d ["},{"line_number":39,"context_line":"    cfg.StrOpt(\u0027ceastor_ip\u0027,"},{"line_number":40,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":41,"context_line":"               help\u003d\u0027IP address of CeaStor\u0027),"},{"line_number":42,"context_line":"    cfg.StrOpt(\u0027ceastor_username\u0027,"},{"line_number":43,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":44,"context_line":"               help\u003d\u0027Username of CeaStor\u0027),"},{"line_number":45,"context_line":"    cfg.StrOpt(\u0027ceastor_password\u0027,"},{"line_number":46,"context_line":"               default\u003d\u0027\u0027,"},{"line_number":47,"context_line":"               help\u003d\u0027Password of CeaStor\u0027,"},{"line_number":48,"context_line":"               secret\u003dTrue),"},{"line_number":49,"context_line":"    cfg.ListOpt(\u0027ceastor_pools\u0027,"},{"line_number":50,"context_line":"                default\u003d[],"},{"line_number":51,"context_line":"                help\u003d\u0027Comma separated list of storage system storage.\u0027"}],"source_content_type":"text/x-python","patch_set":27,"id":"0bf0e094_66128da4","line":48,"range":{"start_line":39,"start_character":4,"end_line":48,"end_character":28},"in_reply_to":"6c5edad5_615dc885","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"4187f88bc7a867fbdcae1054d25e543d56500648","unresolved":true,"context_lines":[{"line_number":48,"context_line":"               secret\u003dTrue),"},{"line_number":49,"context_line":"    cfg.ListOpt(\u0027ceastor_pools\u0027,"},{"line_number":50,"context_line":"                default\u003d[],"},{"line_number":51,"context_line":"                help\u003d\u0027Comma separated list of storage system storage.\u0027"},{"line_number":52,"context_line":"                     \u0027pools for volumes.\u0027),"},{"line_number":53,"context_line":"    cfg.ListOpt(\u0027ceastor_target_gateway_node\u0027,"},{"line_number":54,"context_line":"                default\u003d[],"}],"source_content_type":"text/x-python","patch_set":27,"id":"c22c421f_c647abdb","line":51,"range":{"start_line":51,"start_character":68,"end_line":51,"end_character":69},"updated":"2024-07-01 06:50:23.000000000","message":"nit: space is required after full stop","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":48,"context_line":"               secret\u003dTrue),"},{"line_number":49,"context_line":"    cfg.ListOpt(\u0027ceastor_pools\u0027,"},{"line_number":50,"context_line":"                default\u003d[],"},{"line_number":51,"context_line":"                help\u003d\u0027Comma separated list of storage system storage.\u0027"},{"line_number":52,"context_line":"                     \u0027pools for volumes.\u0027),"},{"line_number":53,"context_line":"    cfg.ListOpt(\u0027ceastor_target_gateway_node\u0027,"},{"line_number":54,"context_line":"                default\u003d[],"}],"source_content_type":"text/x-python","patch_set":27,"id":"f737b2f7_e9208e26","line":51,"range":{"start_line":51,"start_character":68,"end_line":51,"end_character":69},"in_reply_to":"338bcb0c_7293411c","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":48,"context_line":"               secret\u003dTrue),"},{"line_number":49,"context_line":"    cfg.ListOpt(\u0027ceastor_pools\u0027,"},{"line_number":50,"context_line":"                default\u003d[],"},{"line_number":51,"context_line":"                help\u003d\u0027Comma separated list of storage system storage.\u0027"},{"line_number":52,"context_line":"                     \u0027pools for volumes.\u0027),"},{"line_number":53,"context_line":"    cfg.ListOpt(\u0027ceastor_target_gateway_node\u0027,"},{"line_number":54,"context_line":"                default\u003d[],"}],"source_content_type":"text/x-python","patch_set":27,"id":"338bcb0c_7293411c","line":51,"range":{"start_line":51,"start_character":68,"end_line":51,"end_character":69},"in_reply_to":"c22c421f_c647abdb","updated":"2024-07-06 12:50:39.000000000","message":"Actually looks like the period should be removed.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"4187f88bc7a867fbdcae1054d25e543d56500648","unresolved":true,"context_lines":[{"line_number":53,"context_line":"    cfg.ListOpt(\u0027ceastor_target_gateway_node\u0027,"},{"line_number":54,"context_line":"                default\u003d[],"},{"line_number":55,"context_line":"                help\u003d\u0027The target gateway node name of storage system \u0027"},{"line_number":56,"context_line":"                     \u0027storage.\u0027),"},{"line_number":57,"context_line":"    cfg.StrOpt(\u0027ceastor_volume_provisioning_type\u0027,"},{"line_number":58,"context_line":"               default\u003d\u0027Thin\u0027,"},{"line_number":59,"context_line":"               choices\u003d[\u0027Thin\u0027, \u0027Thick\u0027],"}],"source_content_type":"text/x-python","patch_set":27,"id":"9cc4a169_e3a2e2f0","line":56,"updated":"2024-07-01 06:50:23.000000000","message":"nit: instead of \"storage system storage\", can we just say \"storage system\" ?","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":53,"context_line":"    cfg.ListOpt(\u0027ceastor_target_gateway_node\u0027,"},{"line_number":54,"context_line":"                default\u003d[],"},{"line_number":55,"context_line":"                help\u003d\u0027The target gateway node name of storage system \u0027"},{"line_number":56,"context_line":"                     \u0027storage.\u0027),"},{"line_number":57,"context_line":"    cfg.StrOpt(\u0027ceastor_volume_provisioning_type\u0027,"},{"line_number":58,"context_line":"               default\u003d\u0027Thin\u0027,"},{"line_number":59,"context_line":"               choices\u003d[\u0027Thin\u0027, \u0027Thick\u0027],"}],"source_content_type":"text/x-python","patch_set":27,"id":"88279ab5_0f64fcec","line":56,"in_reply_to":"9cc4a169_e3a2e2f0","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":55,"context_line":"                help\u003d\u0027The target gateway node name of storage system \u0027"},{"line_number":56,"context_line":"                     \u0027storage.\u0027),"},{"line_number":57,"context_line":"    cfg.StrOpt(\u0027ceastor_volume_provisioning_type\u0027,"},{"line_number":58,"context_line":"               default\u003d\u0027Thin\u0027,"},{"line_number":59,"context_line":"               choices\u003d[\u0027Thin\u0027, \u0027Thick\u0027],"},{"line_number":60,"context_line":"               help\u003d\u0027The provisioning type of volume in storage system \u0027"},{"line_number":61,"context_line":"                    \u0027storage.\u0027),"}],"source_content_type":"text/x-python","patch_set":27,"id":"65b10a82_e23f257e","line":58,"updated":"2024-07-06 12:50:39.000000000","message":"This should not be done here. See https://docs.openstack.org/cinder/queens/admin/blockstorage-over-subscription.html for how thin/thick provisioning is handled.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":55,"context_line":"                help\u003d\u0027The target gateway node name of storage system \u0027"},{"line_number":56,"context_line":"                     \u0027storage.\u0027),"},{"line_number":57,"context_line":"    cfg.StrOpt(\u0027ceastor_volume_provisioning_type\u0027,"},{"line_number":58,"context_line":"               default\u003d\u0027Thin\u0027,"},{"line_number":59,"context_line":"               choices\u003d[\u0027Thin\u0027, \u0027Thick\u0027],"},{"line_number":60,"context_line":"               help\u003d\u0027The provisioning type of volume in storage system \u0027"},{"line_number":61,"context_line":"                    \u0027storage.\u0027),"}],"source_content_type":"text/x-python","patch_set":27,"id":"aa31cf71_542668ca","line":58,"in_reply_to":"65b10a82_e23f257e","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":76,"context_line":"               min\u003d0, max\u003d16,"},{"line_number":77,"context_line":"               help\u003d\u0027Maximum number of nested volume clones that are \u0027"},{"line_number":78,"context_line":"                    \u0027taken before a flatten occurs. Set to 0 to flatten \u0027"},{"line_number":79,"context_line":"                    \u0027after cloning. \u0027),"},{"line_number":80,"context_line":"    cfg.BoolOpt(\u0027cbd_multiattach\u0027,"},{"line_number":81,"context_line":"                default\u003dTrue,"},{"line_number":82,"context_line":"                help\u003d\u0027If volume support multiattach.\u0027)"}],"source_content_type":"text/x-python","patch_set":27,"id":"c061c480_bd775753","line":79,"updated":"2024-07-06 12:50:39.000000000","message":"nit: extra space on the end should be removed.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":76,"context_line":"               min\u003d0, max\u003d16,"},{"line_number":77,"context_line":"               help\u003d\u0027Maximum number of nested volume clones that are \u0027"},{"line_number":78,"context_line":"                    \u0027taken before a flatten occurs. Set to 0 to flatten \u0027"},{"line_number":79,"context_line":"                    \u0027after cloning. \u0027),"},{"line_number":80,"context_line":"    cfg.BoolOpt(\u0027cbd_multiattach\u0027,"},{"line_number":81,"context_line":"                default\u003dTrue,"},{"line_number":82,"context_line":"                help\u003d\u0027If volume support multiattach.\u0027)"}],"source_content_type":"text/x-python","patch_set":27,"id":"e6e98458_fe71eb10","line":79,"in_reply_to":"c061c480_bd775753","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":77,"context_line":"               help\u003d\u0027Maximum number of nested volume clones that are \u0027"},{"line_number":78,"context_line":"                    \u0027taken before a flatten occurs. Set to 0 to flatten \u0027"},{"line_number":79,"context_line":"                    \u0027after cloning. \u0027),"},{"line_number":80,"context_line":"    cfg.BoolOpt(\u0027cbd_multiattach\u0027,"},{"line_number":81,"context_line":"                default\u003dTrue,"},{"line_number":82,"context_line":"                help\u003d\u0027If volume support multiattach.\u0027)"},{"line_number":83,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":27,"id":"d1f93ae8_4ba6e0c4","line":80,"updated":"2024-07-06 12:50:39.000000000","message":"See https://docs.openstack.org/cinder/latest/admin/volume-multiattach.html for how to handle multiattach.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":77,"context_line":"               help\u003d\u0027Maximum number of nested volume clones that are \u0027"},{"line_number":78,"context_line":"                    \u0027taken before a flatten occurs. Set to 0 to flatten \u0027"},{"line_number":79,"context_line":"                    \u0027after cloning. \u0027),"},{"line_number":80,"context_line":"    cfg.BoolOpt(\u0027cbd_multiattach\u0027,"},{"line_number":81,"context_line":"                default\u003dTrue,"},{"line_number":82,"context_line":"                help\u003d\u0027If volume support multiattach.\u0027)"},{"line_number":83,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":27,"id":"7b806dfc_1aa6f938","line":80,"in_reply_to":"d1f93ae8_4ba6e0c4","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[{"line_number":126,"context_line":"    def do_setup(self, context: context.RequestContext) -\u003e None:"},{"line_number":127,"context_line":"        \"\"\"Check that we have all configuration details from the storage.\"\"\""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"        LOG.debug(\u0027enter: do_setup\u0027)"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"        self.client \u003d ceastor_client.RestCommon("},{"line_number":132,"context_line":"            vip\u003dself.configuration.ceastor_ip,"}],"source_content_type":"text/x-python","patch_set":27,"id":"77f9c39c_ed0cc7f6","line":129,"updated":"2024-07-06 12:50:39.000000000","message":"This can be removed. Throughout you have the trace decorator, but then you also have LOG.debug calls. Leave it to the trace decorator to avoid duplication and noise in the logs.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":126,"context_line":"    def do_setup(self, context: context.RequestContext) -\u003e None:"},{"line_number":127,"context_line":"        \"\"\"Check that we have all configuration details from the storage.\"\"\""},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"        LOG.debug(\u0027enter: do_setup\u0027)"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"        self.client \u003d ceastor_client.RestCommon("},{"line_number":132,"context_line":"            vip\u003dself.configuration.ceastor_ip,"}],"source_content_type":"text/x-python","patch_set":27,"id":"fd2428c7_46c1fda3","line":129,"in_reply_to":"77f9c39c_ed0cc7f6","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"2f016bfcd177981b6103b29cda00d112a44c1b99","unresolved":true,"context_lines":[{"line_number":132,"context_line":"            vip\u003dself.configuration.ceastor_ip,"},{"line_number":133,"context_line":"            user\u003dself.configuration.ceastor_username,"},{"line_number":134,"context_line":"            password\u003dself.configuration.ceastor_password)"},{"line_number":135,"context_line":"        flag \u003d True"},{"line_number":136,"context_line":"        for x in range(3):"},{"line_number":137,"context_line":"            try:"},{"line_number":138,"context_line":"                self.client.login()"},{"line_number":139,"context_line":"            except Exception as e:"},{"line_number":140,"context_line":"                LOG.error(\"Login failed, attempt %d: %s\", x + 1, str(e))"},{"line_number":141,"context_line":"                time.sleep(1)"},{"line_number":142,"context_line":"                continue"},{"line_number":143,"context_line":"            else:"},{"line_number":144,"context_line":"                flag \u003d False"},{"line_number":145,"context_line":"                break"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        if flag:"},{"line_number":148,"context_line":"            self.client.login()"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"        LOG.debug(\u0027leave: do_setup\u0027)"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"    @volume_utils.trace"},{"line_number":153,"context_line":"    def check_for_setup_error(self) -\u003e None:"},{"line_number":154,"context_line":"        \"\"\"Verify that requirements are in place to use CeaStor driver.\"\"\""}],"source_content_type":"text/x-python","patch_set":27,"id":"d45f12ed_5ed9cc46","line":151,"range":{"start_line":135,"start_character":8,"end_line":151,"end_character":0},"updated":"2024-07-05 08:24:09.000000000","message":"I\u0027m not sure I understand the logic here.\nIf login() attempt fails for 3 times in the loop, it will login() again after the loop. The 4th attempt might also fail with exception, and the exception is not handled, right?\nWhy not use range(4) in the loop directly?","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"348c16797e7150d45fe657f66b82d430a71cb5cd","unresolved":false,"context_lines":[{"line_number":132,"context_line":"            vip\u003dself.configuration.ceastor_ip,"},{"line_number":133,"context_line":"            user\u003dself.configuration.ceastor_username,"},{"line_number":134,"context_line":"            password\u003dself.configuration.ceastor_password)"},{"line_number":135,"context_line":"        flag \u003d True"},{"line_number":136,"context_line":"        for x in range(3):"},{"line_number":137,"context_line":"            try:"},{"line_number":138,"context_line":"                self.client.login()"},{"line_number":139,"context_line":"            except Exception as e:"},{"line_number":140,"context_line":"                LOG.error(\"Login failed, attempt %d: %s\", x + 1, str(e))"},{"line_number":141,"context_line":"                time.sleep(1)"},{"line_number":142,"context_line":"                continue"},{"line_number":143,"context_line":"            else:"},{"line_number":144,"context_line":"                flag \u003d False"},{"line_number":145,"context_line":"                break"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        if flag:"},{"line_number":148,"context_line":"            self.client.login()"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"        LOG.debug(\u0027leave: do_setup\u0027)"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"    @volume_utils.trace"},{"line_number":153,"context_line":"    def check_for_setup_error(self) -\u003e None:"},{"line_number":154,"context_line":"        \"\"\"Verify that requirements are in place to use CeaStor driver.\"\"\""}],"source_content_type":"text/x-python","patch_set":27,"id":"3eb4f9ae_c00e6d7a","line":151,"range":{"start_line":135,"start_character":8,"end_line":151,"end_character":0},"in_reply_to":"21bbc75b_22f8ca06","updated":"2024-07-31 06:11:11.000000000","message":"Thank you for pointing out these issues. I have made the necessary changes. Please continue with the review.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":132,"context_line":"            vip\u003dself.configuration.ceastor_ip,"},{"line_number":133,"context_line":"            user\u003dself.configuration.ceastor_username,"},{"line_number":134,"context_line":"            password\u003dself.configuration.ceastor_password)"},{"line_number":135,"context_line":"        flag \u003d True"},{"line_number":136,"context_line":"        for x in range(3):"},{"line_number":137,"context_line":"            try:"},{"line_number":138,"context_line":"                self.client.login()"},{"line_number":139,"context_line":"            except Exception as e:"},{"line_number":140,"context_line":"                LOG.error(\"Login failed, attempt %d: %s\", x + 1, str(e))"},{"line_number":141,"context_line":"                time.sleep(1)"},{"line_number":142,"context_line":"                continue"},{"line_number":143,"context_line":"            else:"},{"line_number":144,"context_line":"                flag \u003d False"},{"line_number":145,"context_line":"                break"},{"line_number":146,"context_line":""},{"line_number":147,"context_line":"        if flag:"},{"line_number":148,"context_line":"            self.client.login()"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"        LOG.debug(\u0027leave: do_setup\u0027)"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"    @volume_utils.trace"},{"line_number":153,"context_line":"    def check_for_setup_error(self) -\u003e None:"},{"line_number":154,"context_line":"        \"\"\"Verify that requirements are in place to use CeaStor driver.\"\"\""}],"source_content_type":"text/x-python","patch_set":27,"id":"21bbc75b_22f8ca06","line":151,"range":{"start_line":135,"start_character":8,"end_line":151,"end_character":0},"in_reply_to":"d45f12ed_5ed9cc46","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"}],"cinder/volume/drivers/ceastor/install.sh":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"177a8d53ff12e5f0f7f79677077ab7c733117c60","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"#intall nmve-tcp/nvme-rdma"},{"line_number":4,"context_line":"function install_nvme(){"},{"line_number":5,"context_line":"    lsmod |grep nvme_tcp \u003e/dev/null"}],"source_content_type":"text/x-sh","patch_set":27,"id":"b5efe5c3_058219b9","line":2,"updated":"2024-07-06 12:30:50.000000000","message":"Where and how is this script expected to be run?","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"#intall nmve-tcp/nvme-rdma"},{"line_number":4,"context_line":"function install_nvme(){"},{"line_number":5,"context_line":"    lsmod |grep nvme_tcp \u003e/dev/null"}],"source_content_type":"text/x-sh","patch_set":27,"id":"a5452eb6_b9647d68","line":2,"in_reply_to":"b5efe5c3_058219b9","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"}],"doc/source/configuration/block-storage/drivers/ceastor-driver.rst":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"a967860f5d5d4581a6769140534e5ee0f391c63f","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":27,"id":"d2323a62_24f195b5","line":86,"updated":"2024-07-06 12:50:39.000000000","message":"Add the following to have a table automatically generated for the driver specific config options:\n\n```\nDriver options\n~~~~~~~~~~~~~~\n\n.. config-table::\n   :config-target: Ceastor\n\n   cinder.volume.drivers.ceastor.driver\n```","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"348c16797e7150d45fe657f66b82d430a71cb5cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"959c818e_43f3aed8","line":86,"in_reply_to":"4ccfaffa_2a526c4a","updated":"2024-07-31 06:11:11.000000000","message":"Thank you for pointing out these issues. I have made the necessary changes. Please continue with the review.","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"},{"author":{"_account_id":37063,"name":"Max","display_name":"Max","email":"cuishuting@cestc.cn","username":"yunfei"},"change_message_id":"d72b1389096c754f9add436beba82ba46c8d7b27","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"4ccfaffa_2a526c4a","line":86,"in_reply_to":"d2323a62_24f195b5","updated":"2024-07-25 07:55:58.000000000","message":"Done","commit_id":"d0b5df0e09faea7017f5bc1449e9fc5c3a1643c2"}]}