-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove obsolete megacli collector. #798
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
README.md
Outdated
@@ -80,7 +80,14 @@ tcpstat | Exposes TCP connection status information from `/proc/net/tcp` and `/p | |||
Name | Description | OS | |||
---------|-------------|---- | |||
gmond | Exposes statistics from Ganglia. | _any_ | |||
megacli | Exposes RAID statistics from MegaCLI. | Linux | |||
|
|||
### Removed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you really need to mention these still?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 for omitting this entirely.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We never updated the README with a mention of the new textfile collector. I wanted there be some way of knowing where to find it.
This collector has been replaced by the textfile collector tool `storcli.py`.
e58e91c
to
c411f33
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This collector has been replaced by the textfile collector tool `storcli.py`.
This collector has been replaced by the textfile collector tool
storcli.py
.