Skip to content

Conversation

@rasathus
Copy link
Contributor

@rasathus rasathus commented Nov 4, 2019

Revised the usage example to show the deployment of the layer from SAR and reference of the layer from the function in the template.

@gojko
Copy link
Contributor

gojko commented Nov 11, 2019

hi - thanks for this. the example template also works as a test script for the build, so it's not really good to use SAR linkage here since it requires a published layer. However, this would be lovely as an additional script (eg example-sar.yaml).

@rasathus rasathus force-pushed the master branch 2 times, most recently from cc19627 to f7ab1bb Compare November 12, 2019 11:21
…SAR and reference of the layer from the consuming function.
@rasathus
Copy link
Contributor Author

Hi. Thanks for the feedback. I've modified the PR to deploy an additional template including an example of deployment of the layer from the published Serverless Application Repository, and its subsequent use.

Apologies for the messy rebase, but I wanted to ensure a clean commit for merging.

Thanks

@gojko gojko merged commit aed8974 into serverlesspub:master Nov 12, 2019
@gojko
Copy link
Contributor

gojko commented Nov 12, 2019

thanks, looks great

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants