Skip to content

fix(server): prevent migration lock transactions#224

Merged
abhinav-galileo merged 2 commits into
mainfrom
fix/migration-lock-autocommit
May 15, 2026
Merged

fix(server): prevent migration lock transactions#224
abhinav-galileo merged 2 commits into
mainfrom
fix/migration-lock-autocommit

Conversation

@abhinav-galileo

Copy link
Copy Markdown
Collaborator

Summary

  • run Alembic with transaction_per_migration to match Galileo API migration behavior
  • keep the migration advisory-lock connection in AUTOCOMMIT so it does not block CREATE INDEX CONCURRENTLY
  • assert the advisory-lock engine is created with AUTOCOMMIT

Validation

  • uv run --package agent-control-server pytest server/tests/test_migrate.py -q
  • uv run ruff check --config pyproject.toml server/alembic/env.py server/src/agent_control_server/migrate.py server/tests/test_migrate.py
  • git diff --check

@codecov

codecov Bot commented May 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Comment thread server/alembic/env.py
@abhinav-galileo abhinav-galileo merged commit e65a2f4 into main May 15, 2026
6 checks passed
@abhinav-galileo abhinav-galileo deleted the fix/migration-lock-autocommit branch May 15, 2026 17:26
galileo-automation pushed a commit that referenced this pull request May 28, 2026
## [2.6.0](ts-sdk-v2.5.0...ts-sdk-v2.6.0) (2026-05-28)

### Features

* **evaluators:** add new lluna client ([#213](#213)) ([f65beb9](f65beb9))
* **sdk:** add otel support ([#177](#177)) ([9530368](9530368))
* **sdk:** add runtime token auth ([#215](#215)) ([6cc0f38](6cc0f38))
* **server:** add control clone-and-bind endpoint ([#229](#229)) ([1728bf9](1728bf9))
* **server:** add runtime auth and namespace scoping ([#214](#214)) ([56e44fe](56e44fe))
* **server:** allow host-owned logging setup ([#227](#227)) ([c0fd159](c0fd159))
* **server:** bundle migrations in wheel and add agent-control-migrate ([#209](#209)) ([8c5c35e](8c5c35e))
* **server:** migrate controls routes to auth framework ([#212](#212)) ([764bd4b](764bd4b))

### Bug Fixes

* **examples:** declare local SDK workspace deps ([#222](#222)) ([d22aa1d](d22aa1d))
* **sdk:**  Get trace context from provider ([#211](#211)) ([1efe30f](1efe30f))
* **sdk-ts:** normalize generated client ([#231](#231)) ([1c097d2](1c097d2))
* **server:** make observability migration retry-safe ([#226](#226)) ([b9dd00d](b9dd00d))
* **server:** prevent migration lock transactions ([#224](#224)) ([e65a2f4](e65a2f4))
* **server:** scope auth upstream CA to HTTP provider ([#232](#232)) ([7a0ce21](7a0ce21))
* **ui:** fix editing of controls in the UI ([#218](#218)) ([981e33d](981e33d)), closes [#Risk](https://github.com/agentcontrol/agent-control/issues/Risk)
@galileo-automation

Copy link
Copy Markdown
Collaborator

🎉 This PR is included in version 2.6.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants