docs(kalshi): document Edge Connect teardown in the runbook - #200
Draft
ben-dz wants to merge 1 commit into
Draft
Conversation
The runbook ends at 'Confirm data path' with no removal path, so the only uninstall guidance for Edge Connect lives in the bridge repo's README. Add a Teardown section covering the part that is easy to get wrong: 'docker stop' is the uninstall, because the entrypoint stays PID 1 so its TERM trap can run a bounded 'doublezero disconnect' before the daemon goes down. A 'docker rm -f' or 'docker kill' bypasses that trap and orphans the onchain session and doublezero1 in the host netns.
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.
The Kalshi runbook ends at "Confirm data path" with no removal path, so the only uninstall guidance for Edge Connect lives in the bridge repo's README. An agent driving this runbook via
get_onboarding_runbookhas nothing to offer a user who asks how to undo it.Adds a
## Teardownsection and one gotcha.The part worth reviewing is the correctness claim, not the prose.
docker stopis the uninstall: the entrypoint stays PID 1 specifically so itsTERMtrap can run a boundeddoublezero disconnectwhile the daemon is still up, and the installer sets--stop-timeout 60to give it room. So a manualdoublezero disconnectfirst is redundant — butdocker rm -fordocker killbypasses the trap and orphans the onchain session anddoublezero1in the host netns. That is gotcha 8 ("Staledoublezero1") arriving self-inflicted, and it was documented nowhere user-facing;connect.shwarns about it only in its own reinstall path. The verification step usesip link show doublezero1rather thandoublezero status, since the latter is only reachable throughdocker execand the container is gone by then.Sources:
docker-entrypoint.sh(the trap and itsSIGNALED/dz_connectedguards),scripts/connect.sh(--stop-timeout 60, and the orphaned-iface warning it emits after a failed graceful stop).Blocker — this page is already truncated
Not introduced here, but it makes this PR ineffective on its own. The MCP fetches runbooks through
docsfetch, which hard-cuts every page atMaxPageBytes = 10000. This page is 11,086 bytes onmain, so the model already never sees the tail — the cut lands mid-sentence in gotcha 7, dropping gotcha 8, the consumer sketch and See also. The section added here sits past that cut.It is systemic, not specific to this page. 9 of the 20 MCP-served pages are over the limit:
troubleshootingat 21,033 bytes loses 52% of itself,contribute43%, andkalshi,contribute-operations,geolocation,glossary,setupandmulticast-adminare all cut. That is worth its own issue againstlake.So: merge this for the content and the docs site, but the MCP will not serve it until
MaxPageBytesis raised or these pages are trimmed. Happy to trim this page under 10,000 in this PR instead if that is the call.