checkov fixes

kunduso · Sep 11, 2024 · 354cbe1 · 354cbe1
1 parent fa0d4ad
commit 354cbe1
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
diff --git a/kms.tf b/kms.tf
@@ -3,7 +3,6 @@ resource "aws_kms_key" "encryption_rds" {
   enable_key_rotation     = true
   description             = "Key to encrypt secret"
   deletion_window_in_days = 7
-  #checkov:skip=CKV2_AWS_64: Not including a KMS Key policy
 }
 #https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/kms_alias
 resource "aws_kms_alias" "encryption_rds" {

diff --git a/security_group.tf b/security_group.tf
@@ -11,8 +11,6 @@ resource "aws_security_group" "rds" {
   tags = {
     "Name" = "${var.name}-rds-sg"
   }
-  # checkov:skip=CKV2_AWS_5: "Ensure that Security Groups are attached to another resource"
-  # This security group is attached to the Amazon ElastiCache Serverless resource
 }
 #https://registry.terraform.io/providers/hashicorp/aws/latest/docs/resources/security_group_rule
 resource "aws_security_group_rule" "ingress_rds_sg" {