Skip to content

Migrate from the localstack CLI to lstk - #28

Merged
HarshCasper merged 4 commits into
mainfrom
devx-1056-migrate-to-lstk
Sep 4, 2026
Merged

Migrate from the localstack CLI to lstk#28
HarshCasper merged 4 commits into
mainfrom
devx-1056-migrate-to-lstk

Conversation

@HarshCasper

Copy link
Copy Markdown
Member

Summary

  • Replace localstack/awslocal/cdklocal invocations with lstk across the Makefile, README, npm scripts, and CI workflows.
  • Replace the setup-localstack action in the main CI workflow with npm install -g @localstack/lstk + lstk start.
  • Disable the PR Preview workflow's automatic trigger — Ephemeral Instances / App Preview aren't supported by lstk yet (same treatment as the quiz app).
  • Remove the dead tflocal install step; this repo has no Terraform.

Testing

  • CI workflow run on this branch.

Part of DEVX-1056

Comment thread .github/workflows/preview.yml Outdated
# Disabled: `lstk aws`/`lstk cdk` (used via the Makefile targets that preview-cmd
# below calls) always target their own locally-managed emulator and ignore
# AWS_ENDPOINT_URL, so they can't reach the remote ephemeral instance this
# workflow provisions. Re-enable once that's resolved.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is already resolved, although the lstk docs haven't been updated to reflect that AWS_ENDPOINT_URL is now recognized.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Done — re-enabled the pull_request trigger and removed the comment block.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Re-enabled, and lstk does reach the ephemeral instance now (all CloudFormation calls succeed). The run still fails one step later though: CDK asset publishing to S3 gets a TLS failure (write EPROTO ... tlsv1 alert internal error) — the upload uses virtual-hosted-style addressing (<bucket>.ls-xxx.sandbox.localstack.cloud), a second subdomain level the sandbox wildcard cert doesn't cover. Looks like a platform-side limitation rather than something to patch in the sample; left the trigger enabled so the behavior is visible. Happy to re-disable if the red check is too noisy until that's fixed.

Comment thread .github/workflows/preview.yml Outdated
Comment thread README.md Outdated
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

⚡️ Running CI build with LocalStack ...

@HarshCasper
HarshCasper merged commit 0747952 into main Sep 4, 2026
1 of 2 checks passed
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.

2 participants