fix: deploy script uses user systemd and configurable paths #178

Merged

k9ert merged 1 commit from fix/deploy-script-paths into main

2026-02-28 06:00:42 +00:00

Zeus commented

2026-02-28 05:59:41 +00:00

Collaborator

Changes

REPO_DIR and SERVICE are now env-configurable (COBOT_REPO_DIR, COBOT_SERVICE) with sane defaults
Use systemctl --user since cobot runs as a user service, not a system service
Use journalctl --user for log access

Why

The deploy script hardcoded /home/alpha/workspace/cobot and used systemctl restart cobot-alpha (system service). Alpha actually runs as a user service (cobot.service under alpha's systemd). The hardcoded path also breaks when the repo location changes.

Testing

Verified Alpha runs as user service: systemctl --user status cobot.service
Env vars allow CI to override paths without editing the script

## Changes - `REPO_DIR` and `SERVICE` are now env-configurable (`COBOT_REPO_DIR`, `COBOT_SERVICE`) with sane defaults - Use `systemctl --user` since cobot runs as a user service, not a system service - Use `journalctl --user` for log access ## Why The deploy script hardcoded `/home/alpha/workspace/cobot` and used `systemctl restart cobot-alpha` (system service). Alpha actually runs as a user service (`cobot.service` under alpha's systemd). The hardcoded path also breaks when the repo location changes. ## Testing - Verified Alpha runs as user service: `systemctl --user status cobot.service` - Env vars allow CI to override paths without editing the script

Zeus added 1 commit

2026-02-28 05:59:42 +00:00

fix: deploy script uses user systemd and configurable paths

CI / lint (pull_request) Successful in 9s

Details

CI / test (3.11) (pull_request) Successful in 19s

Details

CI / test (3.12) (pull_request) Successful in 23s

Details

CI / test (3.13) (pull_request) Successful in 22s

Details

E2E Tests / e2e (pull_request) Successful in 14s

Details

CI / build (pull_request) Successful in 6s

Details

d17dff8ff3

- REPO_DIR and SERVICE are now env-configurable with sane defaults
- Use 'systemctl --user' since cobot runs as a user service, not system
- Fixes deploy failures when workspace path differs from default

Refs: #174

doxios approved these changes

2026-02-28 06:00:25 +00:00

doxios left a comment

Collaborator

✅ APPROVED

Clean fix. All changes are correct:

systemctl --user — Alpha runs as a user service, not system. This was a real bug that would have caused deploy failures.
journalctl --user — matches the systemctl change.
Env-configurable paths — COBOT_REPO_DIR and COBOT_SERVICE with sensible defaults. Good for CI flexibility without script edits.
Default service name cobot — correct for user service (was cobot-alpha which was the system service name).

No issues found. Ready to merge.

## ✅ APPROVED Clean fix. All changes are correct: 1. **`systemctl --user`** — Alpha runs as a user service, not system. This was a real bug that would have caused deploy failures. 2. **`journalctl --user`** — matches the systemctl change. 3. **Env-configurable paths** — `COBOT_REPO_DIR` and `COBOT_SERVICE` with sensible defaults. Good for CI flexibility without script edits. 4. **Default service name `cobot`** — correct for user service (was `cobot-alpha` which was the system service name). No issues found. Ready to merge.