Change in vdsm[master]: vm: do not clean StatsCache on setDownStatus

fromani at redhat.com fromani at redhat.com
Fri Apr 24 15:11:45 UTC 2015


Francesco Romani has uploaded a new change for review.

Change subject: vm: do not clean StatsCache on setDownStatus
......................................................................

vm: do not clean StatsCache on setDownStatus

We should clean StatsCache, but only when a VM
is released, not when it is set to Down.

Otherwise, destroy will fail, leaving VDSM
into an inconsistent state.

Change-Id: I3a511afa7a03f7f3d63409372f21060c2eaa21a6
Signed-off-by: Francesco Romani <fromani at redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 0 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/23/40223/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 25861a1..f8250ea 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -1227,7 +1227,6 @@
             self.guestAgent.stop()
         except Exception:
             pass
-        sampling.stats_cache.remove(self.id)
         self.saveState()
 
     def status(self, fullStatus=True):


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I3a511afa7a03f7f3d63409372f21060c2eaa21a6
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>


More information about the vdsm-patches mailing list