Add possibility to monitor client nodes#14
Add possibility to monitor client nodes#14coder-hugo wants to merge 3 commits intoofesseler:masterfrom
Conversation
|
Thanks for participating!! I'll have a look in a few days. |
|
Thanks for this pull request, unfortunately I have no time to test it. If you like to continue to develop this exporter please let me know. |
|
Yes I'd like to continue to develop this exporter. We are already using an enhanced version of this exporter in production and I think my adaptions would be also useful for others. So it would be great if we find a way to get my changes back to this repo. |
|
I invited you as a collaborator. are you able to commit / accept pull requests? |
|
Yes, I am able to do this. |
|
@coder-hugo Why didn't you accept your pull request? Are there any issues? May I help? |
|
@ofesseler I just forgot about doing it. When I did my last answer I had not the time to check how to "become" a reviewer and approve the pull request. Without an approval I can't merge it. |
0083068 to
780a6b9
Compare
This PR adds a new command line flag
-mounts-onlywhich disables all checks exceptgluster_mount_successfulandgluster_volume_writeable. This is useful for the monitoring of nodes that have just the client package of glusterfs installed to mount gluster volumes.Furthermore I adapted the documentation for the boolean command line flags. The value
true/falsehas to be set as follows for such flags:-flag-name=true/false(or can be left fortrue). I'm not sure whether this is a bug or a feature in go. You can find the relevant code for this here.