)]}'
{"cinder/volume/drivers/vmware/vmdk.py":[{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"28389a20f71800372597e51722ff8524a430f609","unresolved":false,"context_lines":[{"line_number":108,"context_line":"               help\u003d\u0027Directory where virtual disks are stored during volume \u0027"},{"line_number":109,"context_line":"                    \u0027backup and restore.\u0027),"},{"line_number":110,"context_line":"    cfg.StrOpt(\u0027vmware_ca_file\u0027,"},{"line_number":111,"context_line":"               default\u003dNone,"},{"line_number":112,"context_line":"               help\u003d\u0027CA bundle file to verify vCenter server certificate.\u0027)"},{"line_number":113,"context_line":"]"},{"line_number":114,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3acd31a7_794c1b9e","line":111,"updated":"2015-05-05 00:36:10.000000000","message":"Defaults to None are redundant.  This can be removed.","commit_id":"39478338bb4a1cbd625a6176d4403bb34a2a0630"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"831574fa6e02d365731f1d34a4dcaa9478535f97","unresolved":false,"context_lines":[{"line_number":108,"context_line":"               help\u003d\u0027Directory where virtual disks are stored during volume \u0027"},{"line_number":109,"context_line":"                    \u0027backup and restore.\u0027),"},{"line_number":110,"context_line":"    cfg.StrOpt(\u0027vmware_ca_file\u0027,"},{"line_number":111,"context_line":"               default\u003dNone,"},{"line_number":112,"context_line":"               help\u003d\u0027CA bundle file to verify vCenter server certificate.\u0027)"},{"line_number":113,"context_line":"]"},{"line_number":114,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3acd31a7_ae2f0887","line":111,"in_reply_to":"3acd31a7_794c1b9e","updated":"2015-05-05 01:50:11.000000000","message":"Never mind, I found a similar patch that was abandoned in favor of a hacking rule.","commit_id":"39478338bb4a1cbd625a6176d4403bb34a2a0630"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"28389a20f71800372597e51722ff8524a430f609","unresolved":false,"context_lines":[{"line_number":109,"context_line":"                    \u0027backup and restore.\u0027),"},{"line_number":110,"context_line":"    cfg.StrOpt(\u0027vmware_ca_file\u0027,"},{"line_number":111,"context_line":"               default\u003dNone,"},{"line_number":112,"context_line":"               help\u003d\u0027CA bundle file to verify vCenter server certificate.\u0027)"},{"line_number":113,"context_line":"]"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":1,"id":"3acd31a7_d95d6f48","line":112,"updated":"2015-05-05 00:36:10.000000000","message":"What if I have a bundle directory?  This doesn\u0027t appear to support that.","commit_id":"39478338bb4a1cbd625a6176d4403bb34a2a0630"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"831574fa6e02d365731f1d34a4dcaa9478535f97","unresolved":false,"context_lines":[{"line_number":109,"context_line":"                    \u0027backup and restore.\u0027),"},{"line_number":110,"context_line":"    cfg.StrOpt(\u0027vmware_ca_file\u0027,"},{"line_number":111,"context_line":"               default\u003dNone,"},{"line_number":112,"context_line":"               help\u003d\u0027CA bundle file to verify vCenter server certificate.\u0027)"},{"line_number":113,"context_line":"]"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":1,"id":"3acd31a7_2e5ad82a","line":112,"in_reply_to":"3acd31a7_d95d6f48","updated":"2015-05-05 01:50:11.000000000","message":"CA bundle file is fine if deployers can pick up their default ca-certificate store using insecure\u003dFalse and ca_file\u003dNone","commit_id":"39478338bb4a1cbd625a6176d4403bb34a2a0630"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"28389a20f71800372597e51722ff8524a430f609","unresolved":false,"context_lines":[{"line_number":1893,"context_line":"                                                 task_poll_interval,"},{"line_number":1894,"context_line":"                                                 wsdl_loc\u003dwsdl_loc,"},{"line_number":1895,"context_line":"                                                 pbm_wsdl_loc\u003dpbm_wsdl,"},{"line_number":1896,"context_line":"                                                 cacert\u003dca_file)"},{"line_number":1897,"context_line":"        return self._session"},{"line_number":1898,"context_line":""},{"line_number":1899,"context_line":"    def _get_vc_version(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3acd31a7_791adb7e","line":1896,"updated":"2015-05-05 00:36:10.000000000","message":"Why not also support insecure option in VMwareAPISession?","commit_id":"39478338bb4a1cbd625a6176d4403bb34a2a0630"},{"author":{"_account_id":8119,"name":"Eric Brown","email":"eric_wade_brown@yahoo.com","username":"ericwb"},"change_message_id":"831574fa6e02d365731f1d34a4dcaa9478535f97","unresolved":false,"context_lines":[{"line_number":1893,"context_line":"                                                 task_poll_interval,"},{"line_number":1894,"context_line":"                                                 wsdl_loc\u003dwsdl_loc,"},{"line_number":1895,"context_line":"                                                 pbm_wsdl_loc\u003dpbm_wsdl,"},{"line_number":1896,"context_line":"                                                 cacert\u003dca_file)"},{"line_number":1897,"context_line":"        return self._session"},{"line_number":1898,"context_line":""},{"line_number":1899,"context_line":"    def _get_vc_version(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3acd31a7_0e3ddcc3","line":1896,"in_reply_to":"3acd31a7_791adb7e","updated":"2015-05-05 01:50:11.000000000","message":"I created this review to address this:\n\nhttps://review.openstack.org/179991","commit_id":"39478338bb4a1cbd625a6176d4403bb34a2a0630"}]}