Francesco Romani has uploaded a new change for review.

Change subject: vm: make deleteVm private
......................................................................

vm: make deleteVm private

The Vm.deleteVm() method is never used outside the Vm class,
so it should be _private.

Change-Id: I217637023185ec428cb66dfb8fe93b6c3bb98c32
Signed-off-by: Francesco Romani <[email protected]>
---
M vdsm/virt/vm.py
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/84/51384/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 50ea74a..b278b87 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -3878,7 +3878,7 @@
             return response.error('destroyErr')
         return {'status': doneCode}
 
-    def deleteVm(self):
+    def _deleteVm(self):
         """
         Clean VM from the system
         """
@@ -3896,7 +3896,7 @@
         if response.is_error(result):
             return result
         # Clean VM from the system
-        self.deleteVm()
+        self._deleteVm()
 
         return response.success()
 


-- 
To view, visit https://gerrit.ovirt.org/51384
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I217637023185ec428cb66dfb8fe93b6c3bb98c32
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to