Bug 1304755 - admin should have permission to see 'change CD' in admin portal
Summary: admin should have permission to see 'change CD' in admin portal
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: Frontend.WebAdmin
Version: 3.6.3
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: ovirt-3.6.6
: ---
Assignee: Tomas Jelinek
QA Contact: Shira Maximov
URL:
Whiteboard:
Depends On: 1325833
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-02-04 14:29 UTC by Shira Maximov
Modified: 2016-04-26 06:33 UTC (History)
8 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2016-04-26 06:33:31 UTC
oVirt Team: Virt
Embargoed:
mgoldboi: ovirt-3.6.z?
mgoldboi: planning_ack+
oourfali: devel_ack+
mshira: testing_ack?

Attachments (Terms of Use)
Debug of not working vv file (3.16 KB, text/plain)
2016-03-31 16:00 UTC, Shira Maximov 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 53136 0 master MERGED webadmin,userportal: Pass isAdmin flag to UI 2016-02-15 14:34:27 UTC
oVirt gerrit 53530 0 ovirt-engine-3.6 MERGED webadmin,userportal: Pass isAdmin flag to UI 2016-02-15 16:01:35 UTC

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
Note You need to log in before you can comment on or make changes to this bug.