Dan Kenigsberg has posted comments on this change. Change subject: hooks: Add hook for fake VM stats ......................................................................
Patch Set 15: Code-Review-1 (2 comments) http://gerrit.ovirt.org/#/c/25927/15/vdsm/vdsmd.8.in File vdsm/vdsmd.8.in: Line 147: after_get_vm_stats and after_get_all_vm_stats are called upon getVmStats and Line 148: getAllVmStats respectively. Both receive a parameter in _hook_json. Line 149: after_get_vm_stats receives a dictionary of the VM stats: Line 150: .nf Line 151: {"vm_id": "...", ... } please have this hook receive the same type as after_get_all_vm_stats - a list of dicts (but here, with only a single dict). The benefit would be that the same very script can be used for both hooks. Line 152: .fi Line 153: after_get_all_vm_stats receives a list of the above dictionaries: Line 154: .nf Line 155: [ http://gerrit.ovirt.org/#/c/25927/15/vdsm_hooks/fakevmstats/Makefile.am File vdsm_hooks/fakevmstats/Makefile.am: Line 23: CLEANFILES = \ Line 24: config.log Line 25: Line 26: EXTRA_DIST = \ Line 27: after_get_all_vm_stats.py the added indentation seems funny. Line 28: Line 29: install-data-local: Line 30: $(MKDIR_P) $(DESTDIR)$(vdsmhooksdir)/after_get_vm_stats Line 31: $(MKDIR_P) $(DESTDIR)$(vdsmhooksdir)/before_get_vm_stats -- 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: 15 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]> Gerrit-Reviewer: Barak Azulay <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Dima Kuznetsov <[email protected]> Gerrit-Reviewer: Douglas Schilling Landgraf <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Vinzenz Feenstra <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: mooli tayer <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
