Martin Peřina has posted comments on this change.

Change subject: tools: remove log4j usage from engine-config
......................................................................


Patch Set 1:

So, this should be the solution:

1) Add --log-level (default INFO) and --log-file (default console) command line 
args

2) Remove static log4j config in /etc/ovirt-engine/engine-config/log4j.xml

3) Configure log4j dynamically using above args

The same should be applied also to engine-manage-domains.

Agreed?

If so, Eli, will you continue to work on this or should I take over and prepare 
patches?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8cc4eadac817e0527d1286e277b4500f83526c6f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Eli Mesika <[email protected]>
Gerrit-Reviewer: Martin Peřina <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Roy Golan <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to