Change in vdsm[master]: virt: migration: create full status in migration.py

mmirecki at redhat.com mmirecki at redhat.com
Thu Jul 16 07:51:56 UTC 2015


Marcin Mirecki has posted comments on this change.

Change subject: virt: migration: create full status in migration.py
......................................................................


Patch Set 5:

The status of the migration is checked by MigrateStatusVDSCommand.
The best way is probably to debug this:
Place an hook script in before_vm_migrate_source which prints something to std err and returns a 2 exit code.

The reply from the server (in processed form) can be seen here:
MigrateStatusReturnForXmlRpc line: 17
variables: innerMap.responseMap - contains the reply: {status={message=Hook error: <error message>, code=78}}

Handling of the error, logging, and converting vdsm error to engine exception
BrokerCommandBase line:36
note that the error is reported as 16 (EngineError.unexpected), as the engine does not yet have a specific error for this (should probably be added once the vdsm hook error patch is in).

-- 
To view, visit https://gerrit.ovirt.org/43357
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Icbc1525f52dc5ee49647aa6d690f221f2dc96e5d
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: No


More information about the vdsm-patches mailing list