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

Add option to specify target prefix when setting access logging on terraform state bucket #1437

Closed
dsecik opened this issue Nov 24, 2020 · 3 comments
Assignees
Labels
enhancement New feature or request

Comments

@dsecik
Copy link

dsecik commented Nov 24, 2020

In addition to the accesslogging_bucket_name configuration option introduced in #1400 allowing to specify target bucket for s3 access logs, it would be handy to have an ability to specify also the target prefix to override the default targetPrefix := "TFStateLogs/" This will add more flexibility to access logging configuration, for example to be able to send access logs from multiple terraform state buckets to single logging bucket and not mix them together.

@brikis98
Copy link
Member

Seems worthwhile. PR to add this is welcome!

@ina-stoyanova ina-stoyanova self-assigned this Jan 20, 2021
@ina-stoyanova
Copy link
Contributor

I'll pick this up this week!

@ina-stoyanova
Copy link
Contributor

Closing this issue. The feature has been implemented.

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

No branches or pull requests

4 participants