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

Reply via email to