Martin Peřina has posted comments on this change.

Change subject: tools: Replace oVirt logger with slf4j as frontend in 
engine-manage-domains
......................................................................


Patch Set 5:

(1 comment)

http://gerrit.ovirt.org/#/c/32851/5/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/utils/dns/DnsSRVLocator.java
File 
backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/utils/dns/DnsSRVLocator.java:

Line 18
Line 19
Line 20
Line 21
Line 22
> so can we remove this class?
Removal of org.ovirt.engine.core.utils.log will be part of later backend only 
patches


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I82996f8f6ad3b7111b5bb36d46a395d94dd70e80
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Peřina <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Martin Peřina <[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