Skip to content
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

Terratest lambda cloudwatch trigger #9

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

alasseter-dds
Copy link
Contributor

@pjdufour-dds when you get the chance can you take a look at this branch. working to set up terratest module for a scheduled lambda function based on these examples:

I'm uncertain about the following:

  • starting from the simple-test branch as a baseline, I think the only two files that need updating are _test.go and main.tf -- is it also necessary to update handler.py? (I think not)
  • I believe that _test.go requires just a few updates, and I think I've made them successfully but would love your feedback.
  • The blocker for me is main.tf -- lots of stuff in the examples I linked above don't seem to translate. For example, lambda_iam_policy is supposed to be required in data but we don't seem to be using it in our terratest modules. I don't get that.

@alasseter-dds
Copy link
Contributor Author

I updated this branch with the latest PR to master, so it now reflects the most up-to-date changes in module-template.

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

Successfully merging this pull request may close these issues.

1 participant