Skip to content

Create a custom collector to replace active_record type#6378

Merged
TheOneFromNorway merged 1 commit intonextfrom
MAV-5297/aggregate_metrics_by_ip
Mar 19, 2026
Merged

Create a custom collector to replace active_record type#6378
TheOneFromNorway merged 1 commit intonextfrom
MAV-5297/aggregate_metrics_by_ip

Conversation

@TheOneFromNorway
Copy link
Copy Markdown
Contributor

  • This aggregates all metrics by IP to reduce metric spam and inflated costs
  • The prometheus_exporter script needs to be updated to handle this change
    • Making the script possible to run local with additional parameter is a bonus
  • Also drop host_name since it is redunant when we have task definition

Jira Issue - MAV-5297

@TheOneFromNorway TheOneFromNorway added this to the v7.5.0 milestone Mar 19, 2026
@TheOneFromNorway TheOneFromNorway requested a review from a team as a code owner March 19, 2026 11:26
Comment thread lib/prometheus_exporter/custom_active_record_collector.rb Outdated
Comment thread lib/prometheus_exporter/custom_active_record_collector.rb Outdated
Comment thread lib/prometheus_exporter/custom_active_record_collector.rb Outdated
@TheOneFromNorway TheOneFromNorway force-pushed the MAV-5297/aggregate_metrics_by_ip branch 5 times, most recently from f0ce3d6 to ecba37d Compare March 19, 2026 12:33
- This aggregates all metrics by IP to reduce metric spam and inflated
  costs
- The prometheus_exporter script needs to be updated to handle this
  change
  - Making the script possible to run local with additional parameter is
    a bonus
- Also drop host_name since it is redunant when we have task definition
@TheOneFromNorway TheOneFromNorway force-pushed the MAV-5297/aggregate_metrics_by_ip branch from ecba37d to 939618b Compare March 19, 2026 14:13
@TheOneFromNorway TheOneFromNorway merged commit 919dc0e into next Mar 19, 2026
13 checks passed
@TheOneFromNorway TheOneFromNorway deleted the MAV-5297/aggregate_metrics_by_ip branch March 19, 2026 14:30
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