Change in ovirt-engine[ovirt-engine-4.1]: tools: Add time zone to log records

Code Review gerrit at ovirt.org
Tue Feb 14 14:41:00 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: tools: Add time zone to log records
......................................................................


tools: Add time zone to log records

1. Introduces TimeZoneBasedFormatter which bypasses inability of
   java.util.logging framework to specify different time zone used to
   format date/time part of the log record
2. Adds time zone to time stamp to log records for
   ovirt-engine-notifier, engine-config and
   ovirt-engine-extensions-tool

Change-Id: I1e0704c0b07b2c7e233a134f137a89db7fbc6f6e
Bug-Url: https://bugzilla.redhat.com/1408193
Signed-off-by: Martin Perina <mperina at redhat.com>
---
M backend/manager/logutils/src/main/java/org/ovirt/engine/core/logutils/JavaLoggingUtils.java
A backend/manager/logutils/src/main/java/org/ovirt/engine/core/logutils/TimeZoneBasedFormatter.java
M packaging/conf/extensions-tool-logging.properties
M packaging/conf/notifier-logging.properties.in
M packaging/conf/tools-logging.properties
5 files changed, 137 insertions(+), 10 deletions(-)

Approvals:
  Martin Peřina: Verified; Looks good to me, approved
  Juan Hernandez: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/72208
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I1e0704c0b07b2c7e233a134f137a89db7fbc6f6e
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.1
Gerrit-Owner: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list