Skip to content

Surface persistence in start and status commands#244

Open
gtsiolis wants to merge 2 commits into
mainfrom
des-230-surface-persistence-state-in-status-and-start-commands
Open

Surface persistence in start and status commands#244
gtsiolis wants to merge 2 commits into
mainfrom
des-230-surface-persistence-state-in-status-and-start-commands

Conversation

@gtsiolis
Copy link
Copy Markdown
Member

@gtsiolis gtsiolis commented May 13, 2026

Adds a Persistence: Enabled bullet to the start info block and the status output when the running instance has LOCALSTACK_PERSISTENCE=1 set — either via --persist or by inheriting the env var from the host shell. restart benefits transitively since it dispatches through start.

BEFORE AFTER
Screenshot 2026-05-13 at 16 39 33 Frame 852839935
Screenshot 2026-05-13 at 16 39 38 Frame 852839936

@carole-lavillonniere
Copy link
Copy Markdown
Collaborator

@gtsiolis can you open separate PRs for the other changes? There seem to be some changes about the displayed host and some snowflake header changes? Not sure. It's a little hard to read at the moment.

@gtsiolis
Copy link
Copy Markdown
Member Author

gtsiolis commented May 13, 2026

Makes sense. I had a feeling I would have to split this one, will split later. ⏳

@gtsiolis gtsiolis marked this pull request as draft May 13, 2026 14:46
@gtsiolis gtsiolis force-pushed the des-230-surface-persistence-state-in-status-and-start-commands branch from 79e001f to b046e9b Compare May 13, 2026 17:36
@gtsiolis gtsiolis force-pushed the des-230-surface-persistence-state-in-status-and-start-commands branch from b046e9b to dfac5a0 Compare May 13, 2026 17:44
@gtsiolis gtsiolis marked this pull request as ready for review May 13, 2026 17:57
@gtsiolis
Copy link
Copy Markdown
Member Author

Updated PR to include only the Persistence metadata in the header without additional cleanup, plus show the metadata only for AWS emulation. Curious what you think!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants