-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Aws lambda codestart #34129
base: main
Are you sure you want to change the base?
Aws lambda codestart #34129
Conversation
another (future?) option would be to make that script a plugin that the extension could register and then it would be workable on windows too. |
Would it allow customizing the script though by the user? |
You like the idea of depending on a codestart for basic functions to work? i.e. deploy, run, invoke, delete/undeploy? |
I also don't like the idea of adding more to the pom. Extensions should be doing this automatically with config overrides being available. Can we have maven/gradle publish java compile version as an atttribute that extensions could find? |
FYI: Azure Functions is in a good place. Going to be working on Lambda next to add quarkus:run, quarkus:deploy, and @QuarkusIntegrationTest features like I have for Azure Functions. Thinking of adding a quarkus:undeploy generic action too. The only thing missing really is quarkus:invoke, but only Lambda seems to have that action. |
I haven't really put a lot of thought into this. Not sure what you have in mind as a roadmap. I just noticed most of the content is hardcoded and would be good if a user could adjust it w/o a rebuild re-generating it all. So just based on that I'd put it under resources. |
@patriot1burke WDYT about extracting the
manage.sh
and other resources from the extension to the the AWS Lambda codestart?It would be generated in the project's resources directory and copied to
target/aws
directory with filtering enabled during the build. It would allowmaven.compiler.release
that would set theruntimeXX
for the function;