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

enable clouldwatch log exports for aurora postgresql #279

Merged
merged 2 commits into from
Jan 21, 2021

Conversation

srm78
Copy link
Contributor

@srm78 srm78 commented Jan 20, 2021

Enable logs for aurora postgresql

@srm78 srm78 requested a review from ryanking January 20, 2021 23:30
@srm78 srm78 requested a review from edulop91 as a code owner January 20, 2021 23:30
@@ -26,6 +26,7 @@ module "aurora" {
rds_cluster_parameters = var.rds_cluster_parameters
iam_database_authentication_enabled = var.iam_database_authentication_enabled
performance_insights_enabled = var.performance_insights_enabled
enabled_cloudwatch_logs_exports = ["audit", "error", "general", "slowquery", "postgresql"]
Copy link
Contributor

Choose a reason for hiding this comment

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

This works for me, but would it be better to make it something that can be optional for other users?

Copy link
Contributor

@ryanking ryanking Jan 20, 2021

Choose a reason for hiding this comment

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

agreed, though we could add that later.

@srm78 srm78 merged commit ecdcc6f into main Jan 21, 2021
@srm78 srm78 deleted the srm78/enable-logs-postgres branch January 21, 2021 18:55
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.

3 participants