hyperledger / fabric-x-committer / 33759475367

90%

main: 90%

Build Type

Pull #810

github

Committed by liran-funaro

Commit Message

[coordinator] Report the dependency graph queue sizes on scrape

#### Type of change

- Improvement (improvement to code, performance, etc)

#### Description

- Replace the dependency graph manager's ticker-based queue monitor with gauges read on scrape, and
  delete the sampler goroutine.
- Gather the two input queues into a receive-only `managerQueues` struct built before the metrics
  that observe them.
- Rename `gdgWaitingTxQueueSize` and `dependentTransactionsQueueSize` to `gdgWaitingTxCount` and
  `dependentTxCount`, since neither is a queue: both count transactions held in the graph's maps and
  stay maintained incrementally.
- Rename the exported `coordinator_dependency_graph_dependent_transactions_queue_size` to
  `coordinator_dependency_graph_dependent_transactions`, for the same reason.
- Reword `coordinator_global_dependency_graph_size`'s help to lead with what it counts.
- Remove the `queue-monitor-sampling-time` config key, whose only reader was the sampler.

#### Additional details (Optional)

Deployed YAML that still sets `queue-monitor-sampling-time` keeps loading, as `v.Unmarshal` ignores
unknown keys.

Dashboard-breaking: `coordinator_dependency_graph_dependent_transactions_queue_size` becomes
`coordinator_dependency_graph_dependent_transactions`. `coordinator_global_dependency_graph_size`
keeps its name, as it never claimed to be a queue.

One of the per-service PRs split out of #736.

#### Related issues

- resolves partly #771
- related to #730

Signed-off-by: Liran Funaro <liran.funaro@gmail.com>

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Mon Aug 17 11:47:24 2026 +0300
#
# On branch queue-gauge-depgraph
# Your branch and 'origin/queue-gauge-depgraph' have diverged,
# and have 3 and 2 different commits each, respectively.
#
# Changes to be committed:
#	modified:   .bob/skills/developm... (continued)

Pull Request Pull Request #810: [coordinator] Report the dependency graph queue sizes on scrape

Coverage Stats

29 of 29 new or added lines in 5 files covered. (100.0%)

7 existing lines in 2 files now uncovered.

9312 of 10286 relevant lines covered (90.53%)

12225.8 hits per line