Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1228275

Summary: [RFE] Require a specific spice client version when opening console
Product: [Retired] oVirt Reporter: Tomas Jelinek <tjelinek>
Component: ovirt-engine-coreAssignee: Tomas Jelinek <tjelinek>
Status: CLOSED CURRENTRELEASE QA Contact: sefi litmanovich <slitmano>
Severity: medium Docs Contact:
Priority: medium    
Version: 3.6CC: bugs, byount, dblechte, djasa, ecohen, gklein, iheim, istein, lpeer, lsurette, michal.skrivanek, rbalakri, sherold, ssekidde, yeylon
Target Milestone: ---Keywords: FutureFeature
Target Release: 3.6.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: virt
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: 950248
: 1264767 (view as bug list) Environment:
Last Closed: 2015-11-04 11:29:55 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Virt RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1200103, 1202371, 1223469, 1231121, 1295944    
Bug Blocks: 950248, 1264767    

Description Tomas Jelinek 2015-06-04 14:12:41 UTC
Add support for the remote-viewer's "versions" and "newer-version-url" parameters.

Add 2 new fields to the vdc_options also accessible using engine-config tool:
RemoteViewerSupportedVersions and RemoteViewerNewerVersionUrl.


- RemoteViewerSupportedVersions: list of supported versions for OS. Format:
      <os1>:<minimal version supported1>;<os2>:<minimal version supported2>
      example: linux:3.0;windows:2.5
    
- RemoteViewerNewerVersionUrl: when the version check fals, this url is shown to the user by remote-viewer. It supports the following formats:
      + free text (e.g. http://some.url)
      + ${engine_base_url}: a variable replaced by the url on which engine is running (e.g. http://localhost:8080)
      + ${console_client_resources_url}: replaced by the value of the
        "obrand.common.console_client_resources_url" of the branding resource. If
        defined as absolute path, used. If defined as a relative path, it is
        concatinated with the engine base url.
      + any combination of the above (e.g. ${engine_base_url}/my/custom/address)

Comment 1 Max Kovgan 2015-06-28 14:13:33 UTC
ovirt-3.6.0-3 release

Comment 2 sefi litmanovich 2015-07-30 08:11:53 UTC
Verified with ovirt-engine-3.6.0-0.0.master.20150726172446.git65db93d.el6.noarch according to attached polarion test plan.

https://polarion.engineering.redhat.com/polarion/#/project/RHEVM3/testrun?id=3_6_VIRT_Spice_Client_Version_Requirement_2907

Tests on windows client not performed due to unsporting remote-viewer version for windows.

Comment 3 David Jaša 2015-09-11 22:47:21 UTC
I went through the test run and I'm not sure if this we can declare this as implemented/VERIFIED.

The problem is that in all cases, manual update of RemoteViewerSupportedVersions is necessary after each remote viewer version availability in order to trigger the client updates. This is regression from activex's fully automatic updates of all the clients after the update of client available in the engine.

IMO the RemoteViewerSupportedVersions should be set to some special value indicating that the latest versions hosted by the engine instance should be filled by default (e.g. RemoteViewerSupportedVersions=${available_on_engine} for downstream) so that by default, all clients are updated whenever an update is available. The possibility of setting RemoteViewerSupportedVersions to something else should be enough to control the update time in deployments whose administrators wish to do so.

Upsteam could have a different value that would make engine check the upsteam site in some time interval (e.g. daily) and fill that version in .vv file.

Comment 4 Michal Skrivanek 2015-09-17 14:41:11 UTC
That it is perhaps not what we want it to be, that's a separate issue:)
Unfortunately not an easy one to solve. let me elaborate

For Linux we do not have any knowledge of the latest available version. We can perhaps monitor something...when, every day, every week, poll spice-space.org? check RHEL channels? What if they are different on the client

For Windows this is a bit more simple, we only have one build on all Windows and we deliver it (downstream) from the engine host...so we know what is the latest version. But not quite without effort, the current version we can "see" is of SPICE, whereas the application is remote viewer...a different "string" whcih is opaque to us. So not helpful. And upstream we would either have to deliver the client the same way as downstream, or poll some well-known place the same way as for Linux

TL;DR - it deserves a new feature bug and cooperation between multiple teams

the bug is implemented according to description, so I'm moving it back

Comment 5 Sandro Bonazzola 2015-11-04 11:29:55 UTC
oVirt 3.6.0 has been released on November 4th, 2015 and should fix this issue.
If problems still persist, please open a new BZ and reference this one.