Repository navigation
fix(tunnel): preserve owned tasks through heartbeat write failures - #5
Merged
franksong2702 merged 3 commits intoNov 13, 2026
Conversation
franksong2702
marked this pull request as ready for review
November 13, 2026 04:06
franksong2702
merged commit Nov 13, 2026
0f0bef0
into
integrate/reviewed-upstream-20261020
36 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Scope
Backport upstream #1411,
6d50934b451c9a524cf9077dbc67656c2a8e5129, onto maintained8340dd0f. Heartbeat publication failures retain the owned Tunnel and retry at normal cadence, without restart/replay. Initial publication, ACL/locks, seven-second freshness, shutdown and actual task failure remain unchanged. Fork patches are preserved.Verification
Final head
d822a5ce49f80d1d34c54498075e077745c3a4f1, tree85ad415b723b8bd628142adfb6c4663db467eeed, matches local reviewed candidate. Exact-head CI run 43361318270: all 18 jobs succeeded, including Linux Server/Runner/workspace/tooling, Docker, contract/MCP, Windows and macOS execution/build lanes and requiredtest/test-nativeaggregates. No native CI job was skipped in this run. This is CI evidence, not deployed Host acceptance.Formatting, Markdown, diff-check, 43 policy tests and 6 build-identity tests passed locally; independent source review found no blocker. Local Rust dependency download was blocked by cloud network policy, so local full Rust tests are not claimed; actual Rust validation is from CI.
The first two CI runs exposed Cargo 1.100 output-layout changes in unchanged fixture assumptions. Final test-only adaptation instruments the disposable build-script copy with an OUT_DIR execution counter, retaining and strengthening cache/identity assertions. Production build script is unchanged; no gate was removed. Evidence and failed-run history.
No deployment, release, proxy configuration, permission change, service restart or upstream PR. PR #2 remains separate.
Final status
Ready-for-review recheck 43363295747 passed on the same exact head with all 18 jobs successful and no skips. Merged into maintained
integrate/reviewed-upstream-20261020as0f0bef0de3f732b25e7c5d79f58358a7539bc770; read-back merge tree85ad415b723b8bd628142adfb6c4663db467eeedmatches the reviewed and tested candidate. No deployed service, release or restart was performed.