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

Added cloudwatch decompression and data message extraction #12

Merged
merged 4 commits into from
Feb 28, 2024

Conversation

tropnikovvl
Copy link
Contributor

@tropnikovvl tropnikovvl commented Feb 27, 2024

Why this is needed is described in this documentation.

The main caveat is that the feature cannot be enabled “in a simple way” on an already existing kinesis firehose (This PR will not affect existing instances with default settings in any way). This is described in the documentation.

In any case, this works great on the new Kinesis Firehose.

variables.tf Outdated Show resolved Hide resolved
@fdmsantos
Copy link
Owner

Thanks for this PR. I added one comment. I unable to test it now, but i will try to test it tomorrow and then approve PR

@fdmsantos fdmsantos added enhancement New feature or request wip Working in Progress labels Feb 27, 2024
@fdmsantos fdmsantos merged commit 7686e0e into fdmsantos:main Feb 28, 2024
2 of 24 checks passed
fdmsantos pushed a commit that referenced this pull request Feb 28, 2024
## [3.2.0](v3.1.0...v3.2.0) (2024-02-28)

### Features

* Merge pull request [#12](#12) from tropnikovvl/main ([7686e0e](7686e0e))
@fdmsantos
Copy link
Owner

Thanks very much for your collaboration on this repo.

@fdmsantos
Copy link
Owner

This PR is included in version 3.2.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request wip Working in Progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants