)]}'
{"nova/virt/libvirt/driver.py":[{"author":{"_account_id":1711,"name":"Yun Mao","email":"yunmao@gmail.com","username":"yunmao"},"change_message_id":"0029ef7434900e35b0d02a09392ec6e47fab18da","unresolved":false,"context_lines":[{"line_number":584,"context_line":"                container_dir \u003d os.path.join(CONF.instances_path,"},{"line_number":585,"context_line":"                                             instance[\u0027name\u0027],"},{"line_number":586,"context_line":"                                             \u0027rootfs\u0027)"},{"line_number":587,"context_line":"                disk.destroy_container(container_dir\u003dcontainer_dir)"},{"line_number":588,"context_line":"            if os.path.exists(target):"},{"line_number":589,"context_line":"                # If we fail to get rid of the directory"},{"line_number":590,"context_line":"                # tree, this shouldn\u0027t block deletion of"}],"source_content_type":"text/x-python","patch_set":17,"id":"AAAAM3%2F%2FuGw%3D","line":587,"updated":"2013-01-08 19:05:54.000000000","message":"nova/virt/libvirt/driver.py:587: [E1101, LibvirtDriver._cleanup] Module \u0027nova.virt.disk.api\u0027 has no \u0027destroy_container","commit_id":"15011f4d0760de8d20c7399f8daeccee7af6c626"}],"nova/virt/vmwareapi/vmops.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"1bcc7f84c27d4cb234ddb92c3dfef2843a9ac24f","unresolved":false,"context_lines":[{"line_number":604,"context_line":"                        self._session._get_vim(),"},{"line_number":605,"context_line":"                        \"DeleteDatastoreFile_Task\","},{"line_number":606,"context_line":"                        self._session._get_vim().get_service_content()."},{"line_number":607,"context_line":"                        fileManager, name\u003ddir_ds_compliant_path)"},{"line_number":608,"context_line":"                    self._session._wait_for_task(instance[\u0027uuid\u0027], delete_task)"},{"line_number":609,"context_line":"                    LOG.debug(_(\"Deleted contents of the VM from \""},{"line_number":610,"context_line":"                                \"datastore %(datastore_name)s\") %"}],"source_content_type":"text/x-python","patch_set":13,"id":"AAAAM3%2F%2Fy3o%3D","line":607,"updated":"2013-01-04 20:27:21.000000000","message":"This makes the line less comprehensible to me.  I\u0027d prefer if you set up a temporary variable, i.e.:\n\n    vim \u003d self._session._get_vim()\n    delete_task \u003d self._session._call_method(\n        vim,\n        \"DeleteDatastoreFile_Task\",\n        vim.get_service_content().fileManager,\n        name\u003ddir_ds_compliant_path)","commit_id":"329ea6688b91c0499a6d7e233966d14a7e6228cb"},{"author":{"_account_id":1782,"name":"Kravchenko Pavel","email":"kpavel@il.ibm.com","username":"kravchenko-pavel"},"change_message_id":"75f290ecafbab9c4c3cab320211b9e3180f50936","unresolved":false,"context_lines":[{"line_number":604,"context_line":"                        self._session._get_vim(),"},{"line_number":605,"context_line":"                        \"DeleteDatastoreFile_Task\","},{"line_number":606,"context_line":"                        self._session._get_vim().get_service_content()."},{"line_number":607,"context_line":"                        fileManager, name\u003ddir_ds_compliant_path)"},{"line_number":608,"context_line":"                    self._session._wait_for_task(instance[\u0027uuid\u0027], delete_task)"},{"line_number":609,"context_line":"                    LOG.debug(_(\"Deleted contents of the VM from \""},{"line_number":610,"context_line":"                                \"datastore %(datastore_name)s\") %"}],"source_content_type":"text/x-python","patch_set":13,"id":"AAAAM3%2F%2Fx08%3D","line":607,"in_reply_to":"AAAAM3%2F%2Fy3o%3D","updated":"2013-01-06 11:16:34.000000000","message":"Done","commit_id":"329ea6688b91c0499a6d7e233966d14a7e6228cb"}]}