Change in vdsm[master]: vdsm: support VIR_MIGRATE_ABORT_ON_ERROR

danken at redhat.com danken at redhat.com
Wed Jul 31 20:55:29 UTC 2013


Dan Kenigsberg has posted comments on this change.

Change subject: vdsm: support VIR_MIGRATE_ABORT_ON_ERROR
......................................................................


Patch Set 2: I would prefer that you didn't submit this

(4 inline comments)

....................................................
Commit Message
Line 6: 
Line 7: vdsm: support VIR_MIGRATE_ABORT_ON_ERROR
Line 8: 
Line 9: Abort VM migration on EIO by default. The flag is supported
Line 10: since libvirt 1.0.1 upstream, so use getattr() to keep code
I do not see any usage of getattr() (and I don't really want to...), so please fix the doc.
Line 11: compatible with older libvirt versions. In the latter case
Line 12: migration EIO abort will not work.
Line 13: 
Line 14: Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=961154


....................................................
File vdsm_api/vdsmapi-schema.json
Line 5669: # @method:     #optional
Line 5670: #
Line 5671: # @tunneled:   #optional use libvirt's tunneled migration. default it False.
Line 5672: #
Line 5673: # @abort_on_eio:  #optional Abort migration on I/O errors and refuse to migrate
abortOnError is a more conventional name.
Line 5674: #                 VMs, paused due to EIO
Line 5675: #
Line 5676: # @dstqemu:    #optional The destination's host address dedicated for migration.
Line 5677: #


....................................................
File vdsm/vm.py
Line 379:                 # we may return migration stop but it will start at libvirt
Line 380:                 # side
Line 381:                 self._preparingMigrationEvt = False
Line 382:                 if not self._migrationCanceledEvt:
Line 383:                     # Note on VIR_MIGRATE_ABORT_ON_ERROR:
neither do I.
Line 384:                     #
Line 385:                     # The flag is added in libvirt-1.1.0, is available
Line 386:                     # since libvirt-0.10.2-20.el6 in RHEL 6.5, but will not
Line 387:                     # be backported to RHEL 6.4 ever.


Line 385:                     # The flag is added in libvirt-1.1.0, is available
Line 386:                     # since libvirt-0.10.2-20.el6 in RHEL 6.5, but will not
Line 387:                     # be backported to RHEL 6.4 ever.
Line 388:                     #
Line 389:                     # One should not supply "abort_on_eio" having old
So how come True is the default? You have to require that libvirt version if you want to keep this.
Line 390:                     # libvirt versions.
Line 391:                     self._vm._dom.migrateToURI2(
Line 392:                         duri, muri, None,
Line 393:                         libvirt.VIR_MIGRATE_LIVE |


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic7f715c51f28ef2cd01fb95d42553ca10c79ea80
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Peter V. Saveliev <peet at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Peter V. Saveliev <peet at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list