[docs] [services] [9/n] Change docs and script to start docker-composed based localstack
AbandonedPublic
Actions

Authored by will on Jan 5 2024, 12:09 PM.

Details

Reviewers

bartek
varun
ashoat

Summary

This adds a script to run docker-compose based localstack (localstack version 2.3.2) instead of Nix-based localstack (1.4.0).
Additionally, the docs are changed to reflect the new process of starting localstack. This will be temporary until localstack can be updated.

Depends on D10501

Test Plan

Tested usage of script

Diff Detail

Repository

rCOMM Comm

Branch

identity-search

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

will created this revision.Jan 5 2024, 12:09 PM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJan 5 2024, 12:09 PM

will retitled this revision from [docs] [services] Change docs and script to start docker-composed based localstack to [docs] [services] [9/n] Change docs and script to start docker-composed based localstack.Jan 5 2024, 12:10 PM

will edited the summary of this revision. (Show Details)

will added a parent revision: D10501: [terraform] [services] [8/n] Add lambda trigger for reserved usernames.

This approach seems okay, but @atul had some interesting suggestions on ENG-5724 – wondering if you could spend 30-60min giving those a try. Would be great if there's an easy solution that lets us avoid this hacky approach

services/scripts/docker_localstack_up.sh
18	Capitalize Compose
19	Interesting, looks like this script (`init-local-cloud`) has been around for a while. Has it always basically "worked"?
21	Here too

will added inline comments.Jan 5 2024, 12:31 PM

services/scripts/docker_localstack_up.sh
19	Yeah. It's under the section in docs for running in sandbox https://github.com/CommE2E/comm/blob/41a3d5e5d0b4b3dea1d5937bea49949c771727e8/docs/dev_services.md?plain=1#L80