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

SUSPENDED state not detected correctly #13

Open
sempervictus opened this issue Mar 30, 2022 · 4 comments
Open

SUSPENDED state not detected correctly #13

sempervictus opened this issue Mar 30, 2022 · 4 comments
Assignees
Labels

Comments

@sempervictus
Copy link

sempervictus commented Mar 30, 2022

We had a failure go unnoticed this morning - pool shows up as ONLINE in Zenoss (with no IO in the graphs) but went SUSPENDED on the host hours ago.
Zenoss 6.3 with zenpack built off of 8a17ca6
Thanks as always

@daviswr daviswr self-assigned this Mar 30, 2022
@daviswr daviswr added the bug label Mar 30, 2022
@daviswr
Copy link
Owner

daviswr commented Mar 30, 2022

Any chance you could email me the zpool status output from the host? Would need to run the output through the parsing code to see where the breakdown is.

@daviswr
Copy link
Owner

daviswr commented Mar 31, 2022

Pretty sure the problem lies here

zpool-status:
usessh: true
commandTemplate: |-
$$ZENOTHING
${here/PrivEscCmd} ${here/ZpoolPath} status -v "${here/title}" | grep -e "${here/title}" -e scrub -e scan
since the state: line in zpool status output is omitted by the grep before it gets to the parser.

Shouldn't be too bad to correct.

I've opened #14 to track generating events based on state & error fields, too.

@sempervictus
Copy link
Author

Thank you sir, will pull in and test the changes this week

@daviswr
Copy link
Owner

daviswr commented Mar 31, 2022

I believe I've fixed this, but given the commit you're at currently, I'd recommend uninstalling the pack and then building and installing an updated one. Deprecated some relationships that never actually made it into .egg releases and it won't depend on the TALES patch as of last month. Also device-level thresholds will be gone in favor of pool-level.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants