Bug 983019 - [RFE] add support for disabling spice agent based file transfer support
Summary: [RFE] add support for disabling spice agent based file transfer support
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: RFEs
Version: 3.2.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ovirt-3.6.0-rc
: 3.6.0
Assignee: Francesco Romani
QA Contact: sefi litmanovich
URL:
Whiteboard:
Depends On: 983018 1082479
Blocks: 1302542
TreeView+ depends on / blocked
 
Reported: 2013-07-10 10:02 UTC by Hans de Goede
Modified: 2016-03-09 20:31 UTC (History)
10 users (show)

Fixed In Version: 3.6.0-10
Doc Type: Enhancement
Doc Text:
Previously, there was no way to disable the file transfer feature supported by the SPICE console. In this release, the feature can be disabled on a per virtual machine basis from the Administration Portal.
Clone Of:
Environment:
Last Closed: 2016-03-09 20:31:45 UTC
oVirt Team: Virt
Target Upstream Version:
sherold: Triaged+


Attachments (Terms of Use)
engine + vdsm logs and vm dumpxml (820.52 KB, application/x-gzip)
2015-07-30 08:59 UTC, sefi litmanovich
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2016:0376 0 normal SHIPPED_LIVE Red Hat Enterprise Virtualization Manager 3.6.0 2016-03-10 01:20:52 UTC
oVirt gerrit 23270 0 master ABANDONED vm: spice support to agent file transfer disable Never
oVirt gerrit 26917 0 master MERGED spice: allow to disable the agent file transfer Never
oVirt gerrit 42038 0 master MERGED core: SPICE copyPaste/fileTransfer not passed properly Never
oVirt gerrit 44313 0 None None None Never
oVirt gerrit 44815 0 ovirt-engine-3.6 MERGED core: translate vm graphic settings in specParams Never

Description Hans de Goede 2013-07-10 10:02:52 UTC
Starting with RHEL-6.5 spice-vdagent & virt-viewer, spice has agent based file transfer support. In some environments there might be a desire to turn this off.

In order to facilitate this qemu in RHEL-6.5 will have a new -spice option called disable-agent-file-xfer, see bug 961850. Adding support for this new qemu option to libvirt is tracked in bug 983018.

This bug is for tracking adding support for this to RHEV-M.

Comment 2 Francesco Romani 2014-01-15 17:35:04 UTC
libvirt still lacks support for this option. Patch for libvirt posted on libvir-list on January, 2 and under review; should be merged in libvirt 1.2.2.

Comment 11 sefi litmanovich 2015-05-28 16:07:35 UTC
Hi I wasn't able to verify this RFE on:
ovirt-engine-3.6.0-0.0.master.20150519172219.git9a2e2b3.el6.noarch

steps:
1. create vm
2. start vm with "Enable SPICE file transfer" checked to test sanity.
3. in vdsm.log under VM.create didn't see  fileTransferEnable=false parameter as expeceted.
4. stoped vm.
5. edited vm and unchecked "Enable SPICE file transfer" to disable file transfer.
6. start vm.

result:
under VM.create parameter or when dumping the libvirt xml for the vm couldn't see the fileTransferEnable=false as expected.

Comment 12 Francesco Romani 2015-05-29 06:30:36 UTC
Probably broken by https://gerrit.ovirt.org/#/c/30837/

Comment 13 Tomas Jelinek 2015-06-08 11:08:46 UTC
@Francesco: indeed - the problem is in VmInfoBuilderBase does not check if the SPICE device is there but if there are graphics infos (e.g. for infos about IP, port etc) which is available only after VM starts. And that is too late since we are just about to start the VM. Preparing a patch for it.

Comment 14 Francesco Romani 2015-06-08 11:33:10 UTC
(In reply to Tomas Jelinek from comment #13)
> @Francesco: indeed - the problem is in VmInfoBuilderBase does not check if
> the SPICE device is there but if there are graphics infos (e.g. for infos
> about IP, port etc) which is available only after VM starts. And that is too
> late since we are just about to start the VM. Preparing a patch for it.

Thanks a lot Tomas! I was out of date with the last changes, and I was about to ask some advice, but you were faster :)

Comment 15 Max Kovgan 2015-06-28 14:12:22 UTC
ovirt-3.6.0-3 release

Comment 16 sefi litmanovich 2015-07-30 08:58:49 UTC
Failed to verify with 3.6.0-4: 
ovirt-engine-3.6.0-0.0.master.20150726172446.git65db93d.el6.noarch.

same flow as in comment 11. after unchecking file transfer in vm->console menu, restarting the vm, drag and drop file worked.

attaching a tar with engine + vdsm log and vm's dumpxml after disabling file transfer.

Comment 17 sefi litmanovich 2015-07-30 08:59:53 UTC
Created attachment 1057563 [details]
engine + vdsm logs and vm dumpxml

Comment 18 Francesco Romani 2015-08-03 11:40:16 UTC
(In reply to sefi litmanovich from comment #16)
> Failed to verify with 3.6.0-4: 
> ovirt-engine-3.6.0-0.0.master.20150726172446.git65db93d.el6.noarch.
> 
> same flow as in comment 11. after unchecking file transfer in vm->console
> menu, restarting the vm, drag and drop file worked.
> 
> attaching a tar with engine + vdsm log and vm's dumpxml after disabling file
> transfer.

Thanks for finding this out! The problem here is that Engine is sending per-vm options, while VDSM is expecting the parameters inside the devices specParams.

To put it differently,
VDSM works the new way, Engine works the old way. Patch is ongoing.

Comment 19 Francesco Romani 2015-08-17 08:07:56 UTC
fix 44815 merged on 3.6 branch

Comment 20 sefi litmanovich 2015-11-09 11:59:33 UTC
Verified with rhevm-3.6.0.3-0.1.el6.noarch according to steps in comment 11 and test plan attached in tracker.

See test run to review:
https://polarion.engineering.redhat.com/polarion/#/project/RHEVM3/testrun?id=3%5F6%5FVIRT%5FDisabling%5FSPICE%5FFile%5FTransfer%5F20151109

Comment 22 errata-xmlrpc 2016-03-09 20:31:45 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHEA-2016-0376.html


Note You need to log in before you can comment on or make changes to this bug.