Correct exit code for failed function invocation #2836

JoeStanton · 2016-12-01T10:31:16Z

What did you implement:

Exit with a status code that reflects the success/failure of the invoked Lambda function. This is useful when sls invoke is used as part of a shell script.

I'm not sure if there's a better place for this logic?

Todos:

Write tests
Write documentation (N/A)
Fix linting errors
Make sure code coverage hasn't dropped
Provide verification config/commands/resources (sls invoke <x>)
Enable "Allow edits from maintainers" for this PR
Change ready for review message below

Is this ready for review?: YES

…ingle quotes

Add CloudWatchEvent rule name and description fields

Fix order of CLI docs

separate commands into separate pages

added a note about deploying function versions

…i-doc Split up config CLI doc

fixed bug where you can now invoke without any input data

…etrics-support Add service wide metrics support

update csharp example

…-invoke Fix stdin docs for invoke

Lambda has been moved to have it's own permanent free tier. It is no longer part of the Expiring offers.

Update apigateway.md

Updating section on AWS pricing

…rless into redbadger-invoke-exit-code

mthenw · 2016-12-14T09:07:11Z

Ups. I pushed to wrong branch. Sorry for that :/ I will create separate PR with @JoeStanton commits cherry-picked.

Anthony Mendonca and others added 30 commits November 26, 2016 21:50

adding replaceAll to turn any escaped single quotes back to regular s…

e724b81

…ingle quotes

Fix serverless#2784 - slstats error

5a1539f

fix serverless#2784 - fixing lint errors

665400e

fix bug in event source mapping with custom IAM role

edd923f

Add support for directories in url of install command

0e274d4

remove request and response config when using proxy integration

c7c3218

Add core metrics plugin

b05cac7

Add AwsMetrics plugin

83900b3

Add plugins to Plugins.json file

ba27669

Add documentation

2f4b510

Add automatic Period setting and remove period option

80b12f6

Fix typo in test description

f249fa1

Add translation of human friendly startTime syntax

628a55e

Update docs about startTime and endTime

45b6695

Update tests for default value setting of startTime and endTime

5994360

Update documentation for default valued of startTime and endTime

d074078

add init command for easy onboarding

af35ee3

fix tests

3814837

fix linting issues

42ee9eb

fix linting error

efdbe2a

remove unused dependency

693fceb

add support for setting multiple profiles

da19c9e

fix linting errors

08769fc

fix linting error

db23598

Rename from init to setup

6cbe7d8

Split up setup into two plugins

88ce330

Add validation and test coverage for setup plugin

3377108

Add tests for constructor of AwsSetup

ce82aae

Minor refactorings

114f54e

Add tests for setupAws() method

1bfa6a9

mthenw and others added 27 commits December 12, 2016 10:28

Merge pull request serverless#2910 from horike37/issue-2832

e2f5c06

Add CloudWatchEvent rule name and description fields

Fix order of CLI docs

441ae76

Merge pull request serverless#2920 from serverless/fix-order-of-cli-docs

dfac939

Fix order of CLI docs

added a note about deploying function versions

577f379

separate commands into separate pages

c27e01f

Merge pull request serverless#2926 from serverless/doc-updates

c41a39c

separate commands into separate pages

Merge pull request serverless#2923 from serverless/eahefnawy-patch-1

4ffa47f

added a note about deploying function versions

Add service wide metrics support

5e9116d

Split up config CLI doc

fd03e55

Merge pull request serverless#2927 from serverless/split-up-config-cl…

cf18127

…i-doc Split up config CLI doc

fixed bug where you can now invoke without any input data

142946e

handling strings

08009c2

Merge pull request serverless#2928 from serverless/invoke-stdin-bug

cee63f1

fixed bug where you can now invoke without any input data

Update examples with more "real world" dates

df8997e

Refactor functions into reusable util functions

a8540b3

Remove utils file and move helper functions into class file

443865f

Merge pull request serverless#2922 from serverless/add-service-wide-m…

007be3d

…etrics-support Add service wide metrics support

update csharp example

52de4e6

Fix stdin docs for invoke

e069536

Merge pull request serverless#2932 from serverless/update-csharp

0a032f4

update csharp example

Merge pull request serverless#2933 from serverless/fix-stdin-docs-for…

2bdb312

…-invoke Fix stdin docs for invoke

Update apigateway.md

876c1d8

Updating section on AWS pricing

cde9c2d

Lambda has been moved to have it's own permanent free tier. It is no longer part of the Expiring offers.

Merge pull request serverless#2934 from nicka/patch-3

16ac810

Update apigateway.md

Merge pull request serverless#2935 from TheMcMurder/patch-1

ed591ff

Updating section on AWS pricing

Merge branch 'invoke-exit-code' of https://github.com/redbadger/serve…

c55a3af

…rless into redbadger-invoke-exit-code

fix lint errors

be06c47

mthenw closed this Dec 14, 2016

mthenw mentioned this pull request Dec 14, 2016

Fix invoke exit code #2942

Merged

6 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Correct exit code for failed function invocation #2836

Correct exit code for failed function invocation #2836

JoeStanton commented Dec 1, 2016 •

edited

mthenw commented Dec 14, 2016

Correct exit code for failed function invocation #2836

Correct exit code for failed function invocation #2836

Conversation

JoeStanton commented Dec 1, 2016 • edited

What did you implement:

Todos:

mthenw commented Dec 14, 2016

JoeStanton commented Dec 1, 2016 •

edited