Change in vdsm[master]: hooks: Add hook for fake VM stats

vfeenstr at redhat.com vfeenstr at redhat.com
Thu Mar 27 10:32:18 UTC 2014


Vinzenz Feenstra has posted comments on this change.

Change subject: hooks: Add hook for fake VM stats
......................................................................


Patch Set 9:

(2 comments)

http://gerrit.ovirt.org/#/c/25927/9/vdsm_hooks/fakevmstats/after_get_all_vm_stats.py
File vdsm_hooks/fakevmstats/after_get_all_vm_stats.py:

Line 58: gawk-3.1.7-10.el6.x86_64
Line 59: python-repoze-what-pylons-1.0-4.el6.noarch
Line 60: libtalloc-2.0.7-2.el6.x86_64
Line 61: gnutls-utils-2.8.5-13.el6_5.x86_64
Line 62: python-prioritized-methods-0.2.1-5.1.el6.noarch""".split()
> Vinzenz it's for fake VMs ... so no SSO,
Well SSO is the only thing where I definitely know that it relies on this. Other than that I am not sure. That's also the reason why I did not give a -1 or anything. I just wanted to point this out.
Line 63: 
Line 64: 
Line 65: def createDiskUsage(path, fs, total=None, used=None):
Line 66:     if total is None:


Line 97:         stats['network'][net]['txRate'] = '%.1f' % (random.random() * 100)
Line 98:         stats['network'][net]['txDropped'] = str(random.randint(0, 2*32))
Line 99:         stats['network'][net]['txErrors'] = str(random.randint(0, 2*32))
Line 100: 
Line 101:     # Fake guest-agent reports:
> You'll still need to report the session, I would go with Unknown 
well yes, in this case just report UNKNOWN
Line 102:     if not 'memoryStats' in stats.keys():
Line 103:         stats['memoryStats'] = {}
Line 104:         stats['memoryStats']['mem_total'] = str((2**random.randint(0, 5)) * GB)
Line 105:     memUsed = int(random.random() * int(stats['memoryStats']['mem_total']))


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I6a6d13f43495170a5cb86ae0b0efc1ae85ef05ab
Gerrit-PatchSet: 9
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dima Kuznetsov <dkuznets at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list