)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"cbaf05b4116af7c0fa7cffdf334d7a20f12c1377","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2021-09-22 20:27:11 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix PowerFlex connector HTTPS certificate validation"},{"line_number":8,"context_line":"Closes-Bug: #1929223"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: Ia73c391d2fafde119e0bb4914c30b48b4300e330"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"ca56a4d4_2497edfe","line":8,"range":{"start_line":8,"start_character":0,"end_line":8,"end_character":20},"updated":"2021-09-22 17:45:13.000000000","message":":nit: The title needs to be followed by a blank line to be formatted properly. \n\nPlease move this line to L9.","commit_id":"c19370431555016cc92244ea35c415e4f8b5d3e7"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"60feb631962ed2551d4b8e44efd3c1f2a2b6cc86","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2021-09-22 20:27:11 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix PowerFlex connector HTTPS certificate validation"},{"line_number":8,"context_line":"Closes-Bug: #1929223"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: Ia73c391d2fafde119e0bb4914c30b48b4300e330"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5a4da60a_ca9fccc4","line":8,"range":{"start_line":8,"start_character":0,"end_line":8,"end_character":20},"in_reply_to":"ca56a4d4_2497edfe","updated":"2022-02-09 20:18:38.000000000","message":"Done","commit_id":"c19370431555016cc92244ea35c415e4f8b5d3e7"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"e1125c79e1768f3e26abd8e52c68fc5cc2a9dd31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"3df9cca2_5a96cf02","updated":"2022-02-14 15:06:35.000000000","message":"@Simon: the connector is named \u0027scaleio\u0027, but the backend that uses it is PowerFlex (change I9fc2ac1ea094 : ScaleIO -\u003e VxFlex OS, change Iab5d66bbaab6 : VxFlex OS -\u003e PowerFlex).  Also, the configuration is done on the cinder side when you configure the PowerFlex backend.  So I think it\u0027s better to say \u0027PowerFlex\u0027 here.","commit_id":"adde4de38d3bee0e3e3fa9c94de647b28d53a2a8"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"62140a96bcaec574589f48c3fcb81aa91152f45e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"9e086554_2673a60b","updated":"2022-02-09 20:16:03.000000000","message":"Code looks OK.  Corresponding driver change is https://review.opendev.org/c/openstack/cinder/+/810415\n\nHaven\u0027t seen the CI respond, though.","commit_id":"adde4de38d3bee0e3e3fa9c94de647b28d53a2a8"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"acbe97925919f4041b83ba0a5741a2352a5c468c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"14a71c2b_9f6b1153","updated":"2021-10-19 07:56:06.000000000","message":"LGTM","commit_id":"adde4de38d3bee0e3e3fa9c94de647b28d53a2a8"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"abd0252ad4df0bdfb8340324b76ca49720f35627","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e51cd7d7_92d1ad86","updated":"2022-02-10 11:43:29.000000000","message":"LGTM","commit_id":"adde4de38d3bee0e3e3fa9c94de647b28d53a2a8"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"eb3f7637ccddd9cdd9f8242af7836d49f7027fa6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"85e0559b_244394ab","updated":"2022-02-09 15:02:04.000000000","message":"Looks like this should probably get merged.","commit_id":"adde4de38d3bee0e3e3fa9c94de647b28d53a2a8"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"4a5aecbee5f1fc6899117a5508a4b8d4c450f826","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"26d033ad_da7283ce","updated":"2022-02-14 15:43:51.000000000","message":"Merging.","commit_id":"adde4de38d3bee0e3e3fa9c94de647b28d53a2a8"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"bdedd876bc6327d00f8aa98bb6fca485b80b2ffa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"a8987299_73b17f5f","updated":"2022-02-14 15:35:54.000000000","message":"OK - I\u0027m old school and didn\u0027t realize they had renamed it (again)","commit_id":"adde4de38d3bee0e3e3fa9c94de647b28d53a2a8"},{"author":{"_account_id":13425,"name":"Simon Dodsley","email":"simon@purestorage.com","username":"sdodsley"},"change_message_id":"603fc9039ceeecdd74f6a755f10867ad20d51cf3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"afe6a9d2_ba4f3e6d","updated":"2022-02-13 15:13:42.000000000","message":"nit: This is the ScaleIO connector - not PowerFlex - we should make this correct in both the patch name and the commit message, especially as this is a security related patch and the bug referenced is ScaleIO","commit_id":"adde4de38d3bee0e3e3fa9c94de647b28d53a2a8"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b1cd4fdecd612b607326b9ad960b8f2268d6baa7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"15210219_3aea0730","updated":"2022-02-09 20:15:07.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"adde4de38d3bee0e3e3fa9c94de647b28d53a2a8"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3de2834edbd42cd86749d859498d01ab18797786","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"f8d06cec_d2677b5e","updated":"2022-02-12 18:40:40.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"adde4de38d3bee0e3e3fa9c94de647b28d53a2a8"}],"os_brick/initiator/connectors/scaleio.py":[{"author":{"_account_id":13671,"name":"Vladislav Belogrudov","email":"v.belogrudov@yadro.com","username":"vb"},"change_message_id":"fe283bbcc602bb46901999df3bb3f6d1aa843e48","unresolved":true,"context_lines":[{"line_number":339,"context_line":"            connection_properties)"},{"line_number":340,"context_line":"        self.iops_limit \u003d connection_properties[\u0027iopsLimit\u0027]"},{"line_number":341,"context_line":"        self.bandwidth_limit \u003d connection_properties[\u0027bandwidthLimit\u0027]"},{"line_number":342,"context_line":"        self.verify_certificate \u003d connection_properties[\u0027verify_certificate\u0027]"},{"line_number":343,"context_line":"        self.certificate_path \u003d connection_properties[\u0027certificate_path\u0027]"},{"line_number":344,"context_line":"        device_info \u003d {\u0027type\u0027: \u0027block\u0027,"},{"line_number":345,"context_line":"                       \u0027path\u0027: self.volume_path}"}],"source_content_type":"text/x-python","patch_set":2,"id":"ab035b27_2b12882a","line":342,"updated":"2021-09-22 13:35:07.000000000","message":"this is unsafe, we could still have volumes without these two new properties. Let\u0027s better connection_properties.get(\"verify_certificate\")","commit_id":"42c879638b8a0f6320ebf3a8675f186773d0c1d4"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"60feb631962ed2551d4b8e44efd3c1f2a2b6cc86","unresolved":false,"context_lines":[{"line_number":339,"context_line":"            connection_properties)"},{"line_number":340,"context_line":"        self.iops_limit \u003d connection_properties[\u0027iopsLimit\u0027]"},{"line_number":341,"context_line":"        self.bandwidth_limit \u003d connection_properties[\u0027bandwidthLimit\u0027]"},{"line_number":342,"context_line":"        self.verify_certificate \u003d connection_properties[\u0027verify_certificate\u0027]"},{"line_number":343,"context_line":"        self.certificate_path \u003d connection_properties[\u0027certificate_path\u0027]"},{"line_number":344,"context_line":"        device_info \u003d {\u0027type\u0027: \u0027block\u0027,"},{"line_number":345,"context_line":"                       \u0027path\u0027: self.volume_path}"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a275049_4c520f3c","line":342,"in_reply_to":"ab035b27_2b12882a","updated":"2022-02-09 20:18:38.000000000","message":"(Just noting that this was corrected in the current PS.)","commit_id":"42c879638b8a0f6320ebf3a8675f186773d0c1d4"}]}
