Bug 1069735
| Summary: | guest not resized on entering full-screen after upgrade to RHEV 3.3 | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Red Hat Enterprise Virtualization Manager | Reporter: | Luca Villa <luvilla> | ||||
| Component: | mingw-virt-viewer | Assignee: | Marc-Andre Lureau <marcandre.lureau> | ||||
| Status: | CLOSED ERRATA | QA Contact: | Desktop QE <desktop-qa-list> | ||||
| Severity: | urgent | Docs Contact: | |||||
| Priority: | urgent | ||||||
| Version: | 3.3.0 | CC: | acathrow, cfergeau, cpelland, dblechte, djasa, iheim, jjongsma, marcandre.lureau, michal.skrivanek, mkalinin, mkrcmari, pmanzell, tpoitras, uril, yeylon | ||||
| Target Milestone: | --- | Keywords: | Regression, ZStream | ||||
| Target Release: | 3.4.0 | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Whiteboard: | spice | ||||||
| Fixed In Version: | mingw-virt-viewer-0.5.6-21.el6_5 | Doc Type: | Bug Fix | ||||
| Doc Text: |
Previously, upgrading a VDI infrastructure to RHEV 3.3, launching the client using Internet Explorer, and setting the display to full-screen caused unexpected behavior. The display did not resize to full-screen. After fixing the problem, the screen resizes properly.
|
Story Points: | --- | ||||
| Clone Of: | |||||||
| : | 1072968 (view as bug list) | Environment: | |||||
| Last Closed: | 2014-06-09 12:51:52 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: | 1009648 | ||||||
| Bug Blocks: | 1072968 | ||||||
| Attachments: |
|
||||||
Comment on attachment 867938 [details]
quick fix
ACK. It looks correct, and I've verified that it fixes the problem.
the pre-3.3 behavior from RHEV is to always send admin_console==true (whereas in 3.3 the behavior is to send true for webadmin and false for user portal)
The relevant piece of code from xpi plugin does the following:
SendValue(CONTROLLER_FULL_SCREEN,
(m_fullscreen == PR_TRUE ? CONTROLLER_SET_FULL_SCREEN : 0) |
(m_admin_console == PR_FALSE ? CONTROLLER_AUTO_DISPLAY_RES : 0));
as confirmed with Jonathon it should be fine for RHEV 3.3 client and upstream SPICE client to again always send admin_console==true (fixes the bug for rhev client, and on upstream spice client the auto_display_res part is being ignored anyway)
This should be verified on all client variations...just to be sure:)
alright, tested the behavior with Marian it doesn't seem as an option anymore
The problem is that by reverting the above change the feature of fullscreen opening on multiple displays breaks.
to summarize the behavior:
REHV 3.3/oVirt, current RHEL/Fedora or Win client:
current - UP - opens on 2 screen, resize doesn't change resolution (this bug)
WA - opens on 1 screen, resize works
proposed - essentially UP would behave as WA
RHEV 3.3/oVirt, testing scratch build client:
current - UP - opens on 2 screen, resize works
WA - opens on 1 screen, resize works
So it seems the proposed change would cause another regression and the only option is to ship/release the fixed client by Marc-Andre
(In reply to Michal Skrivanek from comment #17) > alright, tested the behavior with Marian it doesn't seem as an option anymore > > The problem is that by reverting the above change the feature of fullscreen > opening on multiple displays breaks. > > to summarize the behavior: > REHV 3.3/oVirt, current RHEL/Fedora or Win client: > current - UP - opens on 2 screen, resize doesn't change resolution (this bug) This is exactly how it was in rhevm 3.2, and we should should keep it this way. > WA - opens on 1 screen, resize works > proposed - essentially UP would behave as WA > > > RHEV 3.3/oVirt, testing scratch build client: > current - UP - opens on 2 screen, resize works > WA - opens on 1 screen, resize works > > > So it seems the proposed change would cause another regression and the only > option is to ship/release the fixed client by Marc-Andre We want exact the same behavior connecting to both rhevm 3.2 and rhevm 3.3. No regressions, no improvements. mingw-virt-viewer-0.5.6-20.el6_5 ready, now needs .cab update *** Bug 1038727 has been marked as a duplicate of this bug. *** 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. http://rhn.redhat.com/errata/RHBA-2014-0644.html |
Created attachment 867938 [details] quick fix