Alon Bar-Lev has posted comments on this change.

Change subject: aaa: Admin user is not properly synchronized
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/34054/1/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/internal/InternalAuthz.java
File 
backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/internal/InternalAuthz.java:

Line 24:     private ExtMap adminUser;
Line 25: 
Line 26:     private String userName;
Line 27: 
Line 28:     private Properties configuration;
please decide if you want to use configuration all over or specific properties.
Line 29: 
Line 30:     private static class Opaque {
Line 31: 
Line 32:         private boolean firstCall;


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I48f68ac6160e4e8c09526ac23a5180feac43361e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: [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