Change in vdsm[master]: Text: add 'dstqemu' to vds client migration command description

danken at redhat.com danken at redhat.com
Wed Oct 2 09:38:56 UTC 2013


Dan Kenigsberg 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>',
That's a big "if" that starts your sentence. With not verification, no documentation, please.
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 <mtayer at redhat.com>
Gerrit-Reviewer: Better Saggi <bettersaggi at gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: mooli tayer <mtayer at redhat.com>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list