Skip to content

Startup: serialize plugin schema initialization - #1115

Open
zardus wants to merge 1 commit into
masterfrom
codex/serialize-dojo-schema-startup
Open

Startup: serialize plugin schema initialization#1115
zardus wants to merge 1 commit into
masterfrom
codex/serialize-dojo-schema-startup

Conversation

@zardus

@zardus zardus commented Jul 26, 2026

Copy link
Copy Markdown
Member

Summary

  • serialize plugin-owned db.metadata.create_all() calls with a PostgreSQL transaction-scoped advisory lock
  • retain the existing behavior on non-PostgreSQL databases
  • cover concurrent first-start schema creation with a regression test

Scope

This is shared application-startup hardening. It is intentionally separate from CI PR #1114 and is not a prerequisite for any of the seven feature PRs.

Validation

  • concurrent schema-creation regression
  • Python syntax and whitespace checks
  • full single-node and multi-node suites in GitHub Actions

@codecov

codecov Bot commented Jul 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

zardus added a commit that referenced this pull request Jul 26, 2026
## Summary

- make the stats-worker cold-start regression deterministic by stopping
the worker, clearing the target cache, and verifying its exact
cold-start initialization
- make database-shell errors fail tests immediately and surface
PostgreSQL diagnostics
- raise the test-step limit from 15 to 20 minutes; recent remote
#1111/#1113 test steps took up to 16m33s

## Scope

This PR is strictly CI/test-only. It changes exactly:

- `.github/workflows/ci.yml`
- `test/test_background_stats.py`
- `test/utils.py`

It contains no runtime, production, image-build, Compose, or dependency
changes.

## Validation

- Python syntax check
- workflow YAML parse
- whitespace check
- full single-node and multi-node suites in GitHub Actions

## Stack

The seven feature PRs (#1105, #1106, #1108, #1110, #1111, #1112, and
#1113) each contain one feature-only commit directly above this branch.

Shared schema-startup hardening is isolated in #1115. Image-build
patch-runner hardening is isolated in #1116. Neither is a prerequisite
for any feature PR.

Merge this PR first, then retarget the nine sibling PRs to `master`
before deleting this branch.
Base automatically changed from codex/ci-harden-dojo-tests to master July 26, 2026 22:31
@zardus
zardus force-pushed the codex/serialize-dojo-schema-startup branch from d864195 to a6087eb Compare July 26, 2026 22:33
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