Igor Lvovsky has posted comments on this change.
Change subject: Excluding some VM commands on migration ......................................................................
Patch Set 1: (1 inline comment)
It looks like I'll need to wait for this patch before my hotplug NIC, because I'll need to add same handling as you added in hotplug disk.
.................................................... File vdsm/define.py Line 49: 'migInProgress':{'status': {'code':49, 'message': 'Command not supported during migration'}}, Ah, I already use 49 and 50 in my hotplugNic. One of us will need to change it (depends of push order of course) :)
-- To view, visit http://gerrit.ovirt.org/1389 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I01a4197cdaa746130152a5768e8a3bb0c8b9de2f Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Eduardo ewarszaw@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com