feat(gateway): real FreeBSD rc.d detection in get_gateway_runtime_snapshot #7

Merged

clawdie merged 1 commit from feat/gateway-freebsd-rcd-detection into main 2026-06-21 11:44:21 +02:00

Conversation 0 Commits 1 Files changed 1 +37

clawdie commented 2026-06-21 11:06:17 +02:00

Owner

Copy link

Deeper fix behind PR #6. get_gateway_runtime_snapshot() only had systemd/launchd branches, so on FreeBSD it fell through to manager="manual process" — hermes status couldn't report the real gateway state.

• is_freebsd() helper.
• _probe_freebsd_service_running() — probes the hermes_daemon rc.d service via service hermes_daemon onestatus (reports running regardless of rcvar enable; works without root; fail-safe), mirroring the launchd probe.
• snapshot branch: manager="rc.d (hermes_daemon)", service_installed (rc.d file present), service_running (onestatus), service_scope="rc.d".

Now hermes status reports installed/running on FreeBSD via the primary path (line 484), not just the fallback string.

Not in scope: get_managed_gateway_pids() restart-drain (systemd/launchd only) — liveness already works via find_gateway_pids(); rc.d restart-drain is a separate follow-up.

py_compile clean; wiring + branch ordering verified. (Runtime-verify on a FreeBSD host.)

🤖 Generated with Claude Code

Deeper fix behind PR #6. `get_gateway_runtime_snapshot()` only had systemd/launchd branches, so on FreeBSD it fell through to `manager="manual process"` — `hermes status` couldn't report the real gateway state. - **`is_freebsd()`** helper. - **`_probe_freebsd_service_running()`** — probes the `hermes_daemon` rc.d service via `service hermes_daemon onestatus` (reports running regardless of rcvar enable; works without root; fail-safe), mirroring the launchd probe. - **snapshot branch:** `manager="rc.d (hermes_daemon)"`, `service_installed` (rc.d file present), `service_running` (onestatus), `service_scope="rc.d"`. Now `hermes status` reports installed/running on FreeBSD via the primary path (line 484), not just the fallback string. Not in scope: `get_managed_gateway_pids()` restart-drain (systemd/launchd only) — liveness already works via `find_gateway_pids()`; rc.d restart-drain is a separate follow-up. `py_compile` clean; wiring + branch ordering verified. (Runtime-verify on a FreeBSD host.) 🤖 Generated with [Claude Code](https://claude.com/claude-code)

clawdie added 1 commit 2026-06-21 11:06:19 +02:00

feat(gateway): real FreeBSD rc.d detection in get_gateway_runtime_snapshot ... 4e7a55704d

`hermes status` reported the gateway as "manual process" on FreeBSD because
get_gateway_runtime_snapshot() only had systemd/launchd branches and fell
through. Add a proper FreeBSD rc.d path so status reports real state:

- is_freebsd() helper (sys.platform.startswith("freebsd")).
- _freebsd_rcd_service_path() + _probe_freebsd_service_running() — probes the
  hermes_daemon rc.d service via `service hermes_daemon onestatus` (reports
  running regardless of the rcvar enable flag; works without root; fail-safe on
  timeout/missing binary), mirroring the launchd probe.
- snapshot branch: manager="rc.d (hermes_daemon)", service_installed (rc.d file
  present), service_running (onestatus), service_scope="rc.d".

This is the deeper fix behind the status.py fallback message (PR #6). Not changed:
get_managed_gateway_pids() restart-drain path (systemd/launchd only) — gateway
liveness already works via find_gateway_pids(); rc.d restart-drain is a separate
follow-up.

py_compile clean; wiring + branch-ordering verified.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

clawdie merged commit 83830b6dcc into main 2026-06-21 11:44:21 +02:00

clawdie referenced this pull request from a commit 2026-06-21 11:44:26 +02:00

Merge pull request 'feat(gateway): real FreeBSD rc.d detection in get_gateway_runtime_snapshot' (#7) from feat/gateway-freebsd-rcd-detection into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: clawdie/hermes-bsd#7

No description provided.