Francesco Romani has uploaded a new change for review.

Change subject: vm: rename statsAge parameter
......................................................................

vm: rename statsAge parameter

rename statsAge -> stats_age: pep8 friendlier, and now
easy to follow the code flow.
It used to be statsAge in camelCase onyl for legacy reasons.

Change-Id: If0b4601d0f789921c1c575f9ad19852d4ebecacc
Signed-off-by: Francesco Romani <from...@redhat.com>
---
M vdsm/virt/vm.py
1 file changed, 3 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/08/65508/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index f52fbdf..8dec15b 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -4666,21 +4666,21 @@
                       'running' if self._guestCpuRunning else 'stopped',
                       reason)
 
-    def _setUnresponsiveIfTimeout(self, stats, statsAge):
+    def _setUnresponsiveIfTimeout(self, stats, stats_age):
         # we care about timeouts only on the stady state
         if self.lastStatus not in (vmstatus.UP, vmstatus.PAUSED):
             return
         if self.isMigrating():
             return
         # we don't care about decimals here
-        if statsAge < config.getint('vars', 'vm_command_timeout'):
+        if stats_age < config.getint('vars', 'vm_command_timeout'):
             return
         if stats['monitorResponse'] == '-1':
             return
 
         self.log.warning('monitor become unresponsive'
                          ' (command timeout, age=%s)',
-                         statsAge)
+                         stats_age)
         stats['monitorResponse'] = '-1'
 
     def updateNumaInfo(self):


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If0b4601d0f789921c1c575f9ad19852d4ebecacc
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to