Bug 1428102 - libglusterfs: Change gmtime_r -> localtime_r for better log readability
Summary: libglusterfs: Change gmtime_r -> localtime_r for better log readability
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: GlusterFS
Classification: Community
Component: core
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-03-01 19:07 UTC by Vijay Bellur
Modified: 2019-04-17 10:28 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-04-17 10:28:58 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Vijay Bellur 2017-03-01 19:07:21 UTC
libglusterfs: Change gmtime_r -> localtime_r for better log readability

Summary:
- We should be logging in our localtime, not in GMT. Makes our logs more readable.
- This is a cherry-pick of D2464690 to 3.8.

Signature: t1:2464690:1442877302:72fa0a0d4f26569e5641c6581c352c1de32f163e

Change-Id: I046158ff91b25c8a80dd7632eece9832f691a98d
Signed-off-by: Shreyas Siravara <sshreyas>
Reviewed-on: https://review.gluster.org/16337
Reviewed-by: Kevin Vigor <kvigor>
CentOS-regression: Gluster Build System <jenkins.org>
NetBSD-regression: NetBSD Build System <jenkins.org>
Smoke: Gluster Build System <jenkins.org>

Comment 1 Worker Ant 2017-03-01 20:54:43 UTC
REVIEW: https://review.gluster.org/16818 (libglusterfs: Change gmtime_r -> localtime_r for better log readability) posted (#1) for review on master by Vijay Bellur (vbellur)

Comment 2 Joe Julian 2017-06-13 17:38:33 UTC
I disagree. When we used to do localtime collating logs between clients and servers was often impossible to get a user to do. The change to always using gmtime everywhere has made debugging much easier.

Comment 3 Yaniv Kaul 2019-04-17 10:28:58 UTC
(In reply to Joe Julian from comment #2)
> I disagree. When we used to do localtime collating logs between clients and
> servers was often impossible to get a user to do. The change to always using
> gmtime everywhere has made debugging much easier.

Closing, as the patch was also not accepted.


Note You need to log in before you can comment on or make changes to this bug.