search

LLNL / lmt

Lustre Monitoring Tools
GNU General Public License v2.0
67 stars 21 forks source link

Missing setxattr mdops #26

Open thiell opened 8 years ago

thiell commented 8 years ago

Hi, I noticed that the setxattr metadata op counter (eg. from lctl get_param mdt.*.md_stats) is missing from cerebro lmt_mdt metrics and thus LMT itself.

I added "setxattr" at the end of optab_mdt_v1 in liblmt/mdc.c so that cerebro is now properly reporting the value. Actually, I'm using cerebro lmt_mdt directly so that's fine with me, however, this quick change breaks LMT tests. Also I suppose other code changes should probably be done to LMT itself for a full support (ltop, etc.).

ofaaland commented 4 years ago

Waiting on this until after #46 is fixed