[LU-16231] Lustre stats header incorrectly using boot time Created: 11/Oct/22  Updated: 20/Dec/22  Resolved: 11/Dec/22

Status: Resolved
Project: Lustre
Component/s: None
Affects Version/s: Lustre 2.15.0
Fix Version/s: Lustre 2.16.0, Lustre 2.15.2

Type: Bug Priority: Major
Reporter: Andreas Dilger Assignee: Andreas Dilger
Resolution: Fixed Votes: 0
Labels: None

Issue Links:
Related
is related to LU-11407 Improve stats data Resolved
is related to LU-15642 restore server read/write latency mea... Open
is related to LU-16228 create lljobstats command Resolved
Severity: 3
Rank (Obsolete): 9223372036854775807

 Description   

Since the landing of patch https://review.whamcloud.com/33201 "LU-11407 obdclass: add start time to stats files" the "snapshot_time" in the Lustre stats files has incorrectly been reporting the timestamp relative to the system boot rather than the Unix wallclock time. This should be reporting the wallclock time so that the stats can be correlated across nodes more easily.

In some cases this may not be noticed, because the monitor is only comparing the relative time interval between samples, and/or records its own timestamp when the stats are collected, but this should be fixed.



 Comments   
Comment by Gerrit Updater [ 11/Oct/22 ]

"Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/48821
Subject: LU-16231 misc: fix stats snapshot_time to use wallclock
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 0ef5ec5cf7a884d86bf175af64b2245897ca6b01

Comment by Gerrit Updater [ 13/Oct/22 ]

"Andreas Dilger <adilger@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/48847
Subject: LU-16231 misc: rename lprocfs_stats functions
Project: fs/lustre-release
Branch: master
Current Patch Set: 1
Commit: 318cdd29423431730e72a2ed9c794360aac7181b

Comment by Gerrit Updater [ 23/Nov/22 ]

"Feng Lei <flei@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/49220
Subject: LU-16231 misc: fix stats snapshot_time to use wallclock
Project: fs/lustre-release
Branch: b2_15
Current Patch Set: 1
Commit: 1796c70079eb995691a39d70db0300436d068980

Comment by Gerrit Updater [ 11/Dec/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48821/
Subject: LU-16231 misc: fix stats snapshot_time to use wallclock
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: e42efe35eec7b9725f7f4fff86aaee04093366b0

Comment by Gerrit Updater [ 11/Dec/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/49220/
Subject: LU-16231 misc: fix stats snapshot_time to use wallclock
Project: fs/lustre-release
Branch: b2_15
Current Patch Set:
Commit: 2e6254c78a4d25dd1e00ee793124de4f00533b9f

Comment by Peter Jones [ 11/Dec/22 ]

Landed for 2.16

Comment by Gerrit Updater [ 20/Dec/22 ]

"Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/48847/
Subject: LU-16231 misc: rename lprocfs_stats functions
Project: fs/lustre-release
Branch: master
Current Patch Set:
Commit: 6293a8723d850d72e7313a92fa967e24d580f59d

Generated at Sat Feb 10 03:25:10 UTC 2024 using Jira 9.4.14#940014-sha1:734e6822bbf0d45eff9af51f82432957f73aa32c.