Change in vdsm[ovirt-3.3]: vm iface stats: fix arithmetic

ybronhei at redhat.com ybronhei at redhat.com
Thu Mar 27 14:38:32 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: vm iface stats: fix arithmetic
......................................................................


vm iface stats: fix arithmetic

My commit 672ff43 has a serious arithmetic flaw: the modulu must be
taken on the sample difference!

Bug-Url: https://bugzilla.redhat.com/1064489
Change-Id: Ie356d5f96c73b65925b38f6149d5eb36f43023ad
Signed-off-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/25960
Reviewed-by: Antoni Segura Puimedon <asegurap at redhat.com>
Tested-by: Martin Pavlik <xpavli44 at gmail.com>
(cherry picked from commit d0c9357bbf6b6c61dc633edba6691a53b592e895)
Reviewed-on: http://gerrit.ovirt.org/26091
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M vdsm/vm.py
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, approved
  Antoni Segura Puimedon: Looks good to me, but someone else must approve
  Dan Kenigsberg: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie356d5f96c73b65925b38f6149d5eb36f43023ad
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.3
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Martin Pavlik <xpavli44 at gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>


More information about the vdsm-patches mailing list