Steven Sistare <steven.sist...@oracle.com> writes:

> On 12/5/2024 10:37 AM, Markus Armbruster wrote:
>> Steve Sistare <steven.sist...@oracle.com> writes:
>> 
>>> Add the 'cpr' channel type, and stash the incoming cpr channel for use
>>> in a subsequent patch.
>>>
>>> Signed-off-by: Steve Sistare <steven.sist...@oracle.com>
>> [...]
>> 
>>> diff --git a/qapi/migration.json b/qapi/migration.json
>>> index a605dc2..a26960b 100644
>>> --- a/qapi/migration.json
>>> +++ b/qapi/migration.json
>>> @@ -1578,11 +1578,12 @@
>>>  # The migration channel-type request options.
>>>  #
>>>  # @main: Main outbound migration channel.
>>> +# @cpr: cpr state channel.
>>>
>> What does "cpr" stand for?
>
> docs/devel/migration/CPR.rst:  CheckPoint and Restart (CPR)

Suggest something like

     # The migration channel-type request options.
     #
     # @main: Main outbound migration channel.
     #
     # @cpr: Checkpoint and restart state channel

A quick glance at docs/devel/migration/CPR.rst makes me wonder: is that
really *developer* documentation?

Should we have something meant for *users*, too?  QAPI docs could then
link to it.


Reply via email to