Closed emileten closed 10 months ago
@emileten We should also consider adding dockerignore file with cdk.out
included at a minimum to reduce overall context size now that you are using Docker builds for all Lambdas.
I addressed comments ; I'll see if I can fit that into eoAPI and after confirming it works I'll fix conflicts and merge.
@vincentsarago FYI in the meanwhile (this took me forever 🙄) I did these changes (I updated the description):
This is tested in deployment and good to go, about to merge!
A large PR with several breaking changes.
⚠️ breaking change. bootstrapper runtime made configurable. This necessitated to 'merge' the
database
andbootstrapper
into a single construct to avoid JSII issues. With this, all apps have their runtime configurable.⚠️ breaking change. switched to
Function
andfromDockerBuild
for lambda functions, because the state ofPythonFunction
makes it very hard to add flexible configurability. Now, users can override any of the lambdaFunction
parameters they want ( code, but also memory, time out...).made VPC optional for lambdas
harmonized pgstac to 0.7.10 in default runtimes.
harmonized python to 3.11 in default runtimes.