)]}'
{"nova/virt/vmwareapi/vmops.py":[{"author":{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},"change_message_id":"5c00511576f10626bfbd29eb2c845521a2b538d0","unresolved":false,"context_lines":[{"line_number":732,"context_line":"                                    self._session._get_vim(),"},{"line_number":733,"context_line":"                                    vm_ref,"},{"line_number":734,"context_line":"                                    \"VirtualMachine\","},{"line_number":735,"context_line":"                                    \"datastore\")"},{"line_number":736,"context_line":"            if ds_ref_ret is None:"},{"line_number":737,"context_line":"                raise exception.DatastoreNotFound()"},{"line_number":738,"context_line":"            ds_ref \u003d ds_ref_ret.ManagedObjectReference[0]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F9Zj0%3D","side":"PARENT","line":735,"updated":"2013-12-11 22:30:43.000000000","message":"this param is unchanged? Why did you move it?","commit_id":"62e549eb026f0422b23039c4cb0e388475a02e6d"},{"author":{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},"change_message_id":"5c00511576f10626bfbd29eb2c845521a2b538d0","unresolved":false,"context_lines":[{"line_number":729,"context_line":"            # Copy the contents of the VM that were there just before the"},{"line_number":730,"context_line":"            # snapshot was taken"},{"line_number":731,"context_line":"            ds_ref_ret \u003d self._session._call_method("},{"line_number":732,"context_line":"                vim_util, \"get_dynamic_property\", vm_ref, \"VirtualMachine\","},{"line_number":733,"context_line":"                \"datastore\")"},{"line_number":734,"context_line":"            if ds_ref_ret is None:"},{"line_number":735,"context_line":"                raise exception.DatastoreNotFound()"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F9ZjQ%3D","line":732,"updated":"2013-12-11 22:30:43.000000000","message":"if you list the parameters on a single line, this will make a smaller diff and highlight how minimal this change really is.","commit_id":"9e0e371f8846f62aebc543d7c1020c482a35a19a"},{"author":{"_account_id":7629,"name":"Shawn Hartsock","email":"hartsock@acm.org","username":"hartsocks"},"change_message_id":"5c00511576f10626bfbd29eb2c845521a2b538d0","unresolved":false,"context_lines":[{"line_number":736,"context_line":"            ds_ref \u003d ds_ref_ret.ManagedObjectReference[0]"},{"line_number":737,"context_line":"            ds_browser \u003d self._session._call_method("},{"line_number":738,"context_line":"                vim_util, \"get_dynamic_property\", ds_ref, \"Datastore\","},{"line_number":739,"context_line":"                \"browser\")"},{"line_number":740,"context_line":"            # Check if the vmware-tmp folder exists or not. If not, create one"},{"line_number":741,"context_line":"            tmp_folder_path \u003d vm_util.build_datastore_path(datastore_name,"},{"line_number":742,"context_line":"                                                           \"vmware-tmp\")"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F9ZjI%3D","line":739,"updated":"2013-12-11 22:30:43.000000000","message":"most of these parameters on both sides are actually the same. You could probably make this diff really tiny.","commit_id":"9e0e371f8846f62aebc543d7c1020c482a35a19a"}]}