[Kimchi-devel] [PATCH v2][Kimchi 0/2] Allow vm update graphics type

Aline Manera alinefm at linux.vnet.ibm.com
Tue Aug 30 16:49:50 UTC 2016



On 08/30/2016 01:43 PM, Ramon Medeiros wrote:
> Just retest right now to make sure.
>
> I have a opensuse vm running on spice.
>
> 1) turned off it
> 2) Run PUT on REST API to update it to vnc
> 3) Turn on
> 4) opened console on vnc without issues.
> 5) turned off again
> 6) Switch to spice
> 7) running without issues again
>
> Tested on my laptop: Fedora 23 x86_64
>

Please, check if the guest XML does not have any leftover related to 
those console type changes.

> On 08/30/2016 01:23 PM, Aline Manera wrote:
>> Hi Ramon,
>>
>> Maybe something else is needed here.
>>
>> When I have a guest configured with SPICE console, the guest XML is 
>> something like below:
>>
>>     <graphics type='spice' autoport='yes' listen='127.0.0.1'>
>>     <channel type='spicevmc'>
>>       <target type='virtio' name='com.redhat.spice.0'/>
>>     </channel>
>>
>> And for a guest with VNC console, the guest XML has:
>>
>> <graphics type='vnc' port='-1' autoport='yes' listen='127.0.0.1'>
>>
>> From this patch, I am seeing you changing only the type parameter in 
>> the graphics tag.
>>
>> Shouldn't the <channel type='spicevmc'>  node be removed when 
>> changing from SPICE to VNC?
>> What about the port number needed for VNC configuration?
>>
>> Is all that being done automatically through libvirt? Have you 
>> confirmed that the console kept working after changing the console type?
>>
>> Regards,
>> Aline Manera
>>
>> On 08/23/2016 01:47 PM, Ramon Medeiros wrote:
>>> Changes:
>>>
>>> v2:
>>> Fix typos
>>> Add tests
>>>
>>> Ramon Medeiros (2):
>>>    Issue #836: Allow user change guest graphics type
>>>    Create test to verify graphics type change
>>>
>>>   API.json            |  6 ++++++
>>>   docs/API.md         |  1 +
>>>   i18n.py             |  2 +-
>>>   model/vms.py        | 10 ++++++++++
>>>   tests/test_model.py | 18 +++++++-----------
>>>   5 files changed, 25 insertions(+), 12 deletions(-)
>>>
>>
>




More information about the Kimchi-devel mailing list