diff options
| author | Markus Armbruster <armbru@redhat.com> | 2015-03-05 19:16:58 +0100 |
|---|---|---|
| committer | Markus Armbruster <armbru@redhat.com> | 2015-06-02 09:59:13 +0200 |
| commit | 13cadefbda71e119db79fe0b7a4efd26a6d005bd (patch) | |
| tree | 42f1ae1d3b9c5c8710e00c397daf716d66f887c4 | |
| parent | 84add864ebd2e6f3c645948ab595d8454165ebc5 (diff) | |
monitor: Improve and document client_migrate_info protocol error
Protocol must be spice, vnc isn't implemented. Fix up documentation.
Attempts to use vnc or any other unknown protocol yield the misleading
error message "Invalid parameter 'protocol'". Improve it to
"Parameter 'protocol' expects spice".
Cc: Gerd Hoffmann <kraxel@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by. Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Luiz Capitulino <lcapitulino@redhat.com>
| -rw-r--r-- | hmp-commands.hx | 8 | ||||
| -rw-r--r-- | monitor.c | 2 | ||||
| -rw-r--r-- | qmp-commands.hx | 14 |
3 files changed, 12 insertions, 12 deletions
diff --git a/hmp-commands.hx b/hmp-commands.hx index e864a6ca81..0cf592bcf8 100644 --- a/hmp-commands.hx +++ b/hmp-commands.hx @@ -1011,7 +1011,7 @@ ETEXI .name = "client_migrate_info", .args_type = "protocol:s,hostname:s,port:i?,tls-port:i?,cert-subject:s?", .params = "protocol hostname port tls-port cert-subject", - .help = "send migration info to spice/vnc client", + .help = "set migration information for remote display", .user_print = monitor_user_noop, .mhandler.cmd_new = client_migrate_info, }, @@ -1019,9 +1019,9 @@ ETEXI STEXI @item client_migrate_info @var{protocol} @var{hostname} @var{port} @var{tls-port} @var{cert-subject} @findex client_migrate_info -Set the spice/vnc connection info for the migration target. The spice/vnc -server will ask the spice/vnc client to automatically reconnect using the -new parameters (if specified) once the vm migration finished successfully. +Set migration information for remote display. This makes the server +ask the client to automatically reconnect using the new parameters +once migration finished successfully. Only implemented for SPICE. ETEXI { @@ -1063,7 +1063,7 @@ static int client_migrate_info(Monitor *mon, const QDict *qdict, return 0; } - qerror_report(QERR_INVALID_PARAMETER, "protocol"); + qerror_report(QERR_INVALID_PARAMETER_VALUE, "protocol", "spice"); return -1; } diff --git a/qmp-commands.hx b/qmp-commands.hx index 14e109eb5c..a9768a25b6 100644 --- a/qmp-commands.hx +++ b/qmp-commands.hx @@ -784,23 +784,23 @@ EQMP .name = "client_migrate_info", .args_type = "protocol:s,hostname:s,port:i?,tls-port:i?,cert-subject:s?", .params = "protocol hostname port tls-port cert-subject", - .help = "send migration info to spice/vnc client", + .help = "set migration information for remote display", .mhandler.cmd_new = client_migrate_info, }, SQMP client_migrate_info ------------------- +------------------- -Set the spice/vnc connection info for the migration target. The spice/vnc -server will ask the spice/vnc client to automatically reconnect using the -new parameters (if specified) once the vm migration finished successfully. +Set migration information for remote display. This makes the server +ask the client to automatically reconnect using the new parameters +once migration finished successfully. Only implemented for SPICE. Arguments: -- "protocol": protocol: "spice" or "vnc" (json-string) +- "protocol": must be "spice" (json-string) - "hostname": migration target hostname (json-string) -- "port": spice/vnc tcp port for plaintext channels (json-int, optional) +- "port": spice tcp port for plaintext channels (json-int, optional) - "tls-port": spice tcp port for tls-secured channels (json-int, optional) - "cert-subject": server certificate subject (json-string, optional) |