ShaoHe Feng has posted comments on this change.
Change subject: dump the core of a VM
......................................................................
Patch Set 10: (2 inline comments)
....................................................
File vdsm/API.py
Line 283:
Line 284: dumpParams.update(params)
Line 285: exclusiveParams = ("live", "crash",
"reset")
Line 286: if [dumpParams[x] for x in exclusiveParams].count(True) > 1:
Line 287: msgstr = "'reset', 'crash', and 'live'
are mutually exclusive."
agree. "core dump failed" can indicate failure explicitly.
Line 288: self.log.debug("Core dump failed. " + msgstr)
Line 289: return reportError(msg=msgstr)
Line 290:
Line 291: v = self._cif.vmContainer.get(self._UUID)
....................................................
File vdsm_cli/vdsClient.py
Line 2416: 'coreDump': (serv.coreDump,
Line 2417: ('<vmId> <file> [live=<True|False>]
'
Line 2418: '[crash=<True|False>]
[bypass-cache=<True|False>] '
Line 2419: '[reset=<True|False>]
[memory-only=<True|False>]',
Line 2420: "get memeory dump or migration file"
surely. It does be helpful for user to avoid the mutual exclusivity operation.
Line 2421: 'optional params:',
Line 2422: 'crash: crash the domain after core
dump'
Line 2423: 'default False',
Line 2424: 'live: perform a live core dump if
supported, '
--
To view, visit
http://gerrit.ovirt.org/7329
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If4aac9e747dc7aa64a6ff5ef256a7a4375aa2bb5
Gerrit-PatchSet: 10
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: ShaoHe Feng <shaohef(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: Ryan Harper <ryanh(a)us.ibm.com>
Gerrit-Reviewer: ShaoHe Feng <shaohef(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Shu Ming <shuming(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com>
Gerrit-Reviewer: Zhou Zheng Sheng <zhshzhou(a)linux.vnet.ibm.com>
Gerrit-Reviewer: oVirt Jenkins CI Server