gluster_exporter icon indicating copy to clipboard operation
gluster_exporter copied to clipboard

Add possibility to monitor client nodes

Open coder-hugo opened this issue 9 years ago • 7 comments

This PR adds a new command line flag -mounts-only which disables all checks except gluster_mount_successful and gluster_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/false has to be set as follows for such flags: -flag-name=true/false (or can be left for true). I'm not sure whether this is a bug or a feature in go. You can find the relevant code for this here.

coder-hugo avatar Apr 26 '17 13:04 coder-hugo

Thanks for participating!! I'll have a look in a few days.

ofesseler avatar May 11 '17 09:05 ofesseler

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.

ofesseler avatar Sep 26 '17 13:09 ofesseler

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.

coder-hugo avatar Sep 27 '17 06:09 coder-hugo

I invited you as a collaborator. are you able to commit / accept pull requests?

ofesseler avatar Sep 29 '17 10:09 ofesseler

Yes, I am able to do this.

coder-hugo avatar Oct 04 '17 06:10 coder-hugo

@coder-hugo Why didn't you accept your pull request? Are there any issues? May I help? regards, Oli

ofesseler avatar Nov 06 '17 12:11 ofesseler

@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. Edit: I can't become a reviewer of my own pull request so I'll do the merge with the cli. Edit2: Even with the cli I can't merge it without an approval. I'll have a deeper look into it later.

coder-hugo avatar Nov 07 '17 07:11 coder-hugo