fix(telemetry): move quorum/proposers attributes to consensus.accept span

Move validation_quorum and proposers_validated attributes from
consensus.accept.apply to consensus.accept span to match the design
spec. Both values are available in onAccept() scope.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Pratik Mankawde
2026-03-31 13:48:05 +01:00
parent 27d2078419
commit aa329d5084

View File

@@ -455,6 +455,10 @@ RCLConsensus::Adaptor::onAccept(
XRPL_TRACE_SET_ATTR("xrpl.consensus.proposers", static_cast<int64_t>(result.proposers));
XRPL_TRACE_SET_ATTR(
"xrpl.consensus.round_time_ms", static_cast<int64_t>(result.roundTime.read().count()));
XRPL_TRACE_SET_ATTR(
"xrpl.consensus.validation_quorum", static_cast<int64_t>(app_.getValidators().quorum()));
XRPL_TRACE_SET_ATTR(
"xrpl.consensus.proposers_validated", static_cast<int64_t>(result.proposers));
app_.getJobQueue().addJob(
jtACCEPT,
@@ -536,10 +540,6 @@ RCLConsensus::Adaptor::doAccept(
XRPL_TRACE_SET_ATTR("xrpl.consensus.proposing", proposing);
XRPL_TRACE_SET_ATTR(
"xrpl.consensus.round_time_ms", static_cast<int64_t>(result.roundTime.read().count()));
XRPL_TRACE_SET_ATTR(
"xrpl.consensus.validation_quorum", static_cast<int64_t>(app_.getValidators().quorum()));
XRPL_TRACE_SET_ATTR(
"xrpl.consensus.proposers_validated", static_cast<int64_t>(result.proposers));
// Parent ledger's close time — enables computing close-time deltas across
// consecutive rounds without correlating separate spans.
XRPL_TRACE_SET_ATTR(