Skip to content

feat: sync metrics with memgraph counters/gauges/histograms#33

Merged
imilinovic merged 2 commits into
mainfrom
fix/missing-metrics
May 21, 2026
Merged

feat: sync metrics with memgraph counters/gauges/histograms#33
imilinovic merged 2 commits into
mainfrom
fix/missing-metrics

Conversation

@imilinovic
Copy link
Copy Markdown
Contributor

@imilinovic imilinovic commented May 11, 2026

Align the exporter with the current memgraph event definitions: add missing index, operator, transaction, constraint, schema/storage info and memory metrics; refresh HA RPC timers and counters; drop entries that no longer exist in memgraph.

Documentation checklist

  • Add the documentation label tag
  • Add the bug / feature label tag
  • Write a release note, including added/changed clauses
    • Added missing Memgraph counters/gauges/histograms (Constraint, SchemaInfo, StorageInfo, Memory groups plus new Index/Operator/Transaction/HA entries) and changed stale names to match upstream (AppendDeltasRpc → PrepareCommitRpc, ScanAllByLabelProperty* → ScanAllByLabelPropertiesOperator). #33
    • Missing metric groups from older Memgraph versions are now tolerated instead of aborting the scrape. #33
  • Link the documentation PR here
    • PR1623 (metrics were out of date in docs as well)
  • Tag someone from docs team in the comments

@imilinovic imilinovic self-assigned this May 11, 2026
@imilinovic imilinovic force-pushed the fix/missing-metrics branch from 6c9be3f to 990637d Compare May 11, 2026 12:58
@imilinovic imilinovic changed the title feat: Sync metrics with memgraph counters/gauges/histograms feat: sync metrics with memgraph counters/gauges/histograms May 11, 2026
@imilinovic imilinovic marked this pull request as ready for review May 11, 2026 13:23
@imilinovic imilinovic requested a review from colinbarry May 11, 2026 13:23
Copy link
Copy Markdown
Collaborator

@as51340 as51340 left a comment

Choose a reason for hiding this comment

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

LGTM, before merging please start it with a standalone and HA and test nothing is broken

Align the exporter with the current memgraph event definitions: add
missing index, operator, transaction, constraint, schema/storage info
and memory metrics; refresh HA RPC timers and counters; drop entries
that no longer exist in memgraph. Use mg_data.get(key, {}) so missing
top-level groups from older memgraph versions don't abort the scrape.
@imilinovic imilinovic force-pushed the fix/missing-metrics branch from 990637d to 6c3f456 Compare May 21, 2026 11:59
ReplicaRecovery{Success,Fail,Skip} counters were constructed without
labels but updated through the labelled update_gauges helper, which
raised on every data-instance scrape and prevented the counters from
ever being published. Add the instance_name label and a matching
update_counters helper that uses .inc() so the dashboards' *_total
series populate correctly.
@imilinovic imilinovic requested a review from as51340 May 21, 2026 12:24
@imilinovic imilinovic merged commit d3de76d into main May 21, 2026
2 checks passed
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.

2 participants