Repository navigation
Cockpit reaches ultra by default; drop runner-host defaults - #112
Merged
Merged
Conversation
runner-host is retired: ultra is now the coordinator and the only runner host. The cockpit's default route lived in six places (CLI help, three CLI fallbacks, the app's settings, the sentinel, the widget sample); it now lives in one, TransportConfig.defaultAliases = ["ultra", "ultra-ts"], and everything else reads it. Another or an extra host stays a setting (Settings -> Connection, or --via), not a release. host-sentinel.sh names the host after its target unless SENTINEL_NAME is set, instead of a hard-coded "runner-host". shoot-motion.mjs and the settle-stale-jobs examples ssh to ultra. Cockpit docs follow the new default; federation.md gets a short current-topology note pointing at "Adding a second runner host". Historical comments that measured something on runner-host are left as they were. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The app keeps writing glance.json while it retries a route, so a running app still pointed at the retired runner-host made `cockpit status` answer "Collector stalled · via app (runner-host)" from a 20-minute-old glance, although ultra answered at once over the CLI's own tunnel. A fresh snapshot is now taken as-is only when the app is connected (live or collector-stale); otherwise the CLI opens its own route and falls back to the app's view (which may carry an out-of-band "host down") only when that fails too. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
runner-host was retired today: ultra is the coordinator (fleetd, replica id
ultra) and the only runner host.What changes
TransportConfig.defaultAliases = ["ultra", "ultra-ts"](plusdefaultRemotePort) in CockpitCore. The CLI's help, its tunnel andsentinel/pair/topfallbacks, the app's default settings, the Terminal shortcut and the widget sample all read it. Adding another host or moving the coordinator again is still a setting (Settings → Connection, or--via), not a release.SENTINEL_TARGETunlessSENTINEL_NAMEis set, so there's no hard-codedrunner-hostany more.glance.jsonwhile it retries a route, so an app still pointed at runner-host madecockpit statusanswer "Collector stalled · via app (runner-host)" from a 20-minute-old glance. A fresh snapshot is now used as-is only while the app is connected. Otherwise the CLI opens its own route, and falls back to the app's view (which may carry an out-of-band host-down) only if that fails too.SnapshotFile.isConnectedis new and has a test.docs/tools/shoot-motion.mjsand thesettle-stale-jobs.mjsexamples now ssh toultra.federation.mdgets a short "Current topology" note that points to "Adding a second runner host".Left alone on purpose: CHANGELOG history, comments that record a measurement taken on runner-host, and generic
your-runner-hostplaceholders. Nothing on the fleetd side changes (the only file underdashboard/is a comment in a one-off script).Tests
swift test(cockpit): 106 passednpm test(dashboard): 1078 passed, 1 skippedscripts/check-docs.sh,scripts/test-sentinel.sh: ok🤖 Generated with Claude Code