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

Om147 #104

Merged
merged 607 commits into from
Feb 21, 2024
Merged

Om147 #104

merged 607 commits into from
Feb 21, 2024

Conversation

mphanias
Copy link
Contributor

  1. Added key System info metrics
    • memory, file-descriptors, network receive/transmit etc.,
    • can be enabled/disabled in ape.toml using flag "refresh_system_stats"
  2. Added config to fetch additional cloud provider metrics
    • region, location, availability-zone
    • details can be fetch by providing "cloud_provider" in ape.toml
    • supported cloud-provider values are "aws", "azure", "gcp"
  3. Aligned and Cleaned up configs
    • removed job allowlist/blocklist - as support for job is removed in this release of exporter
  4. Fixed logic on actual stat-name returned by server, instead of the intermediate/constructed stat-names
    • storage-engine.file, storage-engine.stripe etc., are impacted
  5. Modified the Makefile and GitHub Action script to include tar.gz artefact
  6. Added support for native OpenTelemetry,
    • default mode is "Prometheus"
    • OTel can be enabled/disabled using flag "OPEN_TELEMETRY" in ape.toml
    • both Prometheus and OTel can run together in single exporter instance
    • added a configuration section for OTel configd in ape.toml under "Agent.OepnTelemetry" section
    • currently supports only gRPC based OTel end-points, like OTelCollector
  7. Aligned code according to golang standard package structure
  8. README.md is updated with required corrections and changes

@mphanias mphanias requested a review from sunilvirus February 19, 2024 13:40
@sunilvirus sunilvirus merged commit c80756e into dev Feb 21, 2024
1 check passed
@sunilvirus sunilvirus deleted the OM147 branch February 21, 2024 12:33
geobeau added a commit to geobeau/aerospike-prometheus-exporter that referenced this pull request Aug 27, 2024
It was broken during aerospike#104. Even the comment says it should
be or.
Without this change, aerospike client tries to connect without TLS
geobeau added a commit to criteo-forks/aerospike-prometheus-exporter that referenced this pull request Aug 27, 2024
It was broken during aerospike#104. Even the comment says it should
be or.
Without this change, aerospike client tries to connect without TLS
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.

2 participants