Skip to content
This repository has been archived by the owner on Jul 3, 2024. It is now read-only.

Remove S3 dependency and use EFS for db.json #9

Open
keithcom opened this issue Jul 22, 2020 · 2 comments
Open

Remove S3 dependency and use EFS for db.json #9

keithcom opened this issue Jul 22, 2020 · 2 comments

Comments

@keithcom
Copy link
Contributor

keithcom commented Jul 22, 2020

Update terraform to remove the S3 resource and use EFS to access the db.json file

Notes

@iandvt
Copy link
Contributor

iandvt commented Oct 14, 2020

@ryan-roemer && @kevinmstephens - Should this issue be closed? If we migrated from S3 to EFS then we would need to find a replacement for lowdb-adapter-aws-s3.

@ryan-roemer
Copy link
Member

ryan-roemer commented Oct 14, 2020

I don't think we'd need a replacement per set for lowdb-adapter-aws-s3 as we'd be using local (well, network-mounted) FS where db.json already works in https://www.npmjs.com/package/json-server via local lowdb.

I think a fair question for potentially closing this issue is whether or not the complexity of EFS is larger than S3 and we should just stick with S3 to keep this learning project as easy as possible for folks learning/experimenting with both sls and tf.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants