)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"d095a4342ce3e4d8a8d4815060b67aa81acfaca0","unresolved":false,"context_lines":[{"line_number":15,"context_line":"adds additional connection info in initialize connection so"},{"line_number":16,"context_line":"that the new connector in os-brick can connect to vCenter and"},{"line_number":17,"context_line":"access the volume vmdk to perform connect and disconnect."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Partial-bug: #1602660"},{"line_number":20,"context_line":"Change-Id: I66a30366ccdae74e13f7a0e35cd7bfdfed2785be"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fad55976_b49f8db3","line":18,"updated":"2016-07-15 16:49:14.000000000","message":"This uses the new vmdk connector, so this should be dependent on the os-brick connector patch.  Then this will have to wait until os_brick with your new connector is released and cinder needs to bump version for os_brick.","commit_id":"4a8a00035cab686da7a402740d7acd7d13344bc5"},{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"8b26f36d61c0b8bded6237780c70aeb659e0f8c2","unresolved":false,"context_lines":[{"line_number":15,"context_line":"adds additional connection info in initialize connection so"},{"line_number":16,"context_line":"that the new connector in os-brick can connect to vCenter and"},{"line_number":17,"context_line":"access the volume vmdk to perform connect and disconnect."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Partial-bug: #1602660"},{"line_number":20,"context_line":"Change-Id: I66a30366ccdae74e13f7a0e35cd7bfdfed2785be"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dada55a8_40fe426c","line":18,"in_reply_to":"fad55976_b49f8db3","updated":"2016-07-18 09:41:26.000000000","message":"Done","commit_id":"4a8a00035cab686da7a402740d7acd7d13344bc5"}],"cinder/volume/drivers/vmware/vmdk.py":[{"author":{"_account_id":14305,"name":"Yuriy Nesenenko","email":"ynesenenko@mirantis.com","username":"yuriy_n"},"change_message_id":"2efe0ff675836178b0ee4233d85e0669a859eaed","unresolved":false,"context_lines":[{"line_number":504,"context_line":"        connection_info[\u0027data\u0027] \u003d {"},{"line_number":505,"context_line":"            \u0027volume\u0027: backing.value,"},{"line_number":506,"context_line":"            \u0027volume_id\u0027: volume[\u0027id\u0027],"},{"line_number":507,"context_line":"            \u0027vmdk_size\u0027: volume[\u0027size\u0027] * units.Gi,"},{"line_number":508,"context_line":"        }"},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"        vmdk_path \u003d self.volumeops.get_vmdk_path(backing)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fad55976_933b869c","line":507,"range":{"start_line":507,"start_character":50,"end_line":507,"end_character":51},"updated":"2016-07-15 15:08:33.000000000","message":"\u0027vmdk_path\u0027: self.volumeops.get_path(backing),\n\u0027datastore\u0027: self.volumeops.get_datastore(backing),\n\u0027datacenter\u0027: self.volumeops.get_dc(backing),\nAll these can be added here. Then the line 510-517 will be rebundent.","commit_id":"4a8a00035cab686da7a402740d7acd7d13344bc5"},{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"09ee5c6f5c042df811929a5cd022ef69874a4542","unresolved":false,"context_lines":[{"line_number":504,"context_line":"        connection_info[\u0027data\u0027] \u003d {"},{"line_number":505,"context_line":"            \u0027volume\u0027: backing.value,"},{"line_number":506,"context_line":"            \u0027volume_id\u0027: volume[\u0027id\u0027],"},{"line_number":507,"context_line":"            \u0027vmdk_size\u0027: volume[\u0027size\u0027] * units.Gi,"},{"line_number":508,"context_line":"        }"},{"line_number":509,"context_line":""},{"line_number":510,"context_line":"        vmdk_path \u003d self.volumeops.get_vmdk_path(backing)"}],"source_content_type":"text/x-python","patch_set":1,"id":"dada55a8_57228eba","line":507,"range":{"start_line":507,"start_character":50,"end_line":507,"end_character":51},"in_reply_to":"fad55976_933b869c","updated":"2016-07-18 09:40:15.000000000","message":"IIUC, it\u0027s only a matter of style. I see your point, but I would prefer not to inline function calls in a dictionary initialization.","commit_id":"4a8a00035cab686da7a402740d7acd7d13344bc5"}]}