Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
38 changes: 38 additions & 0 deletions v2/components/snapshot/download.zig
Original file line number Diff line number Diff line change
Expand Up @@ -929,13 +929,20 @@ pub const Downloader = struct {
metrics: Metrics,
logger: tel.Logger("snapshot"),

// See issue #1746.
await_state: tel.BootstrapWait,

const AWAITING_LOG_INTERVAL_NS: u64 = 10 * std.time.ns_per_s;
const AWAITING_WARN_AFTER_NS: u64 = 60 * std.time.ns_per_s;

pub fn init(
gossip_to_snapshot: *SnapshotSourceRing,
known_validators: KnownValidators,
snapshot_dir: std.fs.Dir,
metrics: Metrics,
logger: tel.Logger("snapshot"),
) !Downloader {
const now_ns = lib.clock.monotonic(.ns);
return .{
.ring = try IoUring.init(IO_URING_ENTRIES, 0),
.gossip_iter = gossip_to_snapshot.get(.reader),
Expand All @@ -950,6 +957,7 @@ pub const Downloader = struct {
.run_result = null,
.metrics = metrics,
.logger = logger,
.await_state = .init(AWAITING_LOG_INTERVAL_NS, AWAITING_WARN_AFTER_NS, now_ns),
};
}

Expand All @@ -974,6 +982,7 @@ pub const Downloader = struct {

while (true) {
try self.drainGossip();
self.maybeLogAwaitingPeers();

_ = try self.ring.submit_and_wait(0);
const n = try self.ring.copy_cqes(&cqes, 0);
Expand Down Expand Up @@ -2665,6 +2674,35 @@ pub const Downloader = struct {
};
}

/// See issue #1746.
fn maybeLogAwaitingPeers(self: *Downloader) void {
if (self.download_race.phase != .idle) {
self.await_state.logReady(
self.logger,
"snapshot: found usable peer, starting download",
);
return;
}

const now_ns = lib.clock.monotonic(.ns);
const peers_seen = self.dedupe_map.len;
if (peers_seen == 0) {
self.await_state.logAwaiting(
now_ns,
self.logger,
"snapshot: awaiting peers from gossip ({d}s, none received)",
.{},
);
} else {
self.await_state.logAwaiting(
now_ns,
self.logger,
"snapshot: no usable peers so far (received={d}, active_probes={d}, {d}s)",
.{ peers_seen, self.active_probes },
);
}
}

/// Retires all active download connections except the one at `keep_index`.
/// Used after the race reaches a terminal state (completed or failed) to
/// clean up losers. Late CQEs from retired slots are ignored via gen mismatch.
Expand Down
Loading
Loading