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

:vpc configuration for lambda #48

Merged
merged 3 commits into from Oct 21, 2016

Conversation

Projects
None yet
2 participants
@zrzka
Contributor

zrzka commented Oct 21, 2016

Tested scenarions ...

Create function without vpc

  • --vpc-config not passed

Create function with vpc

  • --vpc-config passed

Update function

  • update-function-configuration called only if it differs (VPC & rest)
  • --vpc-config not passed to update-function-code

@zrzka zrzka referenced this pull request Oct 21, 2016

Closed

VPC configuration #47

@moea

This comment has been minimized.

Show comment
Hide comment
@moea

moea Oct 21, 2016

Member

Thanks! Will take a look and give it a run.

Member

moea commented Oct 21, 2016

Thanks! Will take a look and give it a run.

@moea

This comment has been minimized.

Show comment
Hide comment
@moea

moea Oct 21, 2016

Member

Looks good, will merge and deploy a snapshot.

Member

moea commented Oct 21, 2016

Looks good, will merge and deploy a snapshot.

@moea moea merged commit a6d43cf into nervous-systems:master Oct 21, 2016

@moea

This comment has been minimized.

Show comment
Hide comment
@moea

moea Oct 21, 2016

Member

Done

Member

moea commented Oct 21, 2016

Done

@zrzka zrzka referenced this pull request Jan 5, 2017

Closed

Add support for "new" Lambda env variables #56

2 of 2 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment