Skip to content

Commit 995cb1d

Browse files
committed
bench: Trace message fixes
and forwarding config adoption
1 parent 7e0c0b2 commit 995cb1d

File tree

3 files changed

+12
-16
lines changed

3 files changed

+12
-16
lines changed

bench/locli/src/Cardano/Unlog/LogObject.hs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -358,11 +358,11 @@ interpreters = map3ple Map.fromList . unzip3 . fmap ent $
358358
, (,,,) "TraceBenchTxSubServAck" "TraceBenchTxSubServAck" "TraceBenchTxSubServAck" $
359359
\v -> LOTxsAcked <$> v .: "txIds"
360360

361-
, (,,,) "TraceTxSubmissionCollected" "TraceTxSubmissionCollected" "TraceTxSubmissionCollected" $
361+
, (,,,) "TraceTxSubmissionCollected" "TraceTxSubmissionCollected" "TxSubmission.TxInbound.Collected" $
362362
\v -> LOTxsCollected
363363
<$> v .: "count"
364364

365-
, (,,,) "TraceTxSubmissionProcessed" "TraceTxSubmissionProcessed" "TraceTxSubmissionProcessed" $
365+
, (,,,) "TraceTxSubmissionProcessed" "TraceTxSubmissionProcessed" "TxSubmission.TxInbound.Processed" $
366366
\v -> LOTxsProcessed
367367
<$> v .: "accepted"
368368
<*> v .: "rejected"
@@ -381,7 +381,7 @@ interpreters = map3ple Map.fromList . unzip3 . fmap ent $
381381
\_ -> pure LOMempoolRejectedTx
382382

383383
-- Generator:
384-
, (,,,) "TraceBenchTxSubSummary" "TraceBenchTxSubSummary" "TraceBenchTxSubSummary" $
384+
, (,,,) "TraceBenchTxSubSummary" "TraceBenchTxSubSummary" "Benchmark.BenchTxSubSummary" $
385385
\v -> do
386386
x :: Object <- v .: "summary"
387387
LOGeneratorSummary

bench/tx-generator/src/Cardano/Benchmarking/Tracer.hs

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -95,10 +95,10 @@ initTxGenTracers mbForwarding = do
9595
configureTracers confState initialTraceConfig [tracer]
9696
pure $ Tracer (traceWith tracer)
9797

98-
benchTracer@(Tracer traceBench) <- mkTracer "benchmark"
99-
n2nSubmitTracer <- mkTracer "submitN2N"
100-
connectTracer <- mkTracer "connect"
101-
submitTracer <- mkTracer "submit"
98+
benchTracer@(Tracer traceBench) <- mkTracer "Benchmark"
99+
n2nSubmitTracer <- mkTracer "SubmitN2N"
100+
connectTracer <- mkTracer "Connect"
101+
submitTracer <- mkTracer "Submit"
102102

103103
traceBench $ TraceTxGeneratorVersion Version.txGeneratorVersion
104104

@@ -112,8 +112,9 @@ initTxGenTracers mbForwarding = do
112112
prepareForwardingTracer :: IO (Maybe (Trace IO FormattedMessage))
113113
prepareForwardingTracer = forM mbForwarding $
114114
\(iomgr, networkId, tracerSocket) -> do
115+
let forwardingConf = fromMaybe defaultForwarder (tcForwarder initialTraceConfig)
115116
(forwardSink :: ForwardSink TraceObject, dpStore) <-
116-
initForwarding iomgr (tcForwarder initialTraceConfig) (toNetworkMagic networkId) Nothing $ Just (tracerSocket, Initiator)
117+
initForwarding iomgr forwardingConf (toNetworkMagic networkId) Nothing $ Just (tracerSocket, Initiator)
117118

118119
-- we need to provide NodeInfo DataPoint, to forward generator's name
119120
-- to the acceptor application (for example, 'cardano-tracer').
@@ -161,7 +162,7 @@ initialTraceConfig = TraceConfig {
161162
, setMaxDetail "connect"
162163
, setMaxDetail "submit"
163164
]
164-
, tcForwarder = defaultForwarder
165+
, tcForwarder = Just defaultForwarder
165166
, tcNodeName = Nothing
166167
, tcPeerFrequency = Just 2000 -- Every 2 seconds
167168
, tcResourceFrequency = Just 1000 -- Every second
@@ -278,7 +279,7 @@ instance MetaTrace (TraceBenchTxSubmit TxId) where
278279
namespaceFor TraceBenchTxSubServCons {} = Namespace [] ["BenchTxSubServCons"]
279280
namespaceFor TraceBenchTxSubIdle {} = Namespace [] ["BenchTxSubIdle"]
280281
namespaceFor TraceBenchTxSubRateLimit {} = Namespace [] ["BenchTxSubRateLimit"]
281-
namespaceFor TraceBenchTxSubSummary {} = Namespace [] ["eBenchTxSubSummary"]
282+
namespaceFor TraceBenchTxSubSummary {} = Namespace [] ["BenchTxSubSummary"]
282283
namespaceFor TraceBenchTxSubDebug {} = Namespace [] ["BenchTxSubDebug"]
283284
namespaceFor TraceBenchTxSubError {} = Namespace [] ["BenchTxSubError"]
284285
namespaceFor TraceBenchPlutusBudgetSummary {} = Namespace [] ["BenchPlutusBudgetSummary"]
@@ -301,7 +302,7 @@ instance MetaTrace (TraceBenchTxSubmit TxId) where
301302
, Namespace [] ["BenchTxSubServCons"]
302303
, Namespace [] ["BenchTxSubIdle"]
303304
, Namespace [] ["BenchTxSubRateLimit"]
304-
, Namespace [] ["eBenchTxSubSummary"]
305+
, Namespace [] ["BenchTxSubSummary"]
305306
, Namespace [] ["BenchTxSubDebug"]
306307
, Namespace [] ["BenchTxSubError"]
307308
, Namespace [] ["BenchPlutusBudgetSummary"]

nix/workbench/service/tracing.nix

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -85,11 +85,6 @@ let
8585
"TxSubmission.Remote".severity = "Notice";
8686
"TxSubmission.TxInbound".severity = "Debug";
8787
"TxSubmission.TxOutbound".severity = "Notice";
88-
89-
"TraceBenchTxSubServAck".severity = "Debug";
90-
"TraceBenchTxSubSummary".severity = "Debug";
91-
"TraceTxSubmissionCollected".severity = "Debug";
92-
"TraceTxSubmissionProcessed".severity = "Debug";
9388
};
9489
};
9590

0 commit comments

Comments
 (0)