Better out of capacity error

[asonnino]

Better out-of-capacity error message for consensus adapter

[huitseeker]

Is this a debug or a warn?

Do we have an issue / comment to track the recovery logic, and which side is involved (Narwhal, Sui ...).

[asonnino]

I try to never allow bad clients to make us log warns (only bad does)

[asonnino]

the recovery logic will have to be the retries of the client

[huitseeker]

A way to do this orthogonally to the log level is to set up a span, with context on what's going on. That would allow us to filter what's going on:

sui/sui_core/src/authority_server.rs

Lines 169 to 174 in 1a25e68

	// Enable Trace Propagation across spans/processes using tx_digest
	let span = tracing::debug_span!(
	"process_tx",
	?tx_digest,
	tx_kind = transaction.data.kind_as_str()
	);

[huitseeker]

Thanks for the improvement!

[huitseeker]

A way to do this orthogonally to the log level is to set up a span, with context on what's going on. That would allow us to filter what's going on:

sui/sui_core/src/authority_server.rs

Lines 169 to 174 in 1a25e68

	// Enable Trace Propagation across spans/processes using tx_digest
	let span = tracing::debug_span!(
	"process_tx",
	?tx_digest,
	tx_kind = transaction.data.kind_as_str()
	);