Martin Peřina has posted comments on this change. Change subject: tools: Remove logging when parsing args in engine-manage-domains ......................................................................
Patch Set 2: (1 comment) http://gerrit.ovirt.org/#/c/26640/2/backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/tools/ManageDomainsArguments.java File backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/tools/ManageDomainsArguments.java: Line 412 Line 413 Line 414 Line 415 Line 416 > but here you removed an error message, please change the commit msg. Yes, this error is printed only to console on next line, because we don't have valid logging setup at this moment. -- To view, visit http://gerrit.ovirt.org/26640 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b1c5e76e6dbd2bc2c3d49abac03e10d4dc6faf6 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <[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: 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
