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 <mmire...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmire...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: No
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to