Helm quickstart: db schema init fails, dbscripts dir uses hyphens not underscores

[senthalan]

Description

The Helm tab of the "Get started" guide (guides/getting-started/get-thunderid) creates databases named runtime_transient and runtime_persistent (underscores), then applies schemas with:

for db in configdb runtime_transient entitydb runtime_persistent; do
  kubectl exec thunderid-dbinit -- cat /opt/thunderid/dbscripts/$db/postgres.sql | \
    kubectl exec -i postgresql-0 -- env PGPASSWORD=dbpassword psql -U dbuser -d $db
done

This fails for runtime_transient and runtime_persistent because the schema directories shipped in the release image are named runtime-transient and runtime-persistent (hyphens), not with underscores. configdb and entitydb work fine since those names match their directories exactly.

Steps to Reproduce

1. Follow the Helm tab of the "Get started" guide
2. Run the "Apply the database schemas" step
3. The command fails with something like:

   cat: /opt/thunderid/dbscripts/runtime_transient/postgres.sql: No such file or directory
   

   (same for runtime_persistent)

Version

Latest alpha release

Environment Details (with versions)

• Kubernetes (Helm)
• PostgreSQL (Bitnami chart, oci://registry-1.docker.io/bitnamicharts/postgresql)

[indeewari]

Hi @senthalan ,

There are two ways to fix this

1. Docs-only translation — keep the kebab directories, translate in the loop: db=${schema//-/_}. Minimal, ships immediately.
2. Align the physical artifacts to underscore — rename the two dbscripts/ directories (and the SQLite files) to runtime_transient / runtime_persistent, so the directory name equals the Postgres database name and the config key for all four databases.

I would like to suggest the Option 2 as it

• Removes the root inconsistency; one form across dir, SQLite file, config key, and pg name.
• The Helm quickstart loop needs no translation logic.
• No silent-empty-database trap for future renames.

Cons

• Larger radius than the docs fix: dbscripts dirs, SQLite paths, build scripts, integration test config, the OpenChoreo charts, and the integration-test CI action.
• Revisits a naming decision from the already-merged Rename operation database to runtime-persistent and runtime database to runtime-transient #4077.
• Version skew: the docs' simple loop only works against an image built after this change, so :latest must be re-released before the guide is correct end to end.