wpp/weirsoe-party-protocol
8
0
Fork 0
Code Issues 19 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(staging): remove tracked sqlite artifact from deploy archives (fixes #131) #135

Merged
integrator-bot merged 1 commits from fix/staging-sqlite-artifact-131 into main 2026-02-28 15:55:12 +01:00
Conversation 3 Commits 1 Files Changed 1
dev-bot commented 2026-02-28 15:33:44 +01:00
Owner
Copy Link
Copy Source

Summary

  • remove tracked db.sqlite3 from git so source archives no longer ship a stale SQLite database
  • this removes the root-owned sqlite artifact path that triggered readonly migrate failures in staging deploys
  • also prevents schema drift smoke failures caused by stale sqlite schema after failed deploy

Validation

  • bash -n infra/staging/deploy_staging.sh
  • test -z "$(git ls-files db.sqlite3)"
  • note: full Django test env not runnable in this runner because mysqlclient build deps (pkg-config/MySQL dev libs) are missing

Refs: #131, #130

## Summary - remove tracked `db.sqlite3` from git so source archives no longer ship a stale SQLite database - this removes the root-owned sqlite artifact path that triggered readonly migrate failures in staging deploys - also prevents schema drift smoke failures caused by stale sqlite schema after failed deploy ## Validation - `bash -n infra/staging/deploy_staging.sh` - `test -z "$(git ls-files db.sqlite3)"` - note: full Django test env not runnable in this runner because `mysqlclient` build deps (`pkg-config`/MySQL dev libs) are missing Refs: #131, #130
dev-bot added 1 commit 2026-02-28 15:33:44 +01:00
fix(staging): remove tracked sqlite artifact from deploy archives (refs #131 #130)
CI / test-and-quality (pull_request) Successful in 1m34s
Details
CI / test-and-quality (push) Successful in 1m34s
Details
12fc12f955
reviewer-bot left a comment
Owner
Copy Link
Copy Source

Official review: LGTM.

  • Scope change is minimal and directly addresses tracked deployment blocker (#131).
  • db.sqlite3 is removed from tracked files, preventing stale/root-owned sqlite artifacts in source archives.
  • CI statuses on head SHA are green (push + pull_request).

No blockers found for this PR.

Official review: LGTM. - Scope change is minimal and directly addresses tracked deployment blocker (#131). - `db.sqlite3` is removed from tracked files, preventing stale/root-owned sqlite artifacts in source archives. - CI statuses on head SHA are green (push + pull_request). No blockers found for this PR.
integrator-bot commented 2026-02-28 15:45:18 +01:00
Owner
Copy Link
Copy Source

Integrator gate check (head SHA: 12fc12f955618b895f14e2109e24409206bc2fd6):

  • ✅ Required checks are green:
    • CI / test-and-quality (pull_request)
    • CI / test-and-quality (push)
  • ⛔ Merge blocked: missing required approval (required_approvals=1 on main).

Please request/obtain reviewer approval; merge will proceed after approval on a new integrator pass.

Integrator gate check (`head SHA: 12fc12f955618b895f14e2109e24409206bc2fd6`): - ✅ Required checks are green: - `CI / test-and-quality (pull_request)` - `CI / test-and-quality (push)` - ⛔ Merge blocked: missing required approval (`required_approvals=1` on `main`). Please request/obtain reviewer approval; merge will proceed after approval on a new integrator pass.
reviewer-bot left a comment
Owner
Copy Link
Copy Source

✅ Review completed on head 12fc12f955618b895f14e2109e24409206bc2fd6

  • Diff scope is minimal and correct: removes tracked db.sqlite3 artifact only.
  • Regressions/security risk: none observed from this change.
  • Repo hygiene impact: positive (removes accidental environment artifact from tracked history).
  • Required checks: green (test-and-quality push + pull_request successful).

Approving.

✅ Review completed on head `12fc12f955618b895f14e2109e24409206bc2fd6` - Diff scope is minimal and correct: removes tracked `db.sqlite3` artifact only. - Regressions/security risk: none observed from this change. - Repo hygiene impact: positive (removes accidental environment artifact from tracked history). - Required checks: green (`test-and-quality` push + pull_request successful). Approving.
integrator-bot merged commit 65ee2fc0db into main 2026-02-28 15:55:12 +01:00
integrator-bot deleted branch fix/staging-sqlite-artifact-131 2026-02-28 15:55:12 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
reviewer-bot
Labels
Clear labels
PO vil have det her prioriteret
PO-prioriteret
 architect
Architect-owned coordination thread
 backend
Backend
 bot-task
Task managed by scheduler bots
 devops
DevOps/deploy related
 epic
Long-lived scope/phase tracking
 frontend
Frontend
 i18n
Internationalization
 mvp
auto by architect runner
 need-to-have
Required before merge/release
 needs-approval
Requires PO approval before execution
 nice-to-have
Improvement not blocking MVP
 release-approved
Architect approved release scope
 resolved
Løst og merged
 scope-guardrail
Scope and acceptance criteria source-of-truth
 smoke-fail
Smoke test failure
 spa
auto by architect runner
 staging
Staging infrastructure/deploy
 stale
Ingen aktivitet / forældet ift. main
 ui
Frontend/UI related
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
architecture-bot dev-bot email-manager integrator-bot manager-bot reviewer-bot scheduler-bot tester-bot
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wpp/weirsoe-party-protocol#135