Change in vdsm[master]: vm: fix guest agent stats field

fromani at redhat.com fromani at redhat.com
Mon Jun 16 09:11:25 UTC 2014


Francesco Romani has uploaded a new change for review.

Change subject: vm: fix guest agent stats field
......................................................................

vm: fix guest agent stats field

In commit 068fc28d I mistakenly removed an explicit int() conversion
because I overlooked the code, and thought the result was going to
be an int() anyway.

I was wrong, and the result is a float, which breaks the schema which
dictates the affected field to be an uint.

This patch restores the explicit conversion to make the code compliant
with the schema again.

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


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/81/28781/1

diff --git a/vdsm/virt/vm.py b/vdsm/virt/vm.py
index 4b00c51..d66458d 100644
--- a/vdsm/virt/vm.py
+++ b/vdsm/virt/vm.py
@@ -2811,7 +2811,7 @@
                             int(self.conf['memSize']) * 100)
             else:
                 memUsage = 0
-            stats['memUsage'] = utils.convertToStr(memUsage)
+            stats['memUsage'] = utils.convertToStr(int(memUsage))
         return stats
 
     def isMigrating(self):


-- 
To view, visit http://gerrit.ovirt.org/28781
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iab5ab0475e5fdf560ce07cdc1868690d3eceb125
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