Skip to content

Use forked go-workflows migration table option#543

Draft
rmorlok wants to merge 1 commit into
mainfrom
codex/workflows-fork-migrations
Draft

Use forked go-workflows migration table option#543
rmorlok wants to merge 1 commit into
mainfrom
codex/workflows-fork-migrations

Conversation

@rmorlok

@rmorlok rmorlok commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Temporarily replace github.com/cschleiden/go-workflows with rmorlok/go-workflows at v0.0.0-20260606194005-c8be9ef9bb4e
  • Remove AuthProxy's copied go-workflows SQL migrations and custom golang-migrate runner
  • Configure go-workflows sqlite/postgres backends with authproxy_workflows_schema_migrations via WithMigrationsTable
  • Add a SQLite regression test proving an existing main schema_migrations table at version 10 no longer collides with workflow migrations

Draft status

This is intentionally draft-only while the go-workflows migration-table option lives in the rmorlok fork. Once the upstream repo has the option, this PR should drop the replace and move back to the upstream module/version.

Refs #539.

Verification

  • go test ./internal/workflows ./internal/core ./internal/routes ./internal/tasks ./internal/service/worker ./internal/service/api ./internal/service/public
  • go test -tags integration ./oauth2 -run DisconnectRevocation -count=1 (from integration_tests)
  • ./scripts/preflight.sh
  • go test ./...
  • push hook preflight passed

@rmorlok rmorlok added clone:authproxy3 PR opened from the ~/src/authproxy3 clone project:workflows go-workflows integration and workflow-backed background processing labels Jun 6, 2026
@rmorlok

rmorlok commented Jun 6, 2026

Copy link
Copy Markdown
Owner Author

Blocked on cschleiden/go-workflows#485 merging.

@rmorlok rmorlok added the deploy:demo Deploy a per-PR demo environment label Jun 8, 2026
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

Dev environment is ready

Item Value
URL https://codex-workflows-fork-migrations.dev.authproxy.net/
Namespace dev-codex-workflows-fork-migrations
Image tag pr-543

This comment is updated after each successful deploy.

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

Labels

clone:authproxy3 PR opened from the ~/src/authproxy3 clone deploy:demo Deploy a per-PR demo environment project:workflows go-workflows integration and workflow-backed background processing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant