Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
51 changes: 51 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,51 @@
# Dependabot configuration.
#
# This repo had NO dependency watching until 2026-09-09, which is the
# same gap that produced the Sentinel AI agent's three production
# breakages in two days: mcp 2.x renaming a symbol, Starlette removing
# on_startup, and a ceiling added to stop the second one silently
# pinning the service to a Starlette line with 7 advisories. All three
# surfaced on a rebuild of a repo nobody had rebuilt since June.
#
# `pip-audit --strict` in the Test workflow catches a *known advisory*
# in what is already pinned. It does nothing about a lockfile quietly
# ageing out of support. That is what this is for.
#
# NO AUTO-MERGE WORKFLOW HERE, DELIBERATELY.
#
# This repo now deploys to Fly on push to master. GitHub does not
# trigger `on: push` workflows for commits pushed with GITHUB_TOKEN, so
# an auto-merge action would land a bump on master WITHOUT deploying it
# — master and production drift apart, and nothing reports an error.
# That is a real, observed bug in Sentinel-Command (four frontend bumps
# merged and never shipped on 2026-09-09). A human merge triggers the
# deploy normally, so these PRs are reviewed and merged by hand until
# that repo's PAT fix is proven.
#
# Extra reason to be careful here: this service is Postgres-native and
# its tests need a real Postgres container, so a driver bump that only
# misbehaves against the real engine will not show up anywhere else.
version: 2
updates:
# Python (uv) — pyproject.toml + uv.lock live at the repo root.
- package-ecosystem: "uv"
directory: "/"
schedule:
interval: "weekly"
day: "monday"
open-pull-requests-limit: 5
commit-message:
prefix: "build(deps)"
labels:
- "dependencies"

# GitHub Actions — pinned action versions age out the same way.
- package-ecosystem: "github-actions"
directory: "/"
schedule:
interval: "weekly"
day: "monday"
open-pull-requests-limit: 5
labels:
- "dependencies"
- "ci"
44 changes: 38 additions & 6 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,15 @@
name: Test
name: Test & Deploy

# Deploy automation is deferred deliberately, same rationale as
# Sentinel-License-Service's workflow of the same name: a human should
# be watching the first-ever deploy of new infrastructure (including,
# for this service, the first-ever Postgres instance in this family).
# This workflow only covers lint + dependency audit + tests.
# Tests on every push and PR; deploys to Fly on pushes to master.
#
# Deploy automation used to be deferred here so a human could watch the
# first-ever deploy of new infrastructure. That rationale expired once
# the service was live, and leaving it deferred created a worse problem:
# fly.toml became a file that did nothing. A scale-to-zero change was
# merged with CI green on 2026-09-09 and simply never reached Fly —
# the app kept running always-on until someone noticed and deployed by
# hand. Config that silently doesn't apply is more dangerous than no
# config, so this now ships.
#
# Unlike License-Service's version, this one needs a real Postgres
# service container — JSONB isn't SQLite-portable, and this service is
Expand Down Expand Up @@ -63,3 +68,30 @@ jobs:

- name: Run tests
run: uv run pytest -v

deploy:
name: Deploy to Fly.io
runs-on: ubuntu-latest
needs: test
# Push-only: a PR runs the tests above but never ships.
if: github.event_name == 'push'
# Serialize so two quick pushes don't race on the machine update.
concurrency:
group: deploy-sentinel-sync
cancel-in-progress: true
steps:
- uses: actions/checkout@v7

- uses: superfly/flyctl-actions/setup-flyctl@master

# --ha=false: Fly provisions TWO machines by default and this
# service needs one. It did exactly that on the 2026-09-09 manual
# deploy and the extra machine had to be scaled away by hand.
#
# No --strategy override: this app has no volume, so the default
# rolling strategy works. (Command Center needs `immediate`
# because its web machine mounts a single-attachment volume.)
- name: flyctl deploy
run: flyctl deploy --remote-only --ha=false --yes
env:
FLY_API_TOKEN: ${{ secrets.FLY_API_TOKEN }}
16 changes: 14 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,20 @@ Full procedure, including what deliberately isn't mirrored (node API keys, evide

## Deploy

Single-stage `Dockerfile` (no frontend build — this service has no UI). `fly.toml`'s `release_command` runs `alembic upgrade head` before each deploy starts serving traffic. Needs a `DATABASE_URL` Fly secret pointing at a real Postgres instance (Fly Postgres, Neon, RDS, etc.) — provisioning that instance is a separate infra/cost decision, not something this repo does for you.
Single-stage `Dockerfile` (no frontend build — this service has no UI). `fly.toml`'s `release_command` runs `alembic upgrade head` before each deploy starts serving traffic.

**Deploys from CI.** Every push to `master` runs the tests against a real Postgres, then `flyctl deploy --ha=false`. (`--ha=false` because Fly provisions two machines by default; it did exactly that on the manual deploy and the extra had to be scaled away by hand. No `--strategy` override is needed here — unlike the sibling License service, this app has no volume.)

Deploy automation was deferred while this was new infrastructure. That turned out worse than what it avoided: `fly.toml` became a file that did nothing, and a scale-to-zero change merged with CI fully green on 2026-09-09 without ever reaching Fly.

**Scales to zero.** Self-hosted installs push on a 30-minute background tick, so this is idle ~95% of the time. Safe because boot is ~3s (inside the ~8s Fly's proxy waits for an auto-started machine to bind) and a failed push is fail-soft *and lossless*: `push_pending_changes` never raises, and cursors only advance on confirmed success, so a missed cycle's data simply waits for the next tick with the operator's local SQLite authoritative throughout.

## Status

Built and verified locally: full test suite (unit tests against a real Postgres, including tenant-isolation and deletion-reconciliation regressions) plus a live cross-service integration check against a real running License-Service instance (valid+sync-enabled key → 200 and the row lands correctly scoped by tenant; unknown key → 403; License-Service unreachable → 502). Not yet deployed — no Postgres instance or Fly app provisioned yet.
**Deployed and live** at `https://sentinel-sync.fly.dev`, on the shared `sentinel-postgres` cluster in its own `sentinel_sync` database, access-isolated by role from the other two.

Verified by the full test suite (unit tests against a real Postgres, including tenant-isolation and deletion-reconciliation regressions) plus a live cross-service integration check against a running License-Service instance: valid + sync-enabled key → 200 with the row correctly scoped by tenant; unknown key → 403; License-Service unreachable → 502.

This section previously read "Not yet deployed — no Postgres instance or Fly app provisioned yet", which stopped being true on 2026-09-07.

**No backup dump job, deliberately.** This database holds a *mirror*; every row was pushed from an operator's local SQLite, which stays the source of truth. Losing it entirely costs one sync cycle. It is covered by the cluster-level snapshot — see `DISASTER_RECOVERY.md` in the Command Center repo.