.ci: Improve dockerfile to include caching #708
Conversation
There was a problem hiding this comment.
@seemethere thanks for you contribution! Just checked the chagen. First got all kind of erros. Seems the cause is the javascript heep space. Updated the memory allocated by docker from 2GB to 4GB and all works fine.
Please can you add a note about the required memory?
Done! |
|
@seemethere Can you da me a small favour and rebase your branch due to yesterdays release? |
Improves dockerfile / .ci/build.sh script to output lambda zips in a central location as well as adds buildkit caching to the docker build so that you don't have to re-download the node_modules of each zip directly each time you build with .ci/build.sh This also makes improvements in that if there are only changes in one directory then the cache will remain and the remaining lambdas will not be re-built. Signed-off-by: Eli Uriegas <[email protected]>
seemethere
force-pushed
the
improve_ci_build_script
branch
from
2c58360 to
387d6a7
Compare
Done! |
* .ci: Improve dockerfile to include caching Improves dockerfile / .ci/build.sh script to output lambda zips in a central location as well as adds buildkit caching to the docker build so that you don't have to re-download the node_modules of each zip directly each time you build with .ci/build.sh This also makes improvements in that if there are only changes in one directory then the cache will remain and the remaining lambdas will not be re-built. Signed-off-by: Eli Uriegas <[email protected]> * Add comment about memory size
Improves dockerfile / .ci/build.sh script to output lambda zips in a
central location as well as adds buildkit caching to the docker build so
that you don't have to re-download the node_modules of each zip directly
each time you build with .ci/build.sh
This also makes improvements in that if there are only changes in one
directory then the cache will remain and the remaining lambdas will not
be re-built.
I guess an open question as well should be if there should be an additional github actions wofklow added to this?
Signed-off-by: Eli Uriegas [email protected]