apollo_propeller: add dial upgrade error handling to handler

[sirandreww-starkware]

Note

Low Risk
Logging-only change on the DialUpgradeError path; no protocol or data-flow behavior changes beyond additional rate-limited warnings.

Overview
Enhances Handler dial-upgrade failure handling by matching on StreamUpgradeError and emitting more specific logs for timeouts, negotiation failures (rate-limited), and I/O errors.

Introduces DIAL_UPGRADE_WARNING_INTERVAL_MS to throttle repeated warnings when a peer doesn’t support the Propeller protocol, while keeping the existing state reset of the affected pending outbound substream back to Idle.

^{Written by Cursor Bugbot for commit 7240134. This will update automatically on new commits. Configure here.}

[sirandreww-starkware]

• apollo_propeller: add minimal config #11196 : 2 dependent PRs (#11197 , #11942 )
• apollo_propeller: add behaviour skeleton #11195
• apollo_propeller: add Event enum to types #11194
• apollo_propeller: add dial upgrade error handling to handler #11074 👈 (View in Graphite)
• apollo_propeller: add rate-limited queue length warnings #11073
• apollo_propeller: add concurrent streams support to handler #11072
• apollo_propeller: implement outbound substream with message batching #11071
• apollo_propeller: add handler helper methods #11070
• main-v0.14.2

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

[sirandreww-starkware]

@sirandreww-starkware made 1 comment.
Reviewable status: 0 of 1 files reviewed, 1 unresolved discussion (waiting on @noamsp-starkware and @ShahakShama).

---

crates/apollo_propeller/src/handler.rs line 499 at r6 (raw file):

Previously, ShahakShama wrote…

I think this can be warn. If this bug occurs we want to know about it. maybe warn once every second

Done.

[sirandreww-starkware]

@sirandreww-starkware reviewed 1 file and all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @noamsp-starkware and @ShahakShama).

[ShahakShama]

@ShahakShama reviewed all commit messages, made 1 comment, and resolved 1 discussion.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on noamsp-starkware).

[graphite-app]

Merge activity

• Feb 19, 2:20 PM UTC: This pull request can not be added to the Graphite merge queue. Please try rebasing and resubmitting to merge when ready.
• Feb 19, 2:20 PM UTC: Graphite disabled "merge when ready" on this PR due to: a merge conflict with the target branch; resolve the conflict and try again..

[sirandreww-starkware]

@sirandreww-starkware reviewed 1 file and all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on noamsp-starkware).