jetbridge / cdk-nextjs

Deploy a NextJS application using AWS CDK
https://constructs.dev/packages/cdk-nextjs-standalone
Apache License 2.0
260 stars 43 forks source link

CustomCDKBucketDeployment lambda - Exited with SIGKILL 9 #168

Closed PanosGkionis closed 7 months ago

PanosGkionis commented 8 months ago

We're in the process of creating a substantial application comprising over 230 pages, resulting in a build size exceeding the default 512mb allocation for ephemeral storage. Is it possible to introduce a property for customizing the default ephemeral storage of the CustomCDKBucketDeployment lambda?

Currently, we've manually adjusted the ephemeral storage through the AWS console, and our deployments are presently successful. However, it's our preference to incorporate this configuration into our stack for a more streamlined and automated process.

image

bestickley commented 8 months ago

@PanosGkionis, glad to hear you're pushing this construct to the limit. Yes, I will prioritize the overrides feature which I've discussed over here which will support this. If you're motivated to begin the work, please do! Happy to answer any questions to support you in that effort.

PanosGkionis commented 8 months ago

@bestickley , Thanks for your prompt response. I'm happy about the progress with the construct and appreciate you prioritizing the overrides feature. I'll certainly try to help depending on my workload. Let me know if there is any branch with ongoing work for that.

bestickley commented 8 months ago

@PanosGkionis, you're welcome! There isn't a current branch for the work. I'll let you know when I start one. Hopefully by end of this week.

PanosGkionis commented 8 months ago

@bestickley Hi again, is there ongoing work on this current issue?

bestickley commented 8 months ago

Hi @PanosGkionis, not yet. I have 2 other higher priority issues that popped up I need to get through first.

PanosGkionis commented 8 months ago

@bestickley Thanks, I appreciate your quick answer

bestickley commented 7 months ago

@PanosGkionis, please checkout the new overrides props. It will allow for the customization you need. Would appreciate any feedback you have on the API. Thank you 🙏

PanosGkionis commented 7 months ago

@bestickley I am going to try these changes soon and I will let you know! thank you in advance