-
Notifications
You must be signed in to change notification settings - Fork 24
Make 'grunt packaging' use a different tmp dir #17
Comments
Any thoughts @bigsnarfdude ? |
Looks like the grunt task is running because it gets called ... Running "associateStream:default" (associateStream) task Lemme look into it... |
The active call is the ... lambda.createEventSourceMapping(params, function(err, data) I will manually test this function...looks like the API just changed 2015-03-31 |
Hey Brian aka @BrianODell When you log into the AWS Console Lambda tab does it have something like: https://bigsnarf.files.wordpress.com/2015/07/screen-shot-2015-07-30-at-4-25-32-pm.png But with a Lambda Function named If no, then the last task didnt upload the Lambda properly and it didn't get registered. Hopefully the name is the same too. |
I do not have a lambda function named that. My only lambda function is one that I personally made. |
There are several ways to get the function registered up in AWS Lambda Service. One way is manually upload the zip file. Another is to use the aws lambda create-function And the third is the grunt script ... which is broken. Try uploading the function and either manually associate it to your Kinesis Stream or at the command line re-run Let me know how you make out...Thanks for checking out the code. |
Is the grunt script broken? I read this thread as Brian has changed the name of the Lambda function, so the hardcoded grunt scripts won't work for him... |
I had been playing with Lambda on my own before attempting to run this project. |
Ah got it, sorry. Sounds like our grunt uploader is broken then... |
I got it working! I noticed that I had no 'dist' folder, and managed to track it down to a bug in the 'packaging' task. See my pull request: #18 Thanks for all the responses. The project looks really great! |
Thanks @BrianODell! I'll close your PR and just reference the fix in this ticket so that you don't have to go through the whole CLA rigmarole. For fix: https://github.com/snowplow/aws-lambda-nodejs-example-project/pull/18/files Thanks again! |
Scheduled! |
It's my pleasure. Thanks for making this project. I intend to borrow lots of the grunt and ansible stuff from you. I love the setup of the project. |
Thanks for the kindwords @BrianODell ! |
I have some ideas I'm going to work on to tweak this project. Might there be a good place to chat about this project besides the comments of this issue? Thanks again :) |
Hey @BrianODell - sure thing, you can open a new thread in https://groups.google.com/forum/#!forum/snowplow-user Look forward to your thoughts! |
I was able to recreate the issue. I was able to verify this issue and was able to redirect the the target directory to _dist and pick up the _dist folder. I'm gonna try @BrianODell solution as it is more elegant. |
The output of my grunt commands:
The text was updated successfully, but these errors were encountered: