Add persistence info to start-dev banner#1033
Conversation
|
|
|
A few suggestions on the banner format:
Suggested output: And for file-backed: |
|
Please make sure you have signed the Temporal CLA before this can be merged. |
|
Applied all four banner suggestions in 2b09ade: |
Signed-off-by: Sai Asish Y <say.apm35@gmail.com>
What was changed
temporal server start-devnow prints aPersistence:line at the end of the startup banner.In-memory (no
--db-filename):File-backed:
Why?
Per #634, the start-dev banner did not surface what form of persistence the dev server uses, so users could not tell at a glance whether Workflow Executions would survive a restart.
Checklist
Closes [Feature Request] Add persistence information to
start-devbanner #634How was this tested:
Two new tests in
internal/temporalcli/commands.server_test.gostart the dev server, capture stdout, and assert that the banner containsPersistence:plus the expected backend description. Both pass locally and fail when the banner change is reverted.