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

feat: Add EventBridge and Step Function modules #38

Merged
merged 28 commits into from
Feb 21, 2023

Conversation

victorgu-github
Copy link
Collaborator

What does this PR do?

🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

Motivation

  • Resolves #

Test Results

Additional Notes

@victorgu-github victorgu-github marked this pull request as ready for review January 9, 2023 18:15
@victorgu-github victorgu-github requested a review from a team as a code owner January 9, 2023 18:15
@victorgu-github victorgu-github changed the title fix: avoid duplicate policy for emr container controller add eb and efn modules and new example Feb 9, 2023
@victorgu-github victorgu-github changed the title add eb and efn modules and new example feat: add eb and efn modules and new example Feb 9, 2023
@victorgu-github
Copy link
Collaborator Author

@askulkarni2 requested by Michael Gasch (product manager of EventBridge), I added eventbridge and sfn modules and added an example "event driven pipeline" to demo this new feature. I've tested locally. pls review. thanks.

@victorgu-github victorgu-github changed the title feat: add eb and efn modules and new example feat: add eb and efn modules Feb 15, 2023
@victorgu-github
Copy link
Collaborator Author

@askulkarni2 event-drive-pipeline example is removed. only keep the eb and sfn modules. pls review. thx

Copy link
Contributor

@askulkarni2 askulkarni2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@season1946 minor comment otherwise lgtm!

@@ -52,7 +52,7 @@ locals {
################################################################################

module "eks_blueprints" {
source = "github.com/aws-ia/terraform-aws-eks-blueprints?ref=v4.12.2"
source = "github.com/aws-ia/terraform-aws-eks-blueprints?ref=v4.18.1"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Latest is 4.24.0, can we please change it everywhere in this file?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done. @askulkarni2

@askulkarni2 askulkarni2 changed the title feat: add eb and efn modules feat: Add eventbridge and step-function modules Feb 15, 2023
@askulkarni2 askulkarni2 changed the title feat: Add eventbridge and step-function modules feat: Add EventBridge and Step Function modules Feb 15, 2023
@askulkarni2 askulkarni2 merged commit 411cdae into aws-ia:main Feb 21, 2023
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