diff --git a/OpenTelemetryPlan/Phase9_taskList.md b/OpenTelemetryPlan/Phase9_taskList.md index a29cfb8c79..e45c5bcd77 100644 --- a/OpenTelemetryPlan/Phase9_taskList.md +++ b/OpenTelemetryPlan/Phase9_taskList.md @@ -465,7 +465,7 @@ These metrics serve multiple external consumer categories identified during rese ### Purpose -Phase 9 exports ~68 internal rippled metrics and ships Grafana dashboards for +Phase 9 exports ~68 internal xrpld metrics and ships Grafana dashboards for them. This adds the missing operator-facing piece: **provisioned Grafana alert rules** that fire on the health-critical metrics phase 9 introduces. The phase-9 task list (line 311) and Jira story RIPD-5187 both already list diff --git a/docs/telemetry-runbook.md b/docs/telemetry-runbook.md index 8f99c7adbe..8e832e10aa 100644 --- a/docs/telemetry-runbook.md +++ b/docs/telemetry-runbook.md @@ -888,7 +888,7 @@ Requires `trace_peer=1` in the `[telemetry]` config section. ## Alerting -rippled provisions six Grafana alert rules on the health-critical metrics, so a +xrpld provisions six Grafana alert rules on the health-critical metrics, so a stock stack alerts out of the box with no UI setup. Rules are provisioned from `docker/telemetry/grafana/provisioning/alerting/` and load automatically when the Grafana container starts. They appear under **Alerting → Alert rules**, diff --git a/src/tests/libxrpl/telemetry/ValidationTracker.cpp b/src/tests/libxrpl/telemetry/ValidationTracker.cpp index d2b96aa616..92bd8d63e6 100644 --- a/src/tests/libxrpl/telemetry/ValidationTracker.cpp +++ b/src/tests/libxrpl/telemetry/ValidationTracker.cpp @@ -178,7 +178,7 @@ TEST_F(ValidationTrackerTest, MaxPendingEventsTrimming) for (std::size_t i = 0; i < kCount; ++i) { auto const hash = makeHash(i + 1); - LedgerIndex const seq = static_cast(i + 1); + auto const seq = static_cast(i + 1); tracker_.recordOurValidation(hash, seq); tracker_.recordNetworkValidation(hash, seq); } diff --git a/src/xrpld/telemetry/MetricsRegistry.cpp b/src/xrpld/telemetry/MetricsRegistry.cpp index 040cb47717..bf7f7387ed 100644 --- a/src/xrpld/telemetry/MetricsRegistry.cpp +++ b/src/xrpld/telemetry/MetricsRegistry.cpp @@ -1032,11 +1032,11 @@ MetricsRegistry::registerPeerQualityGauge() int divergedCount = 0; // Encode a version string into BuildInfo's comparable numeric - // form. Peers report the full "rippled-3.3.0-b0" string while + // form. Peers report the full "xrpld-3.3.0-b0" string while // our baseline is the bare "3.3.0-b0", and SemanticVersion // requires a leading digit, so strip any non-digit prefix // first. Numeric encoding avoids the lexicographic bug where - // "2.3.0" > "2.10.0" and "rippled-..." > "3...". + // "2.3.0" > "2.10.0" and "xrpld-..." > "3...". auto const encodeVersion = [](std::string_view v) -> std::uint64_t { auto const firstDigit = v.find_first_of("0123456789"); if (firstDigit == std::string_view::npos) @@ -1242,7 +1242,7 @@ MetricsRegistry::registerStateTrackingGauge() // State value: 0-4 from OperatingMode, 5=validating, 6=proposing. auto const mode = app.getOPs().getOperatingMode(); - double stateValue = static_cast(mode); + auto stateValue = static_cast(mode); // If FULL, refine using consensus info for validating/proposing. if (mode == OperatingMode::FULL) diff --git a/src/xrpld/telemetry/MetricsRegistry.h b/src/xrpld/telemetry/MetricsRegistry.h index 2cc154bc53..280251516d 100644 --- a/src/xrpld/telemetry/MetricsRegistry.h +++ b/src/xrpld/telemetry/MetricsRegistry.h @@ -142,7 +142,8 @@ #ifdef XRPL_ENABLE_TELEMETRY #include #include -#include +#include +#include #include #endif