Change in vdsm[master]: net stats: add tx/rx statistic to getInterfacesStat

[Dan Kenigsberg]

Dan Kenigsberg has posted comments on this change.

Change subject: net stats: add tx/rx statistic to getInterfacesStat
......................................................................


Patch Set 15:

(1 comment)

http://gerrit.ovirt.org/#/c/35061/15/tests/functional/networkTests.py
File tests/functional/networkTests.py:

Line 456:             # running ARP from the interface
Line 457:             rc, out, err = execCmd([_ARPPING_COMMAND.cmd, '-D',
'-I', left,
Line 458:                                     '-c', '1',
IP_ADDRESS_IN_NETWORK])
Line 459:             if rc != 0:
Line 460:                 raise SkipTest('Could not run sysctl command', out,
err)
also, this skiptest string is out of date.
Line 461:             # Sleep 2 seconds for the statistics to be updated
Line 462:             time.sleep(2)
Line 463:             curTxStat = getTxStatsFromInterface(left)
Line 464:             diff = (curTxStat - prevTxStat)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I314a04570dec56677168e0fdb4a7824d299dbd65
Gerrit-PatchSet: 15
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Amir Shladovsky <a.shlado(a)gmail.com&gt;
Gerrit-Reviewer: Amador Pahim <apahim(a)redhat.com&gt;
Gerrit-Reviewer: Antoni Segura Puimedon <toni(a)midokura.com&gt;
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com&gt;
Gerrit-Reviewer: Ido Barkan <ibarkan(a)redhat.com&gt;
Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com&gt;
Gerrit-Reviewer: Petr Horáček <phoracek(a)redhat.com&gt;
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com&gt;
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes