)]}'
{"cinder/volume/drivers/vmware/volumeops.py":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"6c0242617beefdde40d7504a1473572cc8ac1613","unresolved":false,"context_lines":[{"line_number":366,"context_line":"        self._session.invoke_api(vim_util, \u0027cancel_retrieval\u0027,"},{"line_number":367,"context_line":"                                 self._session.vim, retrieve_result)"},{"line_number":368,"context_line":""},{"line_number":369,"context_line":"    # TODO(vbala): move this method to datastore module"},{"line_number":370,"context_line":"    def _is_usable(self, mount_info):"},{"line_number":371,"context_line":"        \"\"\"Check if a datastore is usable as per the given mount info."},{"line_number":372,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"dab17558_2794eae0","line":369,"updated":"2016-05-06 21:58:53.000000000","message":"why are you just adding a TODO in this patch and not moving it ?","commit_id":"12afec15f265e0c3abf681a6c295987017dab779"},{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"84e403559df9dba8b37e079f00c6020344bf33ec","unresolved":false,"context_lines":[{"line_number":366,"context_line":"        self._session.invoke_api(vim_util, \u0027cancel_retrieval\u0027,"},{"line_number":367,"context_line":"                                 self._session.vim, retrieve_result)"},{"line_number":368,"context_line":""},{"line_number":369,"context_line":"    # TODO(vbala): move this method to datastore module"},{"line_number":370,"context_line":"    def _is_usable(self, mount_info):"},{"line_number":371,"context_line":"        \"\"\"Check if a datastore is usable as per the given mount info."},{"line_number":372,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"dab17558_d0e8287f","line":369,"in_reply_to":"dab17558_2794eae0","updated":"2016-05-06 23:14:49.000000000","message":"This is also used by get_connected_hosts which is called from is_datastore_accessible and is_datastore_accessible is called from vmdk module. These methods are datastore related and should be in datastore module, but I thought it is better to have a separate patch for that refactoring.","commit_id":"12afec15f265e0c3abf681a6c295987017dab779"}]}