Point the tailscale sidecar at the mounted serve config #40

Manually merged
marvin8 merged 1 commit from fix/issue-39-serve-config-filename into main 2026-10-17 05:52:51 +00:00
Collaborator

Same mismatch class as #37, one layer up: the sidecar unit set TS_SERVE_CONFIG=/config/xique.json, but the ts-serve directory mounts at /config holding the documented xique-serve.json — so the serve config was never applied and the tailnet URL proxied to nothing (confirmed live: renaming the host copy to xique.json made it work).

The unit now expects /config/xique-serve.json, which is what the shipped file is named and what every doc already says to copy. docs/install.md gains a note that the filename is load-bearing so the next operator does not silently rename it. Docs and unit now agree in one direction, and the host workaround gets reverted as part of adopting the fix (rename back + copy the updated unit).

Closes #39

Same mismatch class as #37, one layer up: the sidecar unit set `TS_SERVE_CONFIG=/config/xique.json`, but the ts-serve directory mounts at `/config` holding the documented `xique-serve.json` — so the serve config was never applied and the tailnet URL proxied to nothing (confirmed live: renaming the host copy to `xique.json` made it work). The unit now expects `/config/xique-serve.json`, which is what the shipped file is named and what every doc already says to copy. `docs/install.md` gains a note that the filename is load-bearing so the next operator does not silently rename it. Docs and unit now agree in one direction, and the host workaround gets reverted as part of adopting the fix (rename back + copy the updated unit). Closes #39
🐛 fix: point the tailscale sidecar at the mounted serve config
Some checks failed
/ gitleaks (pull_request) Successful in 16s
/ pr-review (pull_request) Successful in 51s
/ checks (pull_request) Failing after 3m52s
/ container (pull_request) Has been skipped
/ publish-container (pull_request) Has been skipped
/ deploy-docs (pull_request) Has been skipped
8460acb5ea
marvin8 approved these changes 2026-10-17 05:52:13 +00:00
marvin8 manually merged commit 92470c5d45 into main 2026-10-17 05:52:51 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
marvin8/xique!40
No description provided.