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

Parse unavailable indexer health messages and emit metric #162

Merged
merged 1 commit into from
Apr 28, 2023

Conversation

rtrox
Copy link
Collaborator

@rtrox rtrox commented Apr 27, 2023

Description of the change

Prowlarr doesn't consider indexers that it's marked unavailable due to errors as "disabled", it only considers them "unavailable". This unavailable status doesn't have a metric of it's own, it just shows up as a health message, with the server names in a comma delimited list in the message.

This change looks for IndexerStatusCheck health messages, and emits an extra metric per indexer.

Benefits

This extra metric will make it much simpler to build alarms, as one can key off the number of indexers unavailable (or even % of indexers unavailable by dividing this by indexers_total or indexers_enabled)

Possible drawbacks

Applicable issues

Additional information

Metrics:

# HELP prowlarr_indexer_unavailable Indexers marked unavailable due to repeated errors
# TYPE prowlarr_indexer_unavailable gauge
prowlarr_indexer_unavailable{indexer="ServerOne",url="https://my.instance"} 1
prowlarr_indexer_unavailable{indexer="ServerTwo",url="https://my.instance"} 1
prowlarr_indexer_unavailable{indexer="ServerThree",url="https://my.instance"} 1
prowlarr_indexer_unavailable{indexer="Server.four",url="https://my.instance"} 1

@rtrox rtrox requested a review from onedr0p as a code owner April 27, 2023 23:43
@onedr0p onedr0p merged commit 3cca07b into master Apr 28, 2023
@onedr0p onedr0p deleted the unavailable_indexers branch April 28, 2023 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Parse Indexer Unavailable Errors to emit unavailable_indexers metric
2 participants