Yaniv Bronhaim has posted comments on this change. Change subject: Text: add 'dstqemu' to vds client migration command description ......................................................................
Patch Set 2: (1 comment) .................................................... File client/vdsClient.py Line 1957: ' name:value,name:value' Line 1958: )), Line 1959: 'migrate': (serv.do_migrate, Line 1960: ('vmId=<id> method=<offline|online> src=<host[:port]> ' Line 1961: 'dst=<host[:port]> dstqemu=<host>', If port can be configurable it should be written here as optional parameter. I see in the wiki "mode" field and wonder why it's not part of the described variables. Anyway, this not a new field, nor related to the patch as far as I understand. Line 1962: 'Migrate a desktop from src machine to dst host using ' Line 1963: 'the specified ports' Line 1964: )), Line 1965: 'migrateStatus': (serv.do_mStat, -- To view, visit http://gerrit.ovirt.org/19277 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ib160ef79f842fb2dc75cfb229f281806bc801464 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: mooli tayer <mta...@redhat.com> Gerrit-Reviewer: Better Saggi <bettersa...@gmail.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: mooli tayer <mta...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches