-
Notifications
You must be signed in to change notification settings - Fork 6
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
feature(terraform): init terraform module #83
base: main
Are you sure you want to change the base?
Conversation
hey @pecigonzalo regarding your comment previously:
The problem with Subscriber/Unsubscriber module is that it created resources that Clouformation is not aware of. I wanted to avoid this pattern with terraform, and make it easier to find all the created subscription logs and easily maintain a list of log group names. I agree with you there should be a better way to handle the dynamic log groups. Wondering if current setup + listener only would fit, wdyt? |
I understand the reasoning. The challenge is that the dynamic nature of resource like cloudwatch log groups does not fit well with Terraform rather static and stateful nature. If AWS supported a single global "subscription config" for all CWLG (CloudWatch Log Group) I would certainly prefer to configure this via Terraform. For example, as a "Cloud Engineer", as soon as a new CWLG is created I want to ensure its logs are propagated the logging pipeline. I think the idea of having a CWLG controller, that takes care of configuring them all (kind of what we did with the lambda, and something I've implemented myself at other places) is more maintainable, fit for porpouse and works well in most environments.
So instead of having two ways of doing this, one that is a bit more subpar than the other (terraform), I would advocate for a single recommended approach. Particularly when anyone needing to configure this via Terraform only needs 1 or 2 resources at most, so its not something for which we add a ton of value in our module or they might get more value out of the settings of the module they might using already (be that a dedicated CWLG module or a Lambda or RDS module that already contains some CWLG settings). |
Co-authored-by: Gonzalo Peci <pecigonzalo@users.noreply.github.com>
Terraform modules gonza
67a38a5
to
a817184
Compare
a26e8c8
to
1998907
Compare
443af34
to
ef6f93e
Compare
ef6f93e
to
cc6ed3c
Compare
ac3948a
to
510cfb4
Compare
24b253b
to
dcdd46d
Compare
dcdd46d
to
da7e3a6
Compare
@a-khaledf this is ready to be released. Let's make sure to squash those commits into one. |
This PR creates a terraform module the creates the Forwarder lambda and allows the user to pass a list of log group names to create the needed Subscription filters.