)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"15117b5763888e94875845a1a356964afe78d95c","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     happystacker \u003cjeanpierre.roquesalane@dell.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-09-19 15:01:10 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Dell PowerFlex: Failed to attach a volume when using"},{"line_number":8,"context_line":"self-signed Certificates"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Initially before the change"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"79f8be92_1b60c407","line":7,"updated":"2022-11-04 02:11:19.000000000","message":"Why would you want to make a change to cause that to happen? 😊\n\nCommit message subject line should state what the change is that\u0027s being made, not restate the bug that prompted the change.\n\nPlease keep subject line to one line limited to 52 characters, followed by a blank line, then a more detailed description of the change being made.","commit_id":"e6b96475be3baf61f32ecfcba6c2997f4dc5dae5"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"086da8e74dc1b91f170abf2f9c561bc685221122","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     happystacker \u003cjeanpierre.roquesalane@dell.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-09-19 15:01:10 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Dell PowerFlex: Failed to attach a volume when using"},{"line_number":8,"context_line":"self-signed Certificates"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Initially before the change"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"09e15ab1_3fd24efd","line":7,"in_reply_to":"79f8be92_1b60c407","updated":"2022-11-07 13:06:34.000000000","message":"Done","commit_id":"e6b96475be3baf61f32ecfcba6c2997f4dc5dae5"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":33473,"name":"Atsushi Kawai","display_name":"akawai","email":"atsushi.kawai.bu@hitachi.com","username":"akawai"},"change_message_id":"93c9d2a70ecb0681f337ddaa6f9f454fb68e8653","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"388748dd_9a719498","updated":"2022-10-21 10:03:34.000000000","message":"LGTM","commit_id":"e6b96475be3baf61f32ecfcba6c2997f4dc5dae5"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"3517e09f6e199138e737af39b5794ee3eccc49a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"531806b9_78d3809f","updated":"2022-09-29 09:29:58.000000000","message":"LGTM","commit_id":"e6b96475be3baf61f32ecfcba6c2997f4dc5dae5"},{"author":{"_account_id":13671,"name":"Vladislav Belogrudov","email":"v.belogrudov@yadro.com","username":"vb"},"change_message_id":"792d609693b47ee4cd43f8e875d93cfb2e906908","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"76bcf25c_6950bbab","updated":"2022-09-19 14:46:33.000000000","message":"looks good, I have compared the change with actual connector. Are there specific tests?","commit_id":"e6b96475be3baf61f32ecfcba6c2997f4dc5dae5"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"5a2cf2c61a901581d9e526d2ac6543ec9b3c086c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"cc15f394_ce24bd76","updated":"2022-09-19 15:32:02.000000000","message":"recheck","commit_id":"e6b96475be3baf61f32ecfcba6c2997f4dc5dae5"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"6ec20c0d9e72f5c622709aa15e13bcea94b2a2af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"76fd91d2_37690c68","updated":"2022-09-28 05:29:17.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"e6b96475be3baf61f32ecfcba6c2997f4dc5dae5"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"a90504f764b8ec81fdb889348b7b16a1025eeaad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"78791a01_20c12577","updated":"2022-09-27 13:51:48.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"e6b96475be3baf61f32ecfcba6c2997f4dc5dae5"},{"author":{"_account_id":33807,"name":"Jacob Wang","email":"jacob_wang1@dell.com","username":"jacob0522"},"change_message_id":"ba3d84e333e6e705069f85eb6b72d9d0167d0595","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"aaad4f15_37b8b081","updated":"2022-09-27 11:15:34.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"e6b96475be3baf61f32ecfcba6c2997f4dc5dae5"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"95ce08491644562e79beebe9a24ef05fad8b06f3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e858d8bf_3572f5ff","updated":"2022-09-28 08:06:11.000000000","message":"run-DellEMC PowerFlex CI","commit_id":"e6b96475be3baf61f32ecfcba6c2997f4dc5dae5"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"f12b7d086225ae1a5c345c2ef0bcbdc568cf9ec9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"1d5f40f9_de9a3829","updated":"2022-11-10 16:04:36.000000000","message":"recheck","commit_id":"b7f501aa99071aad5ee5f3f1f1b86e8aad6994e8"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"256aca32409eab74ac9a709254b52f3ad56e5d94","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b6826e03_40853d25","updated":"2022-11-17 08:07:55.000000000","message":"recheck","commit_id":"b7f501aa99071aad5ee5f3f1f1b86e8aad6994e8"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"0a26de627a29256f3578dc53befe8865243f65e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"f3104960_6848b3c7","updated":"2023-03-02 11:28:53.000000000","message":"this requires a releasenote as it\u0027s a bugfix. Also some unit test should be there to check the parameters passed to os-brick after initializing connection.","commit_id":"b7f501aa99071aad5ee5f3f1f1b86e8aad6994e8"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"cca48065425ac2a04eece0bcaa954005d88bf5b2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9963fe09_c1c26f49","in_reply_to":"f3104960_6848b3c7","updated":"2023-03-02 14:51:16.000000000","message":"I have addedd the release note. Can we add the unit tests later? I still need to go through these unit tests as I\u0027m totally new to it.","commit_id":"b7f501aa99071aad5ee5f3f1f1b86e8aad6994e8"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"a15e998f13b447eee6434cfa28738c558b33ee9c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"dff0fdb7_26e7cfca","updated":"2023-03-02 15:39:29.000000000","message":"LGTM, we can leave the UT for later since it\u0027s not a lot of new code introduced. will wait for CI to +2","commit_id":"76f0229b7f495d47de3c6609dfa19e4fc1c34b6f"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"e80d9b1149c0a32461c8b3f4a3681ef88881525b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"4c4338bc_3a7503d9","updated":"2023-03-06 10:23:48.000000000","message":"recheck","commit_id":"76f0229b7f495d47de3c6609dfa19e4fc1c34b6f"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"fcc246133aa3e6a170742d3f979c22bc2a08b05a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"14d16eaa_bd9030a6","updated":"2023-03-06 10:21:40.000000000","message":"recheck openstacksdk-functional-devstack failure","commit_id":"76f0229b7f495d47de3c6609dfa19e4fc1c34b6f"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"7b5084c493f29ca2656785489cfef16f08b4a217","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"272ccfba_cc91e794","in_reply_to":"04856efa_4d533098","updated":"2023-03-15 11:21:37.000000000","message":"This doc[1] might help but the general practice is to look at existing tests and implement new tests referencing the existing ones.\n\n[1] https://docs.openstack.org/cinder/latest/contributor/testing.html","commit_id":"76f0229b7f495d47de3c6609dfa19e4fc1c34b6f"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"7ffee9cc789bc3c1006ad7f0d09042d5ffe718b4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"04856efa_4d533098","in_reply_to":"dff0fdb7_26e7cfca","updated":"2023-03-02 15:43:46.000000000","message":"Cool, do you have any link to material which an help us to understand unit tests?","commit_id":"76f0229b7f495d47de3c6609dfa19e4fc1c34b6f"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"2e8514540a61e67d6b2c420e6d1429cf07f9e4ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"94b5caa2_3440269c","updated":"2023-03-15 11:13:11.000000000","message":"LGTM and DellEMC PowerVault ME CI passed","commit_id":"82823ace4d714ac10427ea3c6fed320c27b56f7d"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"f4b4f263ab6e80f34e8fe2c1d7496692b3161ac5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"76590150_73cc5d6a","updated":"2023-03-15 11:13:49.000000000","message":"Not sure what happened with Jay vote","commit_id":"82823ace4d714ac10427ea3c6fed320c27b56f7d"},{"author":{"_account_id":7198,"name":"Jay Bryant","email":"jungleboyj@electronicjungle.net","username":"jsbryant"},"change_message_id":"bb1655d46624649f602fe760473c4703c26b6551","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c6606726_ae461b05","updated":"2023-03-08 19:12:47.000000000","message":"Passing CI and I think the change makes sense.","commit_id":"82823ace4d714ac10427ea3c6fed320c27b56f7d"},{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"fcc61f00b7771cc51203c436909e728af1f7cec8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"13d8dfef_f4388b3d","updated":"2023-03-08 22:00:42.000000000","message":"This looks good to me.\n","commit_id":"82823ace4d714ac10427ea3c6fed320c27b56f7d"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"7b5084c493f29ca2656785489cfef16f08b4a217","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2031cd4a_a3ef4b78","updated":"2023-03-15 11:21:37.000000000","message":"still missing UT but can be addressed in a followup. Powerflex CI is passing, LGTM.","commit_id":"82823ace4d714ac10427ea3c6fed320c27b56f7d"}],"cinder/volume/drivers/dell_emc/powerflex/driver.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"5f1f023406883f108b228f139821bdf67b026769","unresolved":true,"context_lines":[{"line_number":865,"context_line":"        connection_properties[\"scaleIO_volume_id\"] \u003d vol_or_snap.provider_id"},{"line_number":866,"context_line":"        connection_properties[\"config_group\"] \u003d self.configuration.config_group"},{"line_number":867,"context_line":"        connection_properties[\"failed_over\"] \u003d self._is_failed_over"},{"line_number":868,"context_line":"        connection_properties[\"verify_certificate\"] \u003d ("},{"line_number":869,"context_line":"            self._get_client().verify_certificate"},{"line_number":870,"context_line":"        )"},{"line_number":871,"context_line":"        connection_properties[\"certificate_path\"] \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"62fe659d_ed12f9e6","line":868,"updated":"2022-11-15 15:35:06.000000000","message":"Now that I\u0027ve looked closer, this change is not correct. The driver should be using the common configuration settings for this: https://opendev.org/openstack/cinder/src/branch/master/cinder/volume/driver.py#L195-L199\n\nIf the driver is not currently looking at ``driver_ssl_cert_verify`` I would consider that a bug that should be fixed ASAP.","commit_id":"b7f501aa99071aad5ee5f3f1f1b86e8aad6994e8"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"7e9ed129693b5c0f25b8f3e9e5aa39c827f556c6","unresolved":true,"context_lines":[{"line_number":865,"context_line":"        connection_properties[\"scaleIO_volume_id\"] \u003d vol_or_snap.provider_id"},{"line_number":866,"context_line":"        connection_properties[\"config_group\"] \u003d self.configuration.config_group"},{"line_number":867,"context_line":"        connection_properties[\"failed_over\"] \u003d self._is_failed_over"},{"line_number":868,"context_line":"        connection_properties[\"verify_certificate\"] \u003d ("},{"line_number":869,"context_line":"            self._get_client().verify_certificate"},{"line_number":870,"context_line":"        )"},{"line_number":871,"context_line":"        connection_properties[\"certificate_path\"] \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"c1a969d0_6f714f6d","line":868,"in_reply_to":"57b0813c_f8e8202e","updated":"2023-03-01 15:00:09.000000000","message":"DellEMC Unity CI passed.  I think we need an update from @Sean McGinnis","commit_id":"b7f501aa99071aad5ee5f3f1f1b86e8aad6994e8"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"79ad6719d677c86a91a018845f8da349959ee676","unresolved":true,"context_lines":[{"line_number":865,"context_line":"        connection_properties[\"scaleIO_volume_id\"] \u003d vol_or_snap.provider_id"},{"line_number":866,"context_line":"        connection_properties[\"config_group\"] \u003d self.configuration.config_group"},{"line_number":867,"context_line":"        connection_properties[\"failed_over\"] \u003d self._is_failed_over"},{"line_number":868,"context_line":"        connection_properties[\"verify_certificate\"] \u003d ("},{"line_number":869,"context_line":"            self._get_client().verify_certificate"},{"line_number":870,"context_line":"        )"},{"line_number":871,"context_line":"        connection_properties[\"certificate_path\"] \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"a5017718_8edd94e9","line":868,"in_reply_to":"62fe659d_ed12f9e6","updated":"2022-11-16 15:20:24.000000000","message":"the driver supports the param driver_ssl_cert_verify but when set to False and because it\u0027s not well configured during initialization of the connection, paramaters are not passed to self.connector.connect_volume(connection_properties), resulting in an SSL CERTIFICATE ERROR.","commit_id":"b7f501aa99071aad5ee5f3f1f1b86e8aad6994e8"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"af78e398116eb5ac2e1d1465b8dfd948a0255732","unresolved":true,"context_lines":[{"line_number":865,"context_line":"        connection_properties[\"scaleIO_volume_id\"] \u003d vol_or_snap.provider_id"},{"line_number":866,"context_line":"        connection_properties[\"config_group\"] \u003d self.configuration.config_group"},{"line_number":867,"context_line":"        connection_properties[\"failed_over\"] \u003d self._is_failed_over"},{"line_number":868,"context_line":"        connection_properties[\"verify_certificate\"] \u003d ("},{"line_number":869,"context_line":"            self._get_client().verify_certificate"},{"line_number":870,"context_line":"        )"},{"line_number":871,"context_line":"        connection_properties[\"certificate_path\"] \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"a827f380_2dc73b05","line":868,"in_reply_to":"a5017718_8edd94e9","updated":"2022-11-16 15:25:13.000000000","message":"I\u0027m talking about this request:\n\n        LOG.info(\"map volume request: %(request)s\", {\u0027request\u0027: request})\n        r \u003d requests.post(\n            request,\n            data\u003djson.dumps(params),\n            headers\u003dheaders,\n            auth\u003d(self.server_username, self.server_token),\n            verify\u003dself._verify_cert()\n        )\n        \ncoming from:\nhttps://github.com/openstack/os-brick/blob/0639af4fc8d7f3abdbfe6326b778b219652af864/os_brick/initiator/connectors/scaleio.py#L351","commit_id":"b7f501aa99071aad5ee5f3f1f1b86e8aad6994e8"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"86e18248e6e721669f3256e3c2bfea44c91671e1","unresolved":false,"context_lines":[{"line_number":865,"context_line":"        connection_properties[\"scaleIO_volume_id\"] \u003d vol_or_snap.provider_id"},{"line_number":866,"context_line":"        connection_properties[\"config_group\"] \u003d self.configuration.config_group"},{"line_number":867,"context_line":"        connection_properties[\"failed_over\"] \u003d self._is_failed_over"},{"line_number":868,"context_line":"        connection_properties[\"verify_certificate\"] \u003d ("},{"line_number":869,"context_line":"            self._get_client().verify_certificate"},{"line_number":870,"context_line":"        )"},{"line_number":871,"context_line":"        connection_properties[\"certificate_path\"] \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"57b0813c_f8e8202e","line":868,"in_reply_to":"a827f380_2dc73b05","updated":"2022-12-19 10:16:04.000000000","message":"Done","commit_id":"b7f501aa99071aad5ee5f3f1f1b86e8aad6994e8"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"b85c92ee03aad09f53fe18f77291af0bd41c92ad","unresolved":false,"context_lines":[{"line_number":865,"context_line":"        connection_properties[\"scaleIO_volume_id\"] \u003d vol_or_snap.provider_id"},{"line_number":866,"context_line":"        connection_properties[\"config_group\"] \u003d self.configuration.config_group"},{"line_number":867,"context_line":"        connection_properties[\"failed_over\"] \u003d self._is_failed_over"},{"line_number":868,"context_line":"        connection_properties[\"verify_certificate\"] \u003d ("},{"line_number":869,"context_line":"            self._get_client().verify_certificate"},{"line_number":870,"context_line":"        )"},{"line_number":871,"context_line":"        connection_properties[\"certificate_path\"] \u003d ("}],"source_content_type":"text/x-python","patch_set":3,"id":"d881a4ff_24877a3e","line":868,"in_reply_to":"c1a969d0_6f714f6d","updated":"2023-03-02 14:52:35.000000000","message":"Done","commit_id":"b7f501aa99071aad5ee5f3f1f1b86e8aad6994e8"}],"releasenotes/notes/notes/dell-powerflex-bugfix-1998136-self-signed-certificates-62e3cb444ab7ff2b.yaml":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"51f6e14cd401f3b6726110b836c74bee9aa429a0","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/1998136\u003e`_:"},{"line_number":6,"context_line":"    When using self signed certificates, the option"},{"line_number":7,"context_line":"    sent to os-brick via the connection_properties was"},{"line_number":8,"context_line":"    not correctly handled. This code change fixes it by"},{"line_number":9,"context_line":"    adding the \u0027verify_certificate\u0027 and \u0027certificate_path\u0027"},{"line_number":10,"context_line":"    to the driver when initializing the connection."},{"line_number":11,"context_line":"    This mishandling was introduced due to a regression."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"c72380c1_f609d0ca","line":8,"updated":"2023-03-02 21:42:24.000000000","message":"Do not state \"This code change\". That context is lost in the published release notes. Maybe something like \"It has now been fixed by adding...\"\n\nFor reference, here is what the rendered release notes look like:\n\nhttps://docs.openstack.org/releasenotes/cinder/zed.html","commit_id":"76f0229b7f495d47de3c6609dfa19e4fc1c34b6f"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"9206a3cf16e8a90af20d80ada4b2c3478441b966","unresolved":false,"context_lines":[{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/1998136\u003e`_:"},{"line_number":6,"context_line":"    When using self signed certificates, the option"},{"line_number":7,"context_line":"    sent to os-brick via the connection_properties was"},{"line_number":8,"context_line":"    not correctly handled. This code change fixes it by"},{"line_number":9,"context_line":"    adding the \u0027verify_certificate\u0027 and \u0027certificate_path\u0027"},{"line_number":10,"context_line":"    to the driver when initializing the connection."},{"line_number":11,"context_line":"    This mishandling was introduced due to a regression."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"4dca4e6f_c1685940","line":8,"in_reply_to":"c72380c1_f609d0ca","updated":"2023-03-07 13:00:11.000000000","message":"Done","commit_id":"76f0229b7f495d47de3c6609dfa19e4fc1c34b6f"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"51f6e14cd401f3b6726110b836c74bee9aa429a0","unresolved":true,"context_lines":[{"line_number":8,"context_line":"    not correctly handled. This code change fixes it by"},{"line_number":9,"context_line":"    adding the \u0027verify_certificate\u0027 and \u0027certificate_path\u0027"},{"line_number":10,"context_line":"    to the driver when initializing the connection."},{"line_number":11,"context_line":"    This mishandling was introduced due to a regression."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"64213586_baf24f0a","line":11,"updated":"2023-03-02 21:42:24.000000000","message":"This last line can be removed too.","commit_id":"76f0229b7f495d47de3c6609dfa19e4fc1c34b6f"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"9206a3cf16e8a90af20d80ada4b2c3478441b966","unresolved":false,"context_lines":[{"line_number":8,"context_line":"    not correctly handled. This code change fixes it by"},{"line_number":9,"context_line":"    adding the \u0027verify_certificate\u0027 and \u0027certificate_path\u0027"},{"line_number":10,"context_line":"    to the driver when initializing the connection."},{"line_number":11,"context_line":"    This mishandling was introduced due to a regression."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"29ff729a_f79fab86","line":11,"in_reply_to":"64213586_baf24f0a","updated":"2023-03-07 13:00:11.000000000","message":"Done","commit_id":"76f0229b7f495d47de3c6609dfa19e4fc1c34b6f"}]}