Bug 763433 (GLUSTER-1701) - better statistics gathering in glusterd
Summary: better statistics gathering in glusterd
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: GLUSTER-1701
Product: GlusterFS
Classification: Community
Component: core
Version: mainline
Hardware: All
OS: Linux
low
high
Target Milestone: ---
Assignee: Amar Tumballi
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2010-09-24 15:40 UTC by Amar Tumballi
Modified: 2015-12-01 16:45 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed:
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Vijay Bellur 2010-09-24 15:25:57 UTC 
PATCH: http://patches.gluster.com/patch/4964 in master (stack.h: remove the time/latency measurement code)
Comment 1 Vijay Bellur 2010-09-24 15:26:01 UTC 
PATCH: http://patches.gluster.com/patch/4965 in master (io-stats: bring in latency measurement feature)
Comment 2 Vijay Bellur 2010-09-24 15:26:05 UTC 
PATCH: http://patches.gluster.com/patch/4967 in master (glusterd-volgen: add 'io-stats' xlator on server volfile too)
Comment 3 Vijay Bellur 2010-09-24 15:26:09 UTC 
PATCH: http://patches.gluster.com/patch/4966 in master (glusterd: add new stats related options for 'volume set')
Comment 4 Amar Tumballi 2010-09-24 15:40:12 UTC 
as of now, latency measurement code is hardcoded in 'frame' creation and destroy, which may cause a minor performance glitch in case there are many frames.

Also its better to keep all the stat gathering in one place and make it modular.

There is no way one can trigger stats gathering in gluster CLI as of now.. need to address that too.
Comment 5 Vijay Bellur 2010-10-01 13:33:53 UTC 
PATCH: http://patches.gluster.com/patch/5150 in master (stat enhancements)
Comment 6 Vijay Bellur 2010-10-02 04:03:37 UTC 
PATCH: http://patches.gluster.com/patch/5174 in master (get server xattr only if name is not NULL)
Note You need to log in before you can comment on or make changes to this bug.