Bug 1408190 - [RFE] Update timestamp format in vdsm log to timestamp with timezone
Summary: [RFE] Update timestamp format in vdsm log to timestamp with timezone
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: vdsm
Classification: oVirt
Component: RFEs
Version: 4.20.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ovirt-4.1.1
: 4.19.6
Assignee: Yaniv Bronhaim
QA Contact: Aleksei Slaikovskii
URL:
Whiteboard:
Depends On:
Blocks: oVirt-Metrics-and-Logs
TreeView+ depends on / blocked
 
Reported: 2016-12-22 12:20 UTC by Shirly Radco
Modified: 2017-04-21 09:30 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-04-21 09:30:53 UTC
oVirt Team: Infra
Embargoed:
rule-engine: ovirt-4.1+
rule-engine: exception+
gklein: testing_plan_complete-
ylavi: planning_ack+
mperina: devel_ack+
pstehlik: testing_ack+

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 69846 0 master MERGED Adding TimezoneFormatter to logUtils and use it for default log format 2017-01-24 15:45:11 UTC
oVirt gerrit 72220 0 ovirt-4.1 MERGED Adding TimezoneFormatter to logUtils and use it for default log format 2017-02-15 09:47:15 UTC

Description Shirly Radco 2016-12-22 12:20:31 UTC 
Description of problem:
In order to be able to push the vdsm logs to the new metrics store we require the logs to include the timestamp, so it can be properly stored and later correlated with other logs and metrics.
Comment 1 Shirly Radco 2016-12-22 12:26:10 UTC 
Sorry, The time stamp should include the timezone as mentioned in the header.
Comment 4 Yaniv Kaul 2017-01-08 16:23:18 UTC 
Moving to 4.2, as I seem to recall there was no agreement it could be done for 4.1.
Comment 5 Yaniv Lavi 2017-01-09 09:41:34 UTC 
Martin, didn't we agree on doing this for 4.1, since it very low risk?
Comment 6 Martin Perina 2017-01-09 13:43:28 UTC 
(In reply to Yaniv Dary from comment #5)
> Martin, didn't we agree on doing this for 4.1, since it very low risk?

Yes, this should be included in 4.1 per your request, Yaniv already posted a patch for this issue
Comment 7 Martin Perina 2017-01-18 08:58:56 UTC 
Missed RC build, moving to 4.1.1
Comment 8 Eyal Edri 2017-02-13 13:34:27 UTC 
I don't see 4.1 patch in, how do we know if its in the build or not?
Comment 9 Martin Perina 2017-02-13 14:07:42 UTC 
Patch has not been backported to 4.1 yet, moving back to POST.
Comment 10 Yaniv Bronhaim 2017-02-13 17:31:44 UTC 
sorry.. I recalled we merged it before branching out :| - backported https://gerrit.ovirt.org/#/c/72220/
Comment 11 Aleksei Slaikovskii 2017-02-20 09:02:20 UTC 
Verified on vdsm-4.19.6-1.el7.centos.x86_64
Note You need to log in before you can comment on or make changes to this bug.