Bug 1109917
Summary: | prefer gf_time_fmt() instead of strftime() | ||
---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Kaleb KEITHLEY <kkeithle> |
Component: | core | Assignee: | Kaleb KEITHLEY <kkeithle> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | mainline | CC: | gluster-bugs |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2015-02-12 20:05:43 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Kaleb KEITHLEY
2014-06-16 15:42:35 UTC
REVIEW: http://review.gluster.org/8085 (core: use gf_time_fmt() instead of localtime()+strftime()) posted (#1) for review on master by Kaleb KEITHLEY (kkeithle) REVIEW: http://review.gluster.org/8085 (core: use gf_time_fmt() instead of localtime()+strftime()) posted (#2) for review on master by Kaleb KEITHLEY (kkeithle) REVIEW: http://review.gluster.org/8085 (core: use gf_time_fmt() instead of localtime()+strftime()) posted (#3) for review on master by Kaleb KEITHLEY (kkeithle) REVIEW: http://review.gluster.org/8085 (core: use gf_time_fmt() instead of localtime()+strftime()) posted (#4) for review on master by Harshavardhana (harsha) COMMIT: http://review.gluster.org/8085 committed in master by Vijay Bellur (vbellur) ------ commit 9797bd197d48ca17a124c245295937ac3d48ab58 Author: Kaleb S. KEITHLEY <kkeithle> Date: Mon Jun 16 12:29:03 2014 -0400 core: use gf_time_fmt() instead of localtime()+strftime() gf_time_fmt() has existed since 3.3; it provides consistent timestamps (i.e. UTC times) throughout the implementation. (BTW, the other name for UTC is GMT.) N.B. many (all?) commercial storage solutions use UTC time for logging. This makes for easier debugging across geographically distributed systems. Also adding a "%s" fmt for portably printing time as simple numeric value on systems regardless of whether 32-bit or 64-bit time_t. Plus a minor tweak to return a ptr to the dest-string to allow gf_time_fmt() to be passed as a param in a *printf(). Someday we should pick the "one true" timestamp format and revise all calls to gf_time_fmt() to use it instead of the five or six different formats. Change-Id: I78202ae14b7246fa424efeea56bf2463e14abfb0 BUG: 1109917 Signed-off-by: Kaleb S. KEITHLEY <kkeithle> Reviewed-on: http://review.gluster.org/8085 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Niels de Vos <ndevos> Reviewed-by: Vijay Bellur <vbellur> I believe this has made it into the 3.6 branch and been released. |