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 AWS RDS Read/Write IOPS alarms #16

Merged
merged 1 commit into from
Feb 17, 2024

Conversation

s4ros
Copy link
Contributor

@s4ros s4ros commented Jun 6, 2022

No description provided.

@ondraz
Copy link

ondraz commented Sep 5, 2023

Hi, I would appreciate this PR to be merged in, need it to SOC2. Thanks a lot.

ok_actions = var.actions_ok

dimensions = {
DBInstanceIdentifier = "${var.prefix}rds-${var.db_instance_id}-write-iops-too-high"
Copy link

Choose a reason for hiding this comment

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

this should be just var.db_instance_id

ok_actions = var.actions_ok

dimensions = {
DBInstanceIdentifier = "${var.db_instance_id}-read-iops-too-high"
Copy link

Choose a reason for hiding this comment

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

this should be just var.db_instance_id

@@ -193,3 +193,42 @@ resource "aws_cloudwatch_metric_alarm" "maximum_used_transaction_ids_too_high" {
alarm_actions = var.actions_alarm
ok_actions = var.actions_ok
}
Copy link

Choose a reason for hiding this comment

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

there is missing link to the db

  dimensions = {
    DBInstanceIdentifier = var.db_instance_id
  }

@lorenzoaiello lorenzoaiello merged commit 54e9f42 into lorenzoaiello:master Feb 17, 2024
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