Skip to content

Add local TiDB smoke workflow#263

Open
okJiang wants to merge 3 commits into
mainfrom
at-kay/local-server-smoke
Open

Add local TiDB smoke workflow#263
okJiang wants to merge 3 commits into
mainfrom
at-kay/local-server-smoke

Conversation

@okJiang
Copy link
Copy Markdown
Contributor

@okJiang okJiang commented Apr 22, 2026

Summary

Replaces the previous local PostgreSQL smoke lane with a TiDB-only local smoke path.

This PR now:

  • removes the standalone PG workflow/lane
  • adds e2e/local-tidb-smoke.sh
  • adds .github/workflows/local-tidb-smoke.yml
  • keeps shared API assertions in e2e/local-smoke-checks.sh

The TiDB smoke boots a local TiUP playground cluster, seeds an active tenant directly in the control-plane schema, starts mnemo-server with MNEMO_DB_BACKEND=tidb, and verifies the v1alpha2 memories API on the real TiDB backend path.

Covered flow:

  • healthz and versionz
  • auth failure guardrails
  • create memory with direct content
  • list by tag and extract the created memory ID
  • keyword recall/search with confidence output
  • get, update, delete, and deleted-get verification

CI

Adds .github/workflows/local-tidb-smoke.yml, triggered by PR/push changes to server, the TiDB smoke scripts, or the workflow itself, plus manual workflow_dispatch. It uploads TiDB/server/smoke logs as artifacts.

Validation

  • bash -n e2e/local-smoke-checks.sh
  • bash -n e2e/local-tidb-smoke.sh
  • YAML parse for .github/workflows/local-tidb-smoke.yml
  • Local TiUP playground + mnemo-server run: 19 / 19 passed
  • GitHub Actions: Local TiDB smoke green on commit 6ffd706

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 22, 2026

👷 Deploy request for mem9 pending review.

A Netlify team Owner will need to approve the deploy before you can run your build.

Are you a team Owner? Visit the deploys page to approve it

Need more help? Learn more in the Netlify docs

Name Link
🔨 Latest commit 6ffd706

@okJiang okJiang marked this pull request as ready for review April 22, 2026 09:59
Copy link
Copy Markdown
Contributor Author

@okJiang okJiang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Clean tier 2 implementation. pgvector + direct tenant seed gives us API coverage without external dependencies. 19/19 local validation, actionlint clean, artifact upload on failure. Good follow-through on the three-tier plan.

@okJiang okJiang changed the title Add local server smoke workflow Add local TiDB smoke workflow Apr 24, 2026
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