Skip to content
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

GlusterFS Support #2

Open
jaymefSO opened this issue Aug 15, 2018 · 4 comments
Open

GlusterFS Support #2

jaymefSO opened this issue Aug 15, 2018 · 4 comments
Assignees
Labels
enhancement New feature or request question Further information is requested

Comments

@jaymefSO
Copy link

Thanks for your work on this plugin and the update for new API. I was wondering if there are any plans to add check support for GlusterFS/Brick monitoring. Thanks!

@scrat14
Copy link

scrat14 commented Aug 21, 2018

I also thought about this, but afaik the Gluster information aren't available via API. What I also noticed is, that information in WebUI is a couple of minutes behind information of gluster command (you can see this if you try to self heal a host). Let me talk with the oVirt developers about Gluster information in API and age of this information. If these information are available via API I can add them to the plugin.

scrat14 added a commit that referenced this issue Oct 19, 2018
scrat14 added a commit that referenced this issue Oct 19, 2018
scrat14 added a commit that referenced this issue Oct 19, 2018
@scrat14 scrat14 self-assigned this Oct 19, 2018
@scrat14 scrat14 added the enhancement New feature or request label Oct 19, 2018
@scrat14 scrat14 added this to the check_rhv - 2.0 milestone Oct 19, 2018
@scrat14
Copy link

scrat14 commented Oct 19, 2018

@jaymefSO
Can you please verify if these checks are sufficient for you? Thanks

Gluster volumes

Check for Gluster volume with status up
./check_rhv -H rhvm -f authfile -C Default -l glustervolumes

Check for Gluster volume engine with status up
./check_rhv -H rhvm -f authfile -C Default -l glustervolumes -n engine

Gluster bricks

Check for online Gluster bricks in cluster Default
./check_rhv -H rhvm -f authfile -C Default -l glusterbricks

Check for online Gluster bricks in cluster Default, but only for Gluster volume engine
./check_rhv -H rhvm -f authfile -C Default -l glusterbricks -n engine

@scrat14 scrat14 added the question Further information is requested label Oct 19, 2018
@jaymefSO
Copy link
Author

jaymefSO commented Oct 19, 2018

The latest additions are looking great, I had a chance to test this some more today. Pulling down a host and stopping glusterFS is showing brick status changing almost instantly in the check script. The warning and critical checks seem to be functioning as well.

My only suggestion at this point would be the two following items (if even possible)

  1. When bricks have un-synced entries present i.e. needs healing the check scripts do not indicate such, it will report all bricks/volumes in good standing. It would be nice if un-healed bricks could be reported

  2. Would be nice if we could check disk space usage on gluster volumes and warn if greater than X%

These are both stats present in the engine GUI but not sure if reported via ovirt API.

Thank you so much for implementing these changes, it will help us a lot!

@scrat14
Copy link

scrat14 commented Oct 19, 2018

I would also love to see un-synced entries, but I think this isn't available in API (at least not in RHV 4.1) and during tests I realised that it's a few minutes off (compared to gluster volume command on host directly).

Disk space is available in API, but haven't figured out yet which value is the correct one :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants