Fix: custody backfill sync display incorrect time estimation (#8291)

Fixes #8268


  Switch `est_time` from time until DA boundary slot, to time to finish total custody work from the original earliest data-column slot down to the DA boundary


Co-Authored-By: PoulavBhowmick03 <bpoulav@gmail.com>
This commit is contained in:
Odinson
2025-10-30 08:46:07 +05:30
committed by GitHub
parent 6f0d0dec75
commit 1cee814a95

View File

@@ -298,28 +298,28 @@ pub fn spawn_notifier<T: BeaconChainTypes>(
let speed = speedo.slots_per_second();
let display_speed = speed.is_some_and(|speed| speed != 0.0);
let est_time_in_secs = if let (Some(da_boundary_epoch), Some(original_slot)) = (
beacon_chain.get_column_da_boundary(),
original_earliest_data_column_slot,
) {
let target = original_slot.saturating_sub(
da_boundary_epoch.start_slot(T::EthSpec::slots_per_epoch()),
);
speedo.estimated_time_till_slot(target)
} else {
None
};
if display_speed {
info!(
distance,
speed = sync_speed_pretty(speed),
est_time =
estimated_time_pretty(beacon_chain.get_column_da_boundary().and_then(
|da_boundary| speedo.estimated_time_till_slot(
da_boundary.start_slot(T::EthSpec::slots_per_epoch())
)
)),
est_time = estimated_time_pretty(est_time_in_secs),
"Downloading historical data columns"
);
} else {
info!(
distance,
est_time =
estimated_time_pretty(beacon_chain.get_column_da_boundary().and_then(
|da_boundary| speedo.estimated_time_till_slot(
da_boundary.start_slot(T::EthSpec::slots_per_epoch())
)
)),
est_time = estimated_time_pretty(est_time_in_secs),
"Downloading historical data columns"
);
}