gluster / gluster-block

A framework for gluster block storage
GNU General Public License v2.0
74 stars 32 forks source link

Print log #242

Closed pkalever closed 5 years ago

pkalever commented 5 years ago

What does this PR achieve? Why do we need it?

Problem:

Right now gluster-block-bricks-start.log and gluster-block-upgrade-activities.log gets log timestamp in 12 hours format.

Solution:

PrintLog is using 0..11 i.e. %I instead of 0..23 i.e. we should use %H for printing hours.

Does this PR fix issues?

Fixes: #241 reported: BZ#1720783

Notes for the reviewer

Credits to Pranith Kumar Karampuri pkarampu@redhat.com for chasing this out.

pkalever commented 5 years ago

@pranithk @lxbsz fixed this now, could you please take a look. Thanks!

pkalever commented 5 years ago

Updated tags, thanks @pranithk @lxbsz @amarts for the review.