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

bazulay at redhat.com bazulay at redhat.com
Sun Apr 13 08:24:58 UTC 2014


Barak Azulay has posted comments on this change.

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


Patch Set 12:

(1 comment)

http://gerrit.ovirt.org/#/c/25927/12/vdsm/API.py
File vdsm/API.py:

Line 1201:         """
Line 1202:         vms = self.getVMList()
Line 1203:         statsList = []
Line 1204:         for s in vms['vmList']:
Line 1205:             response = VM(s['vmId']).getStats()
> where does the before_* hook being called?
Dan,

This hook is required for scale testing, there is no point to have a hook per vm in this scenario as it will load the system in unnecessary IO per vm (imagine a fake host with 100 fake VMs).
I would leave it as is.
Line 1206:             if response:
Line 1207:                 statsList.append(response['statsList'][0])
Line 1208:         statsList = hooks.after_get_all_vm_stats(statsList)
Line 1209:         return {'status': doneCode, 'statsList': statsList}


-- 
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: 12
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: Federico Simoncelli <fsimonce 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: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list