Change in vdsm[master]: Add an option to create a watchdog device.
danken at redhat.com
danken at redhat.com
Thu Nov 8 19:10:38 UTC 2012
Dan Kenigsberg has posted comments on this change.
Change subject: Add an option to create a watchdog device.
......................................................................
Patch Set 12: I would prefer that you didn't submit this
(2 inline comments)
Thanks!
Only a minor comment on excessive callback arguments.
....................................................
File vdsm_api/vdsmapi-schema.json
Line 2208: # An enumeration of VM watchdog device types.
Line 2209: #
Line 2210: # @watchdog: Device type watchdog
Line 2211: #
Line 2212: # Since: 4.10.0
we're 4.10.1 already, not that anyone cares
Line 2213: ##
Line 2214: {'enum': 'VmWatchdogDeviceType', 'data': ['watchdog']}
Line 2215:
Line 2216: ##
....................................................
File vdsm/libvirtvm.py
Line 2200: self.startDisksStatsCollection()
Line 2201:
Line 2202: return {'status': doneCode}
Line 2203:
Line 2204: def _onWatchdogEvent(self, name, vmid, action):
why do we need name and vmid as args to this callback? aren't they already in self.conf[vmId] and [vmName]?
ps. vmId is already logged by our SimpleLogAdapter.
Line 2205: def actionToString(action):
Line 2206: # the following action strings come from the comments of
Line 2207: # virDomainEventWatchdogAction in include/libvirt/libvirt.h
Line 2208: # of libvirt source.
--
To view, visit http://gerrit.ovirt.org/7535
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I2b7970a9050ab0279fe03371b9a77692fba30af8
Gerrit-PatchSet: 12
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: ShaoHe Feng <shaohef at linux.vnet.ibm.com>
Gerrit-Reviewer: Adam Litke <agl at us.ibm.com>
Gerrit-Reviewer: Bing Bu Cao <mars at linux.vnet.ibm.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Michael Burns <mburns at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Peter V. Saveliev <peet at redhat.com>
Gerrit-Reviewer: Ryan Harper <ryanh at us.ibm.com>
Gerrit-Reviewer: ShaoHe Feng <shaohef at linux.vnet.ibm.com>
Gerrit-Reviewer: Shu Ming <shuming at linux.vnet.ibm.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: Xu He Jie <xuhj at linux.vnet.ibm.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou at linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list