Bug 1304755

Summary: admin should have permission to see 'change CD' in admin portal
Product: [oVirt] ovirt-engine Reporter: Shira Maximov <mshira>
Component: Frontend.WebAdminAssignee: Tomas Jelinek <tjelinek>
Status: CLOSED CURRENTRELEASE QA Contact: Shira Maximov <mshira>
Severity: medium Docs Contact:
Priority: high    
Version: 3.6.3CC: bugs, cfergeau, mgoldboi, michal.skrivanek, mshira, oourfali, sbonazzo, tjelinek
Target Milestone: ovirt-3.6.6Flags: mgoldboi: ovirt-3.6.z?
mgoldboi: planning_ack+
oourfali: devel_ack+
mshira: testing_ack?
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-04-26 06:33:31 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: 1325833    
Bug Blocks:    
Attachments:
Description Flags
Debug of not working vv file none

Description Shira Maximov 2016-02-04 14:29:04 UTC 
Description of problem:
the admin should have permission to see the 'change CD' in the admin portal,
should be  admin=1 and not  admin=0.

Version-Release number of selected component (if applicable):
 Red Hat Enterprise Virtualization Manager Version: 3.6.3-0.1.el6 

How reproducible:
100%

Steps to Reproduce:
1. create a vm 
2.open console - if the 'change CD' is not available is because there is no permission for the admin
3.

Actual results:


Expected results:


Additional info:
Comment 1 Vojtech Szocs 2016-02-05 14:14:12 UTC 
This is a bug where UI doesn't receive proper user info, which results in passing `admin=0` within console.vv file, regardless if the user is an admin or not.
Comment 2 Red Hat Bugzilla Rules Engine 2016-02-06 14:11:05 UTC 
This bug is not marked for z-stream, yet the milestone is for a z-stream version, therefore the milestone has been reset.
Please set the correct milestone or add the z-stream flag.
Comment 4 Yaniv Kaul 2016-02-25 12:03:58 UTC 
This does not block any RFE, removing the artificial relationship.
Comment 5 Shira Maximov 2016-03-31 16:00:41 UTC 
Created attachment 1142294 [details]
Debug of not working vv file

Failed to verify on :  Red Hat Enterprise Virtualization Manager Version: 3.6.5-0.1.el6 

verification steps:
1. Create a new VM 
2. open VM console- check if change 'CD' tab appears -> The 'Change CD' didn't appear 
3. open the .vv file and check if admin=1 or admin=0 -> admin=1 as expected
4. add permission to admin and check again the 'change CD' tab -> the 'change CD' appears 


I also compared one working and one that not, and the only differences are:
the port number, the password, title, tls-port and vm-guid
Comment 6 Red Hat Bugzilla Rules Engine 2016-04-04 06:17:17 UTC 
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.
Comment 7 Vojtech Szocs 2016-04-04 13:56:28 UTC 
Hm, I don't see any obvious errors in attached log file, seems strange to me.

The UI bug of not having proper `admin` flag in .vv file has been fixed and verified. There must be some other issue.
Comment 8 Oved Ourfali 2016-04-06 10:36:20 UTC 
Moving to virt for further testing.
I suggest also to move to 3.6.6.
Comment 9 Michal Skrivanek 2016-04-18 12:12:44 UTC 
can you please update whether it works after upgrading your client?
Comment 10 Shira Maximov 2016-04-18 12:31:52 UTC 
I upgraded my client to fedora 23, in order to have virt-viewer-3.0-1.fc23.x86_64
and now it's working fine.
you can close this bug.
Comment 11 Tomas Jelinek 2016-04-26 06:33:31 UTC 
It has been fixed in 3.6.5, the problem was the incorrect version of remote-viewer, closing