[libvirt PATCH] qemu: fix response timeout for agent guest-sync

Michal Prívozník mprivozn at redhat.com
Mon Mar 23 13:46:47 UTC 2020


On 23. 3. 2020 12:33, Daniel Henrique Barboza wrote:
> 
> 
> On 3/20/20 7:28 PM, Jonathon Jongsma wrote:
>> The agent 'guest-sync' command historically had a 5s response timeout
>> which was different from other agent commands, which waited forever.
>> When we added the ability to customize the response timeout for guest
>> agent commands, we intended to continue to use 5s for 'guest-sync' when
>> the user specified a response timeout greater than 5s, and use the
>> user-specified timeout if it was below 5s. Unfortunately, when
>> attempting to determine whether the user-specified timeout was less than
>> 5s, we were comparing against an enum value of
>> VIR_DOMAIN_QEMU_AGENT_COMMAND_DEFAULT (which is -1) rather than against
>> the actual time value that it represented (5).
>>
>> This change makes it so that 'guest-sync' now uses the user-specified
>> tiemout if it is less than 5s.
>>
>> Signed-off-by: Jonathon Jongsma <jjongsma at redhat.com>
>> ---
> 
> 
> Reviewed-by: Daniel Henrique Barboza <danielhb413 at gmail.com>
> 

And puhsed.

Michal




More information about the libvir-list mailing list