Bug 1017261

Summary: mingw-virt-viewer exited during repeated migration with large c/p transfer taking place
Product: Red Hat Enterprise Linux 8 Reporter: David Jaša <djasa>
Component: mingw-virt-viewerAssignee: Uri Lublin <uril>
Status: CLOSED ERRATA QA Contact: SPICE QE bug list <spice-qe-bugs>
Severity: low Docs Contact:
Priority: unspecified    
Version: ---CC: ailan, cfergeau, fjin, juzhou, knoel, mdean, mtessun, mxie, rh-spice-bugs, spice-qe-bugs, srevivo, tpelka, tzheng, uril, victortoso, xiaodwan
Target Milestone: rcKeywords: Triaged
Target Release: ---Flags: pm-rhel: mirror+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: mingw-virt-viewer-9.0-3.el8 mingw-spice-gtk-0.38-5.el8 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1418663 (view as bug list) Environment:
Last Closed: 2021-05-18 16:20:32 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Spice RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1418663    
Bug Blocks:    

Description David Jaša 2013-10-09 14:02:56 UTC
Description of problem:
I did a back-and-forth migration of a guest with client to guest copy-paste of large image data (30 MP, 50 MP). During one migration cycle, client exited with these messages (empty lines stripped):
C:\Users\shadowman>"c:\Program Files\VirtViewer\bin\remote-viewer.exe" spice://<IP>:5900/
(remote-viewer.exe:1992): GSpice-WARNING **: Warning no automount-inhibiting implementation available
(remote-viewer.exe:1992): GSpice-CRITICAL **: spice_channel_iterate_read: assertion `c->state != SPICE_CHANNEL_STATE_MIGRATING' failed
(remote-viewer.exe:1992): GSpice-CRITICAL **: spice_channel_iterate_read: assertion `c->state != SPICE_CHANNEL_STATE_MIGRATING' failed
(remote-viewer.exe:1992): GSpice-WARNING **: (../../gtk/spice-session.c:1912):spice_session_channel_new: runtime check failed: (s->playback_channel == NULL)
(remote-viewer.exe:1992): GSpice-CRITICAL **: expected SPICE_MSG_MIGRATE_DATA, got 110
(remote-viewer.exe:1992): GSpice-CRITICAL **: spice_channel_iterate_read: assertion `c->state != SPICE_CHANNEL_STATE_MIGRATING' failed
(remote-viewer.exe:1992): GSpice-CRITICAL **: spice_channel_handle_msg: assertion `base_handlers[type] != NULL' failed


Version-Release number of selected component (if applicable):
mingw-virt-viewer-0.5.6-6.el6_64 (32b)

How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 3 Marc-Andre Lureau 2013-11-07 14:09:58 UTC
raising severity, moving to 3.4? It would be nice to try to fix it in RHEL first, I don't think there is windows specific issues here.

Comment 4 David Blechter 2013-11-08 20:17:32 UTC
(In reply to Marc-Andre Lureau from comment #3)
> raising severity, moving to 3.4? It would be nice to try to fix it in RHEL
> first, I don't think there is windows specific issues here.

can you add the bug in rhel with rhel 6.6 target, and add depend on it in this bug? 

thanks

moving to 3.4 this one

Comment 5 Marc-Andre Lureau 2013-11-08 20:57:31 UTC
(In reply to David Blechter from comment #4)
> (In reply to Marc-Andre Lureau from comment #3)
> > raising severity, moving to 3.4? It would be nice to try to fix it in RHEL
> > first, I don't think there is windows specific issues here.
> 
> can you add the bug in rhel with rhel 6.6 target, and add depend on it in
> this bug? 
> 
ok, but not until we have reproduced on rhel.

Comment 7 Sandro Bonazzola 2015-10-26 12:48:13 UTC
this is an automated message. oVirt 3.6.0 RC3 has been released and GA is targeted to next week, Nov 4th 2015.
Please review this bug and if not a blocker, please postpone to a later release.
All bugs not postponed on GA release will be automatically re-targeted to

- 3.6.1 if severity >= high
- 4.0 if severity < high

Comment 8 David Blechter 2015-11-23 14:01:15 UTC
(In reply to Sandro Bonazzola from comment #7)
> this is an automated message. oVirt 3.6.0 RC3 has been released and GA is
> targeted to next week, Nov 4th 2015.
> Please review this bug and if not a blocker, please postpone to a later
> release.
> All bugs not postponed on GA release will be automatically re-targeted to
> 
> - 3.6.1 if severity >= high
> - 4.0 if severity < high

agree, moving to 4.0

Comment 9 Yaniv Lavi 2016-05-09 11:02:02 UTC
oVirt 4.0 Alpha has been released, moving to oVirt 4.0 Beta target.

Comment 15 Victor Toso 2017-11-30 17:08:45 UTC
Bug 1418663 not fixed in 7.5 and it was moved to 7.6. Moving this to 4.3

Comment 16 Sandro Bonazzola 2017-12-20 13:56:59 UTC
oVirt 4.2.0 has been released on Dec 20th 2017. Please consider re-targeting this bug to next milestone

Comment 17 Sandro Bonazzola 2019-01-28 09:43:38 UTC
This bug has not been marked as blocker for oVirt 4.3.0.
Since we are releasing it tomorrow, January 29th, this bug has been re-targeted to 4.3.1.

Comment 19 Victor Toso 2019-09-03 12:31:36 UTC
Hi, while testing migration I could reproduce similar bug here. I believe that the issue is around authentication actually. RHV issues a password valid for 120s and on migration we can't keep the validation from $src host to $dst host, as far as I can see. This likely would need a new migration protocol, if I'm not mistaken. Does it seem like an important RFE?

If I'm correct, with recent virt-viewer + spice-gtk you would see earlier logs with something like:

> (remote-viewer:20603): GSpice-DEBUG: 11:25:07.415: ../src/channel-main.c:2173 migrate_channel_connect 1:0
...
> (remote-viewer:20603): GSpice-DEBUG: 11:25:07.415: ../src/spice-channel.c:141 0x7e8c50:main-1:0: spice_channel_constructed
...
> (remote-viewer:20603): GSpice-DEBUG: 11:25:07.725: ../src/spice-channel.c:1934 0x7e8c50:main-1:0: switching to tls
> (remote-viewer:20603): GSpice-DEBUG: 11:25:07.725: ../src/spice-channel.c:2680 0x7e8c50:main-1:0: Coroutine exit main-1:0
...
> (remote-viewer:20603): GSpice-DEBUG: 11:25:07.869: ../src/spice-channel.c:2000 0x7e8c50:main-1:0: use mini header: 1
> (remote-viewer:20603): GSpice-DEBUG: 11:25:07.891: ../src/spice-channel.c:1274 0x7e8c50:main-1:0: link result: reply 7
> (remote-viewer:20603): GSpice-DEBUG: 11:25:07.891: ../src/spice-channel.c:2680 0x7e8c50:main-1:0: Coroutine exit main-1:0
> (remote-viewer:20603): GSpice-DEBUG: 11:25:07.891: ../src/spice-channel.c:2871 0x7e8c50:main-1:0: reset 

and then some logs for the other channels like:

> (remote-viewer:20603): GSpice-DEBUG: 11:25:26.493: ../src/spice-channel.c:1066 inputs-3:0: Closing the connection: spice_channel_read() - ret=0

Comment 36 errata-xmlrpc 2021-05-18 16:20:32 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 (spice-client-win bug fix and enhancement update), 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://access.redhat.com/errata/RHEA-2021:1947