-
Notifications
You must be signed in to change notification settings - Fork 290
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
Stream logs from CloudWatch to S3 bucket in core-logging
account
#7705
Conversation
…lish before I write the new firehose module
…se can write to bucket
|
|
|
|
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
A reference to the issue / Description of it
#7607
How does this PR fix the problem?
core-logging
account, along with an SQS notification queue.How has this been tested?
Ran Terraform Plan locally, but this will only create the
core-logging
resources and then allow tests of the CloudWatch log streaming module before a later PR implements it.Deployment Plan / Instructions
Deploy through CI
Checklist (check
x
in[ ]
of list items)Additional comments (if any)
This PR should put things in place to allow me to then test streaming logs from CloudWatch log groups in, say,
core-vpc-sandbox
. It doesn't remove any existing use of kinesis, nor does it change any existing logging functionality.