Oved Ourfali has posted comments on this change.

Change subject: engine: Maintain NetworkUser role id for VnicProfileUser
......................................................................


Patch Set 2: Code-Review+2

(1 comment)

See comment. If it has no meaning then go ahead and merge it.

....................................................
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/auth/ApplicationGuids.java
Line 12:     superUser(new Guid("00000000-0000-0000-0000-000000000001")), 
//$NON-NLS-1$
Line 13:     userTemplateBasedVM(new 
Guid("def00009-0000-0000-0000-def000000009")), //$NON-NLS-1$
Line 14:     quotaConsumer(new Guid("def0000a-0000-0000-0000-def00000000a")), 
//$NON-NLS-1$
Line 15:     dataCenterAdmin(new Guid("def00002-0000-0000-0000-def000000002")), 
//$NON-NLS-1$
Line 16:     vnicProfileUser(new Guid("DEF0000A-0000-0000-0000-DEF000000010")), 
//$NON-NLS-1$
it is all lower case here. Just make sure it has no real meaning in the 
different flows.
Line 17: 
Line 18:     // ad_groups
Line 19:     everyone(new Guid("eee00000-0000-0000-0000-123456789eee")); 
//$NON-NLS-1$
Line 20: 


-- 
To view, visit http://gerrit.ovirt.org/18003
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I2ebd6c4da455696f82fa8f53cfcb35d8c97a479f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to