Skip to content

Aerospike Enterprise Check and Dashboards v1.0.0 #2718

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

Open
wants to merge 46 commits into
base: master
Choose a base branch
from

Conversation

mphanias
Copy link

@mphanias mphanias commented Jun 15, 2025

initial checkin

What does this PR do?

Modified the aerospike integration to include all the metrics exposed by aerospike-pormetheus-exporter and multiple dashboard consuming the exported metrics.

Motivation

Expanding Metric Collection and Dashboards, we want to ensure all Aerospike metrics are available in datadog, so customers can have better visibility and provide actionable insights for key metrics, including performance, latency, and resource utilization.

Review checklist

  • PR has a meaningful title or PR has the no-changelog label attached
  • Feature or bugfix has tests
  • Git history is clean
  • If PR impacts documentation, docs team has been notified or an issue has been opened on the documentation repo
  • If this PR includes a log pipeline, please add a description describing the remappers and processors.

Additional Notes

Anything else we should know when reviewing?

@mphanias mphanias requested review from a team as code owners June 15, 2025 14:11
@mphanias mphanias changed the title aerospike-agent-and-dashboards Aerospike Enterprise CheckV2 and Dashboards v1.0.0 Jun 15, 2025
mphanias added 4 commits June 15, 2025 20:21
fixed lint error and updated classifier tags
removed legacy/openmetricsv2 keywords from metrics
fixed conf.yaml.example
fixed generated files and manifest.json
added code-owner and remove overview dashboard
@mphanias mphanias requested a review from a team as a code owner June 15, 2025 16:00
mphanias and others added 8 commits June 15, 2025 21:48
removed saved-views and monitors entries in manifest.json
corrected test-cases
added e2e test and removed unnecessary integration tests
modified service_checks
removed failing test-case
@urseberry urseberry added the editorial review Waiting on a more in-depth review from a docs team editor label Jun 16, 2025
mphanias and others added 4 commits June 17, 2025 05:50
modified and updated README with installation instructions and overview
added setup section
@mphanias mphanias changed the title Aerospike Enterprise CheckV2 and Dashboards v1.0.0 Aerospike Enterprise Check and Dashboards v1.0.0 Jun 17, 2025
mphanias and others added 17 commits June 24, 2025 09:24
accepted review suggestion - corrected links and remove unnecessary links
modified manifest.json, metadata.csv and dashboard descriptions
incorporated feedback on format, standardisation and verbiage.
@mphanias mphanias requested a review from maycmlee June 24, 2025 06:45
@mphanias
Copy link
Author

@dd-dominic , Thank you for your inputs. As suggested shared email with [email protected]

@mphanias
Copy link
Author

@nubtron - Here are some details why we went with new Aerospike-enterprise check,

First of all, existing Aerospike check provides a good overview of the key metrics and helping customers with overview dashboard.

However we felt,

  1. Existing aerospike check (in the core repo) is providing metrics from aerospike server older versions which are out of support like 4.9
    Aerospike server latest version is 8.2 -
    We plan to support metrics for last few of version as defined by our product strategy instead of support all versions.
  2. There are many metrics which are enhanced or removed or renamed in newer version and some of the names are not a direct match in the current check.
    We wanted to keep the metric names close or same between datadog and prometheus.
  3. We wanted to expose all the metrics exposed by our server so customer can use them to enhance or create new dashboards or monitors

Regards
Phani.

removed some metrics which are removed from 7.0 onwards
@dd-dominic
Copy link
Collaborator

@mphanias please close PR per discussion

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

Successfully merging this pull request may close these issues.

5 participants