Change in vdsm[master]: WIP: fix again stats_cache cleaning

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 2: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1625/ : FAILURE

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18394/ : SUCCESS

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: WIP: fix again stats_cache cleaning

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 2:

Build Started (2/2) -> 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18394/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: WIP: fix again stats_cache cleaning

2015-04-30 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 2: -Verified

Build Started (1/2) -> 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1625/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: WIP: fix again stats_cache cleaning

2015-04-30 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 2:

* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 
'ovirt-3.4', 'ovirt-3.3'])

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: WIP: fix again stats_cache cleaning

2015-04-29 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 1: Verified-1

Build Failed 

http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18351/ : SUCCESS

http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1581/ : FAILURE

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: WIP: fix again stats_cache cleaning

2015-04-29 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 1:

Build Started (2/2) -> 
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/1581/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: WIP: fix again stats_cache cleaning

2015-04-29 Thread oVirt Jenkins CI Server
oVirt Jenkins CI Server has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 1:

Build Started (1/2) -> 
http://jenkins.ovirt.org/job/vdsm_master_pep8_gerrit/18351/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: WIP: fix again stats_cache cleaning

2015-04-29 Thread automation
automat...@ovirt.org has posted comments on this change.

Change subject: WIP: fix again stats_cache cleaning
..


Patch Set 1:

* Update tracker::IGNORE, no Bug-Url found
* Check Bug-Url::WARN, no bug url found, make sure header matches 'Bug-Url: ' 
and is a valid url.
* Check merged to previous::IGNORE, Not in stable branch (['ovirt-3.5', 
'ovirt-3.4', 'ovirt-3.3'])

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches


Change in vdsm[master]: WIP: fix again stats_cache cleaning

2015-04-29 Thread fromani
Francesco Romani has uploaded a new change for review.

Change subject: WIP: fix again stats_cache cleaning
..

WIP: fix again stats_cache cleaning

Change-Id: I98d388b4684be61936316ef749707e4be418af94
Signed-off-by: Francesco Romani 
---
M vdsm/virt/vm.py
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/98/40398/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 3c3efc7..65e9bad 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -1590,6 +1590,7 @@
 utils.rmFile(self._recoveryFile)
 self._guestSockCleanup(self._qemuguestSocketFile)
 self._reattachHostDevices()
+sampling.stats_cache.remove(self.id)
 
 def _isDomainRunning(self):
 try:
@@ -3447,7 +3448,6 @@
 self.lastStatus = vmstatus.POWERING_DOWN
 # Terminate the VM's creation thread.
 self._incomingMigrationFinished.set()
-sampling.stats_cache.remove(self.id)
 self.guestAgent.stop()
 if self._dom:
 result = self._destroyVmGraceful()


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I98d388b4684be61936316ef749707e4be418af94
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani 
___
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches