Skip to content

Conversation

DemetrisChr
Copy link
Contributor

Motivation

The Extended Observability RFC revisions 24 & 25 require us to record additional metrics tags: bucket, scope, collection, outcome, and cluster labels (name & UUID)

Changes

  • Update config parsing to include clusterName & clusterUUID
  • Create a new core::metrics::meter_wrapper class that watches for config updates and keeps track of cluster labels
  • Replace all meter references in the core with the meter wrapper
  • The meter wrapper is responsible for encoding the metric attributes & inserts tags for the cluster labels, if those are supported by the cluster
  • Add some tests for KV

Results

Tests pass

Known issues

Non-KV operation metrics don't currently report collection, bucket and scope names.

@DemetrisChr DemetrisChr force-pushed the CXXCBC-582-contextual-metrics branch 3 times, most recently from 175813d to c55f061 Compare October 15, 2024 15:49
@DemetrisChr DemetrisChr force-pushed the CXXCBC-582-contextual-metrics branch from c55f061 to 6a22460 Compare October 15, 2024 15:59
@avsej avsej merged commit fac2264 into couchbase:main Oct 15, 2024
24 of 27 checks passed
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