Issue Summary

A copy-paste error was identified in the StorageStat class definition, where the prefix attribute is incorrectly set using SessionOpStat.prefix instead of StorageStat.prefix.

Context

The following code snippet demonstrates the issue:

class SessionOpStat(Stat):
    prefix = 'session'
    def __init__(self, name, desc, flags=''):
        Stat.__init__(self, name, SessionOpStat.prefix, desc, flags)
class StorageStat(Stat):
    prefix = 'tiered-storage'
    def __init__(self, name, desc, flags=''):
        Stat.__init__(self, name, SessionOpStat.prefix, desc, flags)

In the StorageStat class, the prefix should reference StorageStat.prefix, but currently references SessionOpStat.prefix, which is incorrect and likely a result of copy-paste.

Proposed Solution

• Update the StorageStat class constructor to use StorageStat.prefix instead of SessionOpStat.prefix.
• Review similar class definitions for potential copy-paste errors.

Original Slack thread
This ticket was generated by AI from a Slack thread.