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

Adding ravendb_database_tasks metric #6

Merged
merged 3 commits into from
Jan 7, 2024

Conversation

malhussan
Copy link
Contributor

@malhussan malhussan commented Jan 3, 2024

Overview

This pull request introduces two new metrics, ravendb_database_active_tasks and ravendb_database_inactive_tasks, to the RavenDB exporter. These metrics provide valuable insights into the active and inactive tasks within the RavenDB database, enhancing the monitoring capabilities of the exporter.

Motivation

The addition of these metrics addresses a gap in the current monitoring capabilities of the RavenDB exporter. By exposing information about active and inactive tasks, users gain a deeper understanding of the database's workload and resource utilization. This is particularly crucial for performance analysis, capacity planning, and proactive issue detection through alerting on inactive tasks.

Edit:

This pull request now adds only one metric: ravendb_database_tasks

@marcinbudny
Copy link
Owner

Hi, thank you for the PR. I will try to look into it soon.

Copy link
Owner

@marcinbudny marcinbudny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please add the new metrics to the metric list in readme.

collector.go Outdated Show resolved Hide resolved
collector.go Outdated Show resolved Hide resolved
@malhussan
Copy link
Contributor Author

Thank you for the review @marcinbudny. I added 88b48e2 to resolve the remarks.

@malhussan malhussan changed the title Adding ravendb_database_active_tasks and ravendb_database_inactive_tasks metrics Adding ravendb_database_tasks metric Jan 5, 2024
collector.go Outdated Show resolved Hide resolved
@marcinbudny marcinbudny merged commit e281174 into marcinbudny:master Jan 7, 2024
1 check passed
@marcinbudny
Copy link
Owner

Version 0.5.0 is now released with this change, thanks for the contribution.

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.

None yet

2 participants