From f80faea85d3c53762fab54312541fcf678f22773 Mon Sep 17 00:00:00 2001 From: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com> Date: Fri, 4 Sep 2026 11:47:33 +0100 Subject: [PATCH] fix(telemetry): match the renamed close-time attrs in expected_spans.json The close-time span attributes name their unit and epoch: close_time_ripple_epoch_s, parent_close_time_ripple_epoch_s and close_time_self_ripple_epoch_s. The harness inventory still required the unsuffixed keys, so the attribute checks for consensus.accept.apply and ledger.build failed on every validation run while the spans themselves were correct. Rename the four required_attributes entries to the keys the code emits. --- docker/telemetry/workload/expected_spans.json | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/docker/telemetry/workload/expected_spans.json b/docker/telemetry/workload/expected_spans.json index b2278943a4..3309bf86cb 100644 --- a/docker/telemetry/workload/expected_spans.json +++ b/docker/telemetry/workload/expected_spans.json @@ -235,9 +235,9 @@ "parent": "consensus.accept", "required_attributes": [ "ledger_seq", - "close_time", - "parent_close_time", - "close_time_self", + "close_time_ripple_epoch_s", + "parent_close_time_ripple_epoch_s", + "close_time_self_ripple_epoch_s", "close_time_vote_bins", "resolution_direction" ], @@ -288,7 +288,7 @@ "parent": null, "required_attributes": [ "ledger_seq", - "close_time", + "close_time_ripple_epoch_s", "close_time_correct", "close_resolution_ms" ],