Skip to content

LKMA-203 deploy runbook updates - #22

Merged
nicolasumaras merged 1 commit into
mainfrom
lkma-203-deploy-runbook-updates
Jun 26, 2026
Merged

LKMA-203 deploy runbook updates#22
nicolasumaras merged 1 commit into
mainfrom
lkma-203-deploy-runbook-updates

Conversation

@nicolasumaras

Copy link
Copy Markdown
Owner

Summary

  • Document Debian Bookworm packages for the SpanDSP worker, pppd, nftables, and iptables-nft fallback
  • Install the built softmodem worker to /opt/sipfax/bin/sipfax-softmodem and align PPP hook paths with /etc/ppp/ip-up.d/sipfax and /etc/ppp/ip-down.d/sipfax
  • Document per-call chap-secrets/pap-secrets lifecycle plus operator verification commands for systemd, /healthz, pppd, nftables, and media.modem
  • Add deploy asset tests covering the runbook and installer expectations

Verification

  • node --test test/deploy-assets.test.js
  • bash -n deploy/install-systemd.sh

Issue: LKMA-203

@nicolasumaras

Copy link
Copy Markdown
Owner Author

Reviewer verdict: APPROVE on head d361ce4.

(Posting as a comment rather than a review because GitHub blocks self-approval on this repo — the PR author and reviewer share a git identity.)

Verified against LKMA-203 deliverables:

  • Bookworm apt section installs libspandsp2 + libspandsp-dev + build-essential + pkg-config + python3 + ppp + nftables + iptables, with rationale.
  • /opt/sipfax/bin/sipfax-softmodem install in install-systemd.sh creates the dir 0755 root:sipfax and installs the binary 0755 root:sipfax; installer aborts cleanly if the vendored binary is missing.
  • pppd hooks installed at /etc/ppp/ip-up.d/sipfax and /etc/ppp/ip-down.d/sipfax with the privilege boundary called out for Operator review.
  • chap-secrets/pap-secrets per-call 0600 lifecycle documented in both runbook and sipfax.env.example.
  • All five verification commands present: systemctl status sipfax, curl /healthz, pppd --version, nft list ruleset | head, and jq '.media.modem' control-fd snapshot.

Independent checks on the exact PR head:

  • bash -n deploy/install-systemd.sh: OK.
  • node --test test/deploy-assets.test.js: 5/5 pass.
  • GitHub CI 'test' check: SUCCESS.
  • mergeStateStatus: CLEAN, mergeable: MERGEABLE.

Merging now.

@nicolasumaras
nicolasumaras merged commit 8894d10 into main Jun 26, 2026
1 check passed
@nicolasumaras
nicolasumaras deleted the lkma-203-deploy-runbook-updates branch June 26, 2026 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant