qmp: Make "id" optional again even in "oob" monitors
authorMarkus Armbruster <armbru@redhat.com>
Tue, 3 Jul 2018 08:53:33 +0000 (10:53 +0200)
committerMarkus Armbruster <armbru@redhat.com>
Tue, 3 Jul 2018 21:18:56 +0000 (23:18 +0200)
commit80cd93bd966bbb0907caa7f1d5676342f27f8f9e
tree34621689d680819983c4c8e7ebfd5a3c222db452
parent2970b4461f5f0dd0798774618713aadd457e8d48
qmp: Make "id" optional again even in "oob" monitors

Commit cf869d53172 "qmp: support out-of-band (oob) execution" made
"id" mandatory for all commands when the client accepted capability
"oob".  This is rather onerous when you play with QMP by hand, and
unnecessarily so: only out-of-band commands need an ID for reliable
matching of response to command.

Revert that part of commit cf869d53172 for now, but have documentation
advise on the need to use "id" with out-of-band commands.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Message-Id: <20180703085358.13941-8-armbru@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
docs/interop/qmp-spec.txt
monitor.c