)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"b062dade1c3dfefd8ce1e04fabf882d91d4d7b30","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"XtremIO: Set the location of a CA certificate"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* the current CA veriffication will only work for known CA which"},{"line_number":10,"context_line":"  are bundled with requests."},{"line_number":11,"context_line":"  the extra parameter allow to configure a custom location for"},{"line_number":12,"context_line":"  the CA bundle"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a7b0d38_8a6d7506","line":9,"updated":"2015-12-23 15:37:38.000000000","message":"s/the current/The current\n\ns/veriffication/verification\n\ns/known CA/known CAs\n\ns/the extra parameter allow/The extra parameter allows","commit_id":"acbf85465f68cb5c383c34ae99e28d602a0b5978"},{"author":{"_account_id":11418,"name":"Shay Halsband","email":"shay.halsband@emc.com","username":"shyh"},"change_message_id":"aeabd1aadc64fe8b624203db6a4d0be73bfcd154","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"XtremIO: Set the location of a CA certificate"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"* the current CA veriffication will only work for known CA which"},{"line_number":10,"context_line":"  are bundled with requests."},{"line_number":11,"context_line":"  the extra parameter allow to configure a custom location for"},{"line_number":12,"context_line":"  the CA bundle"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"1a7b0d38_753d6eba","line":9,"in_reply_to":"1a7b0d38_8a6d7506","updated":"2015-12-24 10:39:05.000000000","message":"Done","commit_id":"acbf85465f68cb5c383c34ae99e28d602a0b5978"}],"cinder/volume/drivers/emc/xtremio.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"6527f06f23b72e17106954ba652fc681d9e00665","unresolved":false,"context_lines":[{"line_number":92,"context_line":"            verify_path \u003d (self.configuration."},{"line_number":93,"context_line":"                           safe_get(\u0027driver_ssl_cert_path\u0027) or None)"},{"line_number":94,"context_line":"            if verify_path:"},{"line_number":95,"context_line":"                self.verify \u003d verify_path"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    def get_base_url(self, ver):"},{"line_number":98,"context_line":"        if ver \u003d\u003d \u0027v1\u0027:"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a7b0d38_3319ecd7","line":95,"range":{"start_line":95,"start_character":21,"end_line":95,"end_character":27},"updated":"2015-12-28 21:46:03.000000000","message":"It looks wrong. ScaleIO driver uses self.server_certificate_path. Should we use the same config option and attribute here?","commit_id":"07be46ed877f3dcc3f0afe95d3d27f5223dd19aa"},{"author":{"_account_id":11418,"name":"Shay Halsband","email":"shay.halsband@emc.com","username":"shyh"},"change_message_id":"c68b64a591f27acc01674e015d1116b6e0ecf8a7","unresolved":false,"context_lines":[{"line_number":92,"context_line":"            verify_path \u003d (self.configuration."},{"line_number":93,"context_line":"                           safe_get(\u0027driver_ssl_cert_path\u0027) or None)"},{"line_number":94,"context_line":"            if verify_path:"},{"line_number":95,"context_line":"                self.verify \u003d verify_path"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    def get_base_url(self, ver):"},{"line_number":98,"context_line":"        if ver \u003d\u003d \u0027v1\u0027:"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_090a0927","line":95,"range":{"start_line":95,"start_character":21,"end_line":95,"end_character":27},"in_reply_to":"1a7b0d38_3319ecd7","updated":"2015-12-30 15:35:29.000000000","message":"ScaleIO uses their own param (sio_verify_server_certificate), as this is at least the second driver that uses this parameter it should be shared to all drivers in driver.py","commit_id":"07be46ed877f3dcc3f0afe95d3d27f5223dd19aa"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"252f779d09961ceb915689ab14d652fe11b996a9","unresolved":false,"context_lines":[{"line_number":92,"context_line":"            verify_path \u003d (self.configuration."},{"line_number":93,"context_line":"                           safe_get(\u0027driver_ssl_cert_path\u0027) or None)"},{"line_number":94,"context_line":"            if verify_path:"},{"line_number":95,"context_line":"                self.verify \u003d verify_path"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    def get_base_url(self, ver):"},{"line_number":98,"context_line":"        if ver \u003d\u003d \u0027v1\u0027:"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_adef42af","line":95,"range":{"start_line":95,"start_character":21,"end_line":95,"end_character":27},"in_reply_to":"fa69d971_090a0927","updated":"2015-12-30 19:37:01.000000000","message":"Good point.  We can change ScaleIO driver to use the common option in driver.py in a different patch.  Thanks.","commit_id":"07be46ed877f3dcc3f0afe95d3d27f5223dd19aa"}]}