From d0aee19ed48850de81d92fa2e6cd10ce70c59092 Mon Sep 17 00:00:00 2001 From: Tsvetomir Dimitrov Date: Mon, 8 Dec 2025 15:22:15 +0200 Subject: [PATCH 1/3] Some logs in can_build_upon --- cumulus/client/consensus/aura/src/collators/mod.rs | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/cumulus/client/consensus/aura/src/collators/mod.rs b/cumulus/client/consensus/aura/src/collators/mod.rs index d938dca69282f..787e19b545120 100644 --- a/cumulus/client/consensus/aura/src/collators/mod.rs +++ b/cumulus/client/consensus/aura/src/collators/mod.rs @@ -242,8 +242,18 @@ where P::Signature: Codec, { let runtime_api = client.runtime_api(); - let authorities = runtime_api.authorities(parent_hash).ok()?; - let author_pub = aura_internal::claim_slot::

(para_slot, &authorities, keystore).await?; + let authorities = runtime_api.authorities(parent_hash).ok(); + let author_pub = aura_internal::claim_slot::

(para_slot, &authorities, keystore).await; + + tracing::trace!( + target: crate::LOG_TARGET, + ?authorities, + author_pub?, + "in can_build_upon" + ); + + let authorities = authorities?; + let author_pub = author_pub?; // This function is typically called when we want to build block N. At that point, the // unincluded segment in the runtime is unaware of the hash of block N-1. If the unincluded From 33f933449653fc4937693ba3971192fcbbb3a29a Mon Sep 17 00:00:00 2001 From: Tsvetomir Dimitrov Date: Mon, 8 Dec 2025 17:46:07 +0200 Subject: [PATCH 2/3] fix compilation error --- .../consensus/aura/src/collators/mod.rs | 25 +++++++++++-------- 1 file changed, 14 insertions(+), 11 deletions(-) diff --git a/cumulus/client/consensus/aura/src/collators/mod.rs b/cumulus/client/consensus/aura/src/collators/mod.rs index 787e19b545120..9854b4705b618 100644 --- a/cumulus/client/consensus/aura/src/collators/mod.rs +++ b/cumulus/client/consensus/aura/src/collators/mod.rs @@ -82,7 +82,7 @@ impl BackingGroupConnectionHelper { if Some(current_slot) <= self.our_slot { // Current slot or next slot is ours. // We already sent pre-connect message, no need to proceed further. - return + return; } let next_slot = current_slot + 1; @@ -133,7 +133,7 @@ async fn check_validation_code_or_log( %para_id, "Failed to fetch validation code hash", ); - return + return; }, }; @@ -186,7 +186,7 @@ async fn scheduling_lookahead( if parachain_host_runtime_api_version < RuntimeApiRequest::SCHEDULING_LOOKAHEAD_RUNTIME_REQUIREMENT { - return None + return None; } match relay_client.scheduling_lookahead(relay_parent).await { @@ -243,16 +243,19 @@ where { let runtime_api = client.runtime_api(); let authorities = runtime_api.authorities(parent_hash).ok(); - let author_pub = aura_internal::claim_slot::

(para_slot, &authorities, keystore).await; - - tracing::trace!( + tracing::debug!( target: crate::LOG_TARGET, - ?authorities, - author_pub?, + authorities_is_none=authorities.is_none(), "in can_build_upon" ); - let authorities = authorities?; + + let author_pub = aura_internal::claim_slot::

(para_slot, &authorities, keystore).await; + tracing::debug!( + target: crate::LOG_TARGET, + author_pub_is_none=author_pub.is_none(), + "in can_build_upon" + ); let author_pub = author_pub?; // This function is typically called when we want to build block N. At that point, the @@ -316,7 +319,7 @@ where "Could not fetch potential parents to build upon" ); - return None + return None; }, Ok(x) => x, }; @@ -709,7 +712,7 @@ impl RelayParentData { let Self { relay_parent, mut descendants } = self; if descendants.is_empty() { - return Default::default() + return Default::default(); } let mut result = vec![relay_parent]; From 51840901a1223738d2e6f3d1ebd038484360908d Mon Sep 17 00:00:00 2001 From: Tsvetomir Dimitrov Date: Mon, 8 Dec 2025 18:20:21 +0200 Subject: [PATCH 3/3] =?UTF-8?q?=F0=9F=98=AD?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- cumulus/client/consensus/aura/src/collators/mod.rs | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/cumulus/client/consensus/aura/src/collators/mod.rs b/cumulus/client/consensus/aura/src/collators/mod.rs index 9854b4705b618..1fc85f9b106a6 100644 --- a/cumulus/client/consensus/aura/src/collators/mod.rs +++ b/cumulus/client/consensus/aura/src/collators/mod.rs @@ -241,7 +241,15 @@ where P::Public: Codec, P::Signature: Codec, { + tracing::debug!( + target: crate::LOG_TARGET, + "before client.runtime_api()" + ); let runtime_api = client.runtime_api(); + tracing::debug!( + target: crate::LOG_TARGET, + "after client.runtime_api()" + ); let authorities = runtime_api.authorities(parent_hash).ok(); tracing::debug!( target: crate::LOG_TARGET,