Zappa icon indicating copy to clipboard operation
Zappa copied to clipboard

Support AWS Lambdas ephemeral storage setting (#1120)

Open gbartosz opened this issue 2 years ago • 3 comments

Description

Since march 2022 AWS allows for setting custom ephemeral storage size for Lambda (a storage mounted in a /tmp directory). Boto also supports this setting since v1.21.26. This PR extends zappa_settings.json with ephemeral_storage key with default value of {"Size": 512}.

GitHub Issues

https://github.com/zappa/Zappa/issues/1120

gbartosz avatar Apr 19 '22 07:04 gbartosz

Hi, Can we add it to next release?

elip-OE avatar Jul 03 '22 07:07 elip-OE

@gbartosz can you please run the above? many thanks!

elip-OE avatar Aug 24 '22 04:08 elip-OE

Coverage Status

Coverage increased (+0.01%) to 74.359% when pulling 3eebbdc7b4caa2897bb7afec060c66aeb6df2476 on gbartosz:feature/ephemeral_storage into 048fbe49050b572c7f04d82c15d8fe32ceb6236a on zappa:master.

coveralls avatar Sep 07 '22 03:09 coveralls

@elip-OE

Can you add a testcase confirming that the given cli value is properly changed and doesn't exceed the max configurable ephemeral_storage value? (throws proper error

monkut avatar Nov 05 '22 02:11 monkut

Hi @gbartosz @elip-OE @mdw123 what is the status of this PR? I'm happy to merge it if we get the PR comment by @monkut in there. (I've removed the next-release-candidate label because it's not a priority for the upcoming release, but we can get it done in the next or the one after, if this is solved 🤩)

hellno avatar May 03 '23 10:05 hellno

Hi @hellno ! I'm really want this feature to be released, but I can't write a testcase. It would be great if someone in the team can do it instead.

In case this feature is not in priority, you can add an extra conf that goes as-is into the aws API, which will be fully flexible.

elip-OE avatar May 03 '23 14:05 elip-OE

Hi @hellno @monkut , I'm also quite keen on this feature. Given this PR seems to have gone stale, i've opened a new PR with the requested tests and validation here: https://github.com/zappa/Zappa/pull/1259

Meepit avatar Jul 27 '23 19:07 Meepit

@Meepit Thanks for the PR update, closing this one.

monkut avatar Aug 16 '23 03:08 monkut