From d5c2964fea46e445aa2bec23640bea81365d44c0 Mon Sep 17 00:00:00 2001 From: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com> Date: Thu, 17 Sep 2026 23:03:28 +0100 Subject: [PATCH] test(telemetry): drop the suppressed requirement from tx.receive tx.receive no longer carries a suppressed attribute: the span is created only once the node has decided to process the transaction, so there is no dropped copy for the attribute to describe. The validator fails a span that is missing a required attribute, so leaving it listed turns the telemetry-validation leg red. --- docker/telemetry/workload/expected_spans.json | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/docker/telemetry/workload/expected_spans.json b/docker/telemetry/workload/expected_spans.json index 375c29be22..752ac13c25 100644 --- a/docker/telemetry/workload/expected_spans.json +++ b/docker/telemetry/workload/expected_spans.json @@ -55,9 +55,9 @@ "name": "tx.receive", "category": "transaction", "parent": null, - "required_attributes": ["tx_hash", "peer_id", "suppressed"], + "required_attributes": ["tx_hash", "peer_id"], "config_flag": "trace_transactions", - "note": "Cross-node span: parent context propagated from the sender's tx.process via protobuf. Also carries tx_type and peer_version. tx_status is only set when a tx is suppressed/known-bad, so it is not a required attribute on every tx.receive." + "note": "Cross-node span: parent context propagated from the sender's tx.process via protobuf. Also carries tx_type and peer_version. The span is created only after the node decides to process the transaction, so a relayed duplicate produces no span at all; how many were dropped is the transactions_duplicate traffic category. tx_status is set only on the paths that drop a transaction after that point, so it is not a required attribute." }, { "name": "tx.apply",